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.

Sign up for GitHub

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

Jump to bottom

Make command secondary header endian agnostic (match GSFC spec) #65

Closed
skliper opened this issue Mar 25, 2020 · 0 comments · Fixed by #66 or #69
Closed

Make command secondary header endian agnostic (match GSFC spec) #65

skliper opened this issue Mar 25, 2020 · 0 comments · Fixed by #66 or #69
Assignees
@skliper
Labels
bug Something isn't working
Milestone
Bootes

Comments

@skliper
Copy link
Contributor

skliper commented Mar 25, 2020

Is your feature request related to a problem? Please describe.
See nasa/cFE#297

Command secondary header has always been defined by GSFC as endian agnostic, code behavior didn't match definition.

Describe the solution you'd like
Make command utility (cmdUtil) command secondary header endian agnostic.

Describe alternatives you've considered
None

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper changed the title Command secondary header now endian agnostic Make command secondary header endian agnostic (match GSFC spec) Mar 25, 2020
@skliper skliper self-assigned this Mar 25, 2020
skliper added a commit to skliper/cFS-GroundSystem that referenced this issue Mar 25, 2020
@skliper
Fix nasa#65, CCSDS sec cmd hdr consistancy
6a981d9 
Implement CCSDS command secondary header such that it
is endian agnostic.  Really just sets command code
appropriately.
@skliper skliper mentioned this issue Mar 25, 2020
Merged
astrogeco added a commit that referenced this issue Apr 6, 2020
@astrogeco
Merge pull request #66 from skliper/fix65-sec-cmd-hdr
a9c3cab 
Fix #65, CCSDS Command Secondary Header Endian Agnostic
@skliper skliper added this to the 2.2.0 milestone Apr 8, 2020
@astrogeco astrogeco mentioned this issue Apr 9, 2020
Merged
@astrogeco astrogeco added the bug Something isn't working label Oct 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@skliper skliper

Labels
bug Something isn't working
Projects
None yet
Milestone
Bootes
2 participants
@skliper @astrogeco