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 #147, Pad GUI headers to 64 bit boundary #148

Merged
merged 1 commit into from
Jan 25, 2021

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Jan 8, 2021

Describe the contribution
Fix #147 - rounds headers up to match nasa/cFE#1077

Testing performed
Tested cmds and tlm with a cFE using extended headers and without. 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
Copy link
Contributor

CCB:2021-01-13 APPROVED

Adds specific as opposed to generic offsets

@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:46
@astrogeco astrogeco merged commit 7f667e0 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 fix147-gui_padding 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 cmd/tlm padding support to GUI
2 participants