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 #1738, Add cast to MIR prints #1739

Merged
merged 1 commit into from
Aug 2, 2021
Merged

Conversation

pepepr08
Copy link
Contributor

@pepepr08 pepepr08 commented Jul 30, 2021

Describe the contribution
Fixes #1738 Added cast to MIR prints in time_UT.c so that the RTEMS build doesn't break.

Testing performed
Steps taken to test the contribution:

  1. Build and run time UT tests:
    make -C build/native/default_cpu1/time
    make -C build/native/default_cpu1/time test
  2. Build and run cFS tests:
    make
    make test

Expected behavior changes
RTEMS workflow should build and run successfully.

System(s) tested on

  • Ubuntu 18.04 VM

Contributor Info - All information REQUIRED for consideration of pull request
Jose F. Martinez Pedraza / NASA GSFC

@pepepr08 pepepr08 added the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label Jul 30, 2021
@skliper skliper linked an issue Jul 30, 2021 that may be closed by this pull request
@astrogeco astrogeco merged commit c4ae5b2 into nasa:main Aug 2, 2021
astrogeco added a commit to nasa/cFS that referenced this pull request Aug 2, 2021
@astrogeco astrogeco changed the title Fix #1738 Add cast to MIR prints Fix #1738, Add cast to MIR prints Aug 2, 2021
@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 Aug 4, 2021
astrogeco added a commit to nasa/cFS that referenced this pull request Aug 6, 2021
**Combines**

nasa/cFE#1759, v6.8.0-rc1+dev810
nasa/osal#1126, v5.1.0-rc1+dev586

**Includes**

*cFE*

- nasa/cFE#1752, Add null pointer check to table GetAddresses and ReleaseAddresses
- nasa/cFE#1742, Remove SB get last message sender info requirement
- nasa/cFE#1732, Fix #1725 Update UTs to use UtAssert_MIR
- nasa/cFE#1736, Add Functional Tests cFE Message ID
- nasa/cFE#1707, Add Time Conversion Functional Test
- nasa/cFE#1739, Add cast to MIR prints

*osal*

- nasa/osal#1122, Add UtAssert_MIR macro
- nasa/osal#1125, add generic asserts from CFE coverage testing
- nasa/osal#1121, add osapi-shell-stubs.c to OSAL stub library

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: Jose F Martinez Pedraza <pepepr08@users.noreply.github.com>
Co-authored-by: Niall Mullane <nmullane@users.noreply.github.com>
Co-authored-by: Ariel Adams <ArielSAdamsNASA@users.noreply.github.com>
Co-authored-by: Paul <pavll@users.noreply.github.com>
@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 CCB:FastTrack
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add (unsigned int) cast to MIR prints in time_UT.c
3 participants