You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
In Caelum when setting set(MISSION_INCLUDE_CCSDSEXT_HEADER TRUE) to use extended headers, I get a build error due to duplicate typedef of the MSG types:
cfe/modules/msg/option_inc/default_cfe_msg_hdr_priext.h:83:3: error: conflicting types for ‘CFE_MSG_Message_t’
83 | } CFE_MSG_Message_t;
| ^~~~~~~~~~~~~~~~~
cfe/modules/core_api/fsw/inc/cfe_msg_api_typedefs.h:104:31: note: previous declaration of ‘CFE_MSG_Message_t’ was here
104 | typedef union CFE_MSG_Message CFE_MSG_Message_t;
| ^~~~~~~~~~~~~~~~~
To Reproduce
Enable extended headers in Caelum config, and build.
Expected behavior
Build should succeed.
Code snips
This is because the "typedef" cannot be in both places. In contrast, the standard header (non-extended) defines only the union/struct, not typedef'ed:
Do not double-typedef the CFE_MSG types when using extended headers.
This also corrects a call to UT_SetForceFail in the extended
header test, which was renamed.
Describe the bug
In Caelum when setting
set(MISSION_INCLUDE_CCSDSEXT_HEADER TRUE)
to use extended headers, I get a build error due to duplicate typedef of the MSG types:To Reproduce
Enable extended headers in Caelum config, and build.
Expected behavior
Build should succeed.
Code snips
This is because the "typedef" cannot be in both places. In contrast, the standard header (non-extended) defines only the union/struct, not typedef'ed:
cFE/modules/msg/option_inc/default_cfe_msg_hdr_pri.h
Lines 65 to 69 in 98f78e8
Whereas the extended header version has a typedef:
cFE/modules/msg/option_inc/default_cfe_msg_hdr_priext.h
Lines 63 to 67 in 98f78e8
System observed on:
Ubuntu
Reporter Info
Joseph Hickey, Vantage Systems, Inc.
The text was updated successfully, but these errors were encountered: