Skip to content

Conversation

@Brian-Campuzano
Copy link
Collaborator

@Brian-Campuzano Brian-Campuzano commented May 13, 2025

This fixes the include path change that was made in nasa/fprime#3563.

Without this fix I get numerous compilation errors when running fpp-to-cpp/test/check-cpp

compiling AArrayAc.cpp with default guards
In file included from /home/campuzan/projects/fprime/Fw/Types/BasicTypes.hpp:14,
                 from /home/campuzan/projects/fprime/Fw/FPrimeBasicTypes.h:24,
                 from /home/campuzan/projects/fprime/Fw/FPrimeBasicTypes.hpp:21,
                 from /home/campuzan/projects/fprime/Fw/Types/Assert.hpp:4,
                 from AArrayAc.cpp:7:
/home/campuzan/projects/fprime/Fw/Types/BasicTypes.h:41:10: fatal error: config/FPrimeNumericalConfig.h: No such file or directory
   41 | #include <config/FPrimeNumericalConfig.h>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

@bocchino
Copy link
Collaborator

Thanks, good catch!

Remove unused -I option
Copy link
Collaborator

@bocchino bocchino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@bocchino bocchino merged commit 824d14e into main May 13, 2025
11 checks passed
@bocchino bocchino deleted the include_fix branch May 13, 2025 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants