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 #1693, correct function name in UT_BSP_Unlock #1694

Merged

Conversation

jphickey
Copy link
Contributor

Describe the contribution
Corrects the log message in UT_BSP_Unlock (in cfe_assert module) to match the OSAL function called (OS_MutSemGive).

Fixes #1693

Testing performed
Build and run tests

Expected behavior changes
None

System(s) tested on
Ubuntu

Additional context
Just for correctness, this log never happens in normal operation.

Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

Corrects the log message in UT_BSP_Unlock (in cfe_assert module)
to match the OSAL function called (OS_MutSemGive).
@jphickey jphickey added the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label Jul 20, 2021
@astrogeco astrogeco changed the base branch from main to integration-candidate July 21, 2021 13:54
@astrogeco astrogeco merged commit 8bbdd16 into nasa:integration-candidate Jul 21, 2021
astrogeco added a commit to nasa/cFS that referenced this pull request Jul 21, 2021
nasa/cFE#1668, improve SB coverage test

nasa/cFE#1694, correct function name in UT_BSP_Unlock
astrogeco added a commit to nasa/cFS that referenced this pull request Jul 21, 2021
**Combines**

nasa/cFE#1665, v6.8.0-rc1+dev762
nasa/osal#1113, v5.1.0-rc1+dev573

**Includes**

nasa/cFE#1664, remove default .dat extension
nasa/cFE#1660, Change CI to use Test Log.
nasa/cFE#1670, Update API doxygen list
nasa/cFE#1671, update documentation for CFE_ES_GetPoolBufInfo
nasa/cFE#1674, CFE_SB_MsgHdrSize returns size_t
nasa/cFE#1668, improve SB coverage test
nasa/cFE#1694, correct function name in UT_BSP_Unlock

nasa/osal#1106, Add independent OS_rename functional test parameter checks

Co-authored-by: Jacob Hageman <skliper@users.noreply.github.com>
Co-authored-by: Joseph Hickey <jphickey@users.noreply.github.com>
Co-authored-by: Alex Campbell <zanzaben@users.noreply.github.com>
Co-authored-by: Oliver Hamburger <oliverhamburger@users.noreply.github.com>
@astrogeco astrogeco added CCB:Approved Indicates code review and approval by community CCB and removed CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Jul 21, 2021
@astrogeco
Copy link
Contributor

CCB:2021-07-21 APPROVED

@jphickey jphickey deleted the fix-1693-logmessage-mismatch branch August 3, 2021 15:19
@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
CCB:Approved Indicates code review and approval by community CCB
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Correct syslog message in UT_BSP_Unlock
3 participants