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 #86, split configuration #89

Merged
merged 1 commit into from
May 4, 2023
Merged

Conversation

jphickey
Copy link
Contributor

Checklist (Please check before submitting)

Describe the contribution
Use the "generate_config_includefile" for all headers that can come from alternate source (e.g. generated or mission-supplied).

Fixes #86

Testing performed
Build and run LC in variety of configurations

Expected behavior changes
Better organization, more modular

System(s) tested on
Debian

Additional context
See nasa/HS#87 for reference - this is basically the same change applied to LC, making it consistent.

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

@jphickey jphickey force-pushed the fix-86-split-cfg branch 4 times, most recently from 0e85b52 to 6acc627 Compare April 28, 2023 14:35
Use the "generate_config_includefile" for all headers that can come from
alternate source (e.g. generated or mission-supplied).
@jphickey
Copy link
Contributor Author

Fixed! Now this adheres to the naming convention in nasa/cFE#2306, can review at next CCB.

@dzbaker dzbaker merged commit 17987cc into nasa:main May 4, 2023
@jphickey jphickey deleted the fix-86-split-cfg branch May 5, 2023 14:41
@dmknutsen dmknutsen added this to the Equuleus milestone May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Split "platform_cfg" into external and internal components
4 participants