-
Notifications
You must be signed in to change notification settings - Fork 236
Description
Is your feature request related to a problem? Please describe.
After the changes introduced by nasa/osal#312, the OSAL CMake script will no longer force setting of CMAKE_C_FLAGS directly in the CFE build by overwriting this value (which is good).
Instead, any required compile definitions and related information will be conveyed in the more appropriate manner, through the INTERFACE_COMPILE_DEFINITIONS and INTERFACE_INCLUDE_DIRECTORIES on the "osal" library target.
Describe the solution you'd like
To work with this change, the CFE needs to explicitly check these properties on the osal target and use the values. By setting a directory-scope property at the top level, it will apply to all code.
Describe alternatives you've considered
CMake does this automatically so long as the executable directly links with OSAL. However, this doesn't apply to the apps which are built as a MODULE. This is why it is easier to set the directory property so its all-inclusive.
Requester Info
Joseph Hickey, Vantage Systems, Inc.