The include file wrapper script does not pick up a "cfe" prefix file #2391
Comments
jphickey
added a commit
that referenced
this issue
Jul 12, 2023
This allows the script to pick up the file if it was named simply "cfe_msgids.h" or "cfe_platform_cfg.h", rather than insisting it be named with a platform-specific prefix
2 tasks
dzbaker
added a commit
that referenced
this issue
Jul 12, 2023
Fix #2391, add cfe to prefix list for msgids.h and platform_cfg.h
dzbaker
added a commit
that referenced
this issue
Jul 12, 2023
Fix #2391, add cfe to prefix list for msgids.h and platform_cfg.h
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
The include file wrapper script looks for a name-qualified file to use for e.g.
cfe_msgids.handcfe_platform_cfg.h. This means that the file provided in the "defs" directory needs to be named based on the config, for exampledefault_platform_cfg.horcpu1_platform_cfg.h.. then this file gets wrapped ascfe_platform_cfg.hfor C source files to use.This pattern has some advantages, in that only one copy of a file actually named
cfe_platform_cfg.hwill exist on the disk, therefore it is clear(er) which one is actually getting picked up.However, this name difference/mapping isn't well documented and can be confusing or unexpected to some users.
To Reproduce
Create a customized
cfe_msgids.hfile in the defs dir, and run "make prep"Observe that the default msgids from CFE were picked up, not the customized version.
Expected behavior
The customized version should be picked up.
System observed on:
Debian
Reporter Info
Joseph Hickey, Vantage Systems, Inc.
The text was updated successfully, but these errors were encountered: