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 #197, Update readme file for pc-rtems PSP and added a new readme file for RTEMS 5 #220

Merged
merged 1 commit into from Dec 9, 2020

Conversation

ghost
Copy link

@ghost ghost commented Nov 25, 2020

Describe the contribution
Fix #197

This change updates the README.txt file and adds a README_RTEMS_5.txt for the pc-rtems PSP. No code changes in the PSP were necessary to support RTEMS 5.1 for the pc-rtems PSP. The changes include removing references to the CEXP module loader, and describing the development environment setup for RTEMS 5.

Testing performed
No testing is required for these README changes, but I did create both the RTEMS 4.11 and RTEMS 5.1 development environments according to the README files. I also tested building and running the cFS bundle on both RTEMS 4.11 and RTEMS 5.1.

Expected behavior changes

  • No impact to behavior

System(s) tested on

  • Hardware: QEMU i686
  • OS: RTEMS
  • Versions: RTEMS 4.11 and RTEMS 5.1

Additional context
N/A

Third party code
N/A

Contributor Info - All information REQUIRED for consideration of pull request
Alan Cudmore, NASA/GSFC Code 582.0

@ghost ghost marked this pull request as draft November 30, 2020 14:29
@ghost ghost marked this pull request as ready for review November 30, 2020 14:55
@ghost ghost marked this pull request as draft November 30, 2020 14:57
@ghost ghost marked this pull request as ready for review November 30, 2020 21:02
@ghost ghost added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label Nov 30, 2020
@astrogeco astrogeco added CCB-20201202 and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Dec 2, 2020
@astrogeco
Copy link
Contributor

CCB 20201202 - APPROVED

  • Combine into a single ReadMe and have the RTEMS 4 as an appendix
  • Maybe reformat from .txt to markdown (open a new issue)
  • @jphickey to open issue for cleanup of ifdefs using cmake

@astrogeco
Copy link
Contributor

CCB 20201202 - APPROVED

* Combine into a single ReadMe and have the RTEMS 4 as an appendix

* Maybe reformat from .txt to markdown (open a new issue)

* @jphickey to open issue for cleanup of ifdefs using cmake

See #224 and #225

@astrogeco astrogeco changed the title fix #197 - updated readme file for pc-rtems PSP and added a new readme file for RTEMS 5 Fix #197, Update readme file for pc-rtems PSP and added a new readme file for RTEMS 5 Dec 9, 2020
@astrogeco astrogeco changed the base branch from main to integration-candidate December 9, 2020 15:21
@astrogeco astrogeco merged commit aee47de into nasa:integration-candidate Dec 9, 2020
astrogeco added a commit to nasa/cFS that referenced this pull request Dec 9, 2020
astrogeco added a commit to nasa/cFS that referenced this pull request Dec 9, 2020
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 support for RTEMS 5.1 - Cmake updates and pc-rtems PSP readme updates
1 participant