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 #144, Pad cmdUtil header to 64 bit boundary #145

Merged
merged 1 commit into from
Jan 25, 2021

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Jan 8, 2021

Describe the contribution
Fix #144 - rounds header up to match nasa/cFE#1077

Testing performed
Sent command to cFE with extended header, confirmed round up worked.

Expected behavior changes
Breaking change, behaves the same if cFE is run with nasa/cFE#1077

System(s) tested on

Additional context
None

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

@skliper skliper added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label Jan 8, 2021
@skliper skliper added this to the 3.0.0 milestone Jan 8, 2021
@astrogeco astrogeco added CCB:2021-01-13 and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Jan 13, 2021
@astrogeco astrogeco changed the base branch from main to integration-candidate January 25, 2021 17:52
@astrogeco astrogeco merged commit caa7e0b into nasa:integration-candidate Jan 25, 2021
astrogeco added a commit to nasa/cFS that referenced this pull request Jan 25, 2021
@skliper skliper deleted the fix144-cmdutil_pad branch February 1, 2021 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add command padding to cmdUtil
2 participants