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 #1081, Correct deprecation directive typo #1082

Merged

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Jan 11, 2021

Describe the contribution
Fix #1081 - fixed 2 deprecation directive typos

Testing performed
Standard build and unit test

Expected behavior changes
Deprecation of CFE_SB_RcvMsg works correctly (won't show up if CFE_OMIT_DEPRECATED_6_8 is defined)

System(s) tested on

  • Hardware: cFS Dev Server
  • OS: Ubuntu 18.04
  • Versions: Bundle main + this change

Additional context
None

Third party code
None

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

@skliper skliper requested a review from CDKnightNASA January 11, 2021 16:31
@astrogeco astrogeco changed the base branch from main to integration-candidate January 13, 2021 04:08
@astrogeco astrogeco merged commit 3120c02 into nasa:integration-candidate Jan 13, 2021
@astrogeco astrogeco added CCB:Ignore Pull Request can be ignored. Will be re-examined at by next CCB. CCB:Ready Ready for discussion at the Configuration Control Board (CCB) and removed CCB:Ignore Pull Request can be ignored. Will be re-examined at by next CCB. labels Jan 13, 2021
astrogeco added a commit to nasa/cFS that referenced this pull request Jan 13, 2021
@astrogeco astrogeco removed the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label Jan 13, 2021
@skliper skliper deleted the fix1081-deprecate_typo branch February 1, 2021 22:05
@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cfe_sb.h -- #if instead of #ifdef CFE_OMIT_DEPRECATED_6_8
2 participants