-
Notifications
You must be signed in to change notification settings - Fork 203
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
cFE Integration candidate: Caelum-rc4+dev49 #2325
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Verify the JSON files generated from EDS match the expected content and CTF-based tests also pass. This required a few touchups to XML files to adjust sizes and/or padding.
If the passed-in table name does not use a qualifier suffix, then generate one by computing a hash of the context information. This ensures that the generated intermediate target names will be unique.
Organize ES header file contents according to the convention
Organize SB header file contents according to the convention
Organize TBL header file contents according to the convention
Organize TIME header file contents according to the convention
Organize EVS header file contents according to the convention
Organize FS header file contents according to the convention
Organize CORE_API header file contents according to the convention
Organize CORE_PRIVATE header file contents according to the convention
Organize CFE_TESTCASE header file contents according to the convention
Corrects the paths and references in the documentation build to reflect the header name changes. Removes the overrides from "sample_defs" as these are no longer strictly required, user only needs to override if they really need to change something.
Fix #2320, reorganize headers
Fix #2310, Update XML files per interoperability testing
Fix #2315, uniqueness of tbl target names
** mixed-endian environments. This will become obsolete once EDS is | ||
** available and the CFE time tone message is defined. | ||
*/ | ||
#undef CFE_PLATFORM_TIME_CFG_BIGENDIAN |
Check notice
Code scanning / CodeQL
Use of #undef
The #undef directive shall not be used.
** mixed-endian environments. This will become obsolete once EDS is | ||
** available and the CFE time tone message is defined. | ||
*/ | ||
#undef CFE_PLATFORM_TIME_CFG_BIGENDIAN |
Check notice
Code scanning / CodeQL
Disallowed preprocessor use
This preprocessor directive is not allowed.
This is an optional component and thus may not be part of the build. Hotfix based on workflow testing.
Fix #2320, reorganize headers
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Checklist (Please check before submitting)
Describe the contribution
Testing performed
cFS Bundle Checks
cFE Checks
Expected behavior changes
See PRs
System(s) tested on
Ubuntu 20.04
Additional context
Add any other context about the contribution here.
Third party code
If included, identify any third party code and provide text file of license
Contributor Info - All information REQUIRED for consideration of pull request
@jphickey