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

cFE Integration candidate: 2021-04-02 #1279

Merged
merged 5 commits into from
Apr 2, 2021
Merged

Conversation

astrogeco
Copy link
Contributor

@astrogeco astrogeco commented Apr 2, 2021

Change Description and Impacts

PR #1274

Fix #1273, remove PspConfig global object

Removes PspConfig member from the GLOBAL_CONFIGDATA object. Updates the only remaining reference to this object inside the CFE_PSP_VERSION and uses the new Version API instead. Updates the OSAL version print to use the version API call and CFE uses the macro directly.

PR #1278

Fix #1277, ZeroCopy typo in ReadMe.md

Replaces duplicate mention of the removed CFE_SB_ZeroCopyGetPtr with the correct removal target of CFE_SB_ZeroCopyReleasePtr

Tests

cfe https://github.com/nasa/cFE/pull/1279/checks
bundle https://github.com/nasa/cFS/pull/233/checks

Additional Context

Part of nasa/cFS#233

Authors

@jphickey
@astrogeco

astrogeco and others added 4 commits April 2, 2021 08:56
Fixes mislabeled function in v6.8.0-rc1+dev484 change description
The `PspConfig` member is removed from the `GLOBAL_CONFIGDATA` object.

Updates the only remaining ref to this object inside the CFE_PSP_VERSION
macro to use the API function instead.

This also updates the OSAL and CFE version print to _not_ depend on the
global object too - OSAL becomes an API call and CFE can just use the
macro directly because its the same library (itself) so no linking
concern/issue.
@astrogeco astrogeco changed the title cFE Integration candidate: 2021-04-06 cFE Integration candidate: 2021-04-02 Apr 2, 2021
@astrogeco astrogeco marked this pull request as ready for review April 2, 2021 22:11
@astrogeco astrogeco merged commit 0850b19 into main Apr 2, 2021
astrogeco added a commit to nasa/cFS that referenced this pull request Apr 2, 2021
astrogeco added a commit to nasa/cFS that referenced this pull request Apr 2, 2021
*Submodule ICs:*
nasa/cFE#1279, nasa/PSP#288

*Submodule IC Contents:*

nasa/cFE#1274 - Fix #1273, remove PspConfig global object
nasa/cFE#1278 - Fix #1277, ZeroCopy typo in ReadMe

nasa/PSP#287 - Fix #280, remove psp configdata global object
nasa/PSP#283 - Fix #281, cleanup memory range table
astrogeco added a commit to nasa/cFS that referenced this pull request Apr 2, 2021
*Submodule ICs:*
nasa/cFE#1279, nasa/PSP#288

*Contents:*

nasa/cFE#1274 - Fix #1273, remove PspConfig global object
nasa/cFE#1278 - Fix #1277, ZeroCopy typo in ReadMe

nasa/PSP#287 - Fix #280, remove psp configdata global object
nasa/PSP#283 - Fix #281, cleanup memory range table
astrogeco added a commit to nasa/cFS that referenced this pull request Apr 2, 2021
*Submodule ICs:*
nasa/cFE#1279, nasa/PSP#288

*Contents:*

nasa/cFE#1274 - Fix #1273, remove PspConfig global object
nasa/cFE#1278 - Fix #1277, ZeroCopy typo in ReadMe

nasa/PSP#287 - Fix #280, remove psp configdata global object
nasa/PSP#283 - Fix #281, cleanup memory range table
@skliper skliper added this to the 7.0.0 milestone Sep 24, 2021
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.

Duplicate Function in Readme Remove "PspConfig" member in config structure
3 participants