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 #828, spacecraft id hex, add to Bootes #848

Closed
wants to merge 15 commits into from

Conversation

astrogeco
Copy link
Contributor

Describe the contribution
Fix #828 - set spacecraft ID in sample targets.cmake to 0x42

Testing performed
Nominal build and test

Expected behavior changes
Spacecraft ID back to historical value (some toolchains depend on it). Verified at PSP startup, reported as 66 (0x42).

System(s) tested on

  • Hardware: cFS Dev Server
  • OS: Ubuntu 18.04
  • Versions: bundle main (w/ cfe/osal main) + this commit

Additional context
None

Third party code
None

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

CDKnightNASA and others added 15 commits July 31, 2020 12:26
 nasa#787
 - cES1005.1: Remove application load address from start app command
 - cTBL6003.1: Clarify table validation pior to activate
 - cEVS3103.4, cEVS3103.5, cEVS3001: Remove SB qualifier
 nasa#587
 - cSB4344: Added get packet type API requirement
 - cSB4345: Added message ID validation API requirement
Provides ability to load UT assert as a CFE app, and an example
of using this to test some basic CFE ES functions.
…_deprecated

fix nasa#759 - deprecates GetLastSenderId()
Provides ability to load UT assert as a CFE app, and an example
of using this to test some basic CFE ES functions.

This introduces a separate library for the basic UT assert functions
(cfe_assert) and an app that executes the test (cfe_testrunner) rather
than combining these into a single module.
Revert "Fix nasa#778, add cfe assert and example lib"
@astrogeco
Copy link
Contributor Author

Closing, will cherry-pick last commit into new branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants