-
Notifications
You must be signed in to change notification settings - Fork 202
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
Remove fsw/mission_inc and fsw/platform_inc (classic build support) #344
Comments
Imported from trac issue 313. Created by jhageman on 2019-07-12T15:16:50, last modified: 2019-08-14T14:08:00 |
@jphickey @acudmore what are our remaining dependencies on these directories? Could one of you list them here? |
I don't believe CFE has any dependencies on these at all. But sometimes apps directly include them (e.g. SAMPLE_APP or the LAB apps maybe?). Should be an obvious build failure if there was a dependency though. My recommendation is to take these out, and grep for "mission_inc" or "platform_inc" globally across the code base to verify that nothing directly references it. If those strings don't exist in any source files and the code builds and runs OK, can be pretty sure that nothing needed them. |
Fix #344, Remove mission/platform include dirs
Directories are only used by classic build and duplicate defines in sample_defs. Should be removed.
Targeted for 7.0 since it will break classic build (deprecated in 6.7).
The text was updated successfully, but these errors were encountered: