Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #2391, add cfe to prefix list for msgids.h and platform_cfg.h #2392

Merged
merged 1 commit into from
Jul 12, 2023

Conversation

jphickey
Copy link
Contributor

Checklist (Please check before submitting)

Describe the contribution
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

Fixes #2391

Testing performed
Build and run with stakeholder configuration

Expected behavior changes
A file named cfe_msgids.h is picked up by the build system, as expected.

System(s) tested on
Debian

Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

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
@dmknutsen dmknutsen self-requested a review July 12, 2023 13:59
@dzbaker dzbaker merged commit f6d7f8c into main Jul 12, 2023
40 checks passed
@dzbaker dzbaker added the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label Jul 12, 2023
@jphickey jphickey deleted the fix-2391-cfe-include-wrappers branch July 14, 2023 17:20
@dzbaker dzbaker added CCB:Approved Indicates code review and approval by community CCB and removed CCB:Ready Ready for discussion at the Configuration Control Board (CCB) CCB:FastTrack labels Jul 20, 2023
@dmknutsen dmknutsen added this to the Equuleus milestone Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CCB:Approved Indicates code review and approval by community CCB enhancement Equuleus-rc1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The include file wrapper script does not pick up a "cfe" prefix file
4 participants