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.

Sign up for GitHub

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

Jump to bottom

Output format string as debug message in OS_printf stub #746

Closed
skliper opened this issue Jan 7, 2021 · 0 comments · Fixed by #747 or #767
Closed

Output format string as debug message in OS_printf stub #746

skliper opened this issue Jan 7, 2021 · 0 comments · Fixed by #747 or #767
Labels
enhancement unit-test Tickets related to the OSAL unit testing (functional and/or coverage)
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented Jan 7, 2021

Is your feature request related to a problem? Please describe.
OS_printf stub is silent, more helpful when debugging tests to see at minimum the format string

Describe the solution you'd like
Output the format string as a debug messae similar to CFE_EVS_SendEvent

Describe alternatives you've considered
Could put in a hook, but this is general and useful enough to add

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added this to the 6.0.0 milestone Jan 7, 2021
skliper added a commit to skliper/osal that referenced this issue Jan 7, 2021
@skliper
Fix nasa#746, Add UtDebug message from OS_printf stub
5bbf041
@skliper skliper mentioned this issue Jan 7, 2021
Merged
@skliper skliper added the unit-test Tickets related to the OSAL unit testing (functional and/or coverage) label Jan 11, 2021
astrogeco added a commit that referenced this issue Jan 21, 2021
@astrogeco
Merge pull request #747 from skliper/fix746-os_printf_stub_print
709b582 
Fix #746, Add UtDebug message from OS_printf stub
@astrogeco astrogeco mentioned this issue Jan 26, 2021
Merged
jphickey added a commit to jphickey/osal that referenced this issue Aug 10, 2022
@jphickey
Fix nasa#746, simplify dependency handling
aea76b9 
Put more dependencies into the "mission_defaults.cmake" file for
more visibility and ease of configuration.  This now includes all
"implicit" modules such as cfe-core, osal, and psp.

Also push the calls to "generate_config_includefile" to a sub-script
which can be distributed with each app and evaluated as part of the
build.  This reduces dependencies on special naming conventions like
"fsw/mission_inc" and "fsw/platform_inc", and apps can explicitly manage
the files that users are expected to override.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement unit-test Tickets related to the OSAL unit testing (functional and/or coverage)
Projects
None yet
Milestone
Caelum
1 participant
@skliper