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

Change UT_ClearForceFail to UT_ClearDefaultReturnValue #724

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

Change UT_ClearForceFail to UT_ClearDefaultReturnValue #724

skliper opened this issue Jan 4, 2021 · 0 comments · Fixed by #725 or #767
Assignees
@zanzaben
Labels
deprecate unit-test Tickets related to the OSAL unit testing (functional and/or coverage)
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented Jan 4, 2021

Is your feature request related to a problem? Please describe.
UT_SetForceFail was deprecated in favor of UT_SetDefaultReturnValue, so UT_ClearForceFail is no longer a mirror

Describe the solution you'd like
Change UT_ClearForceFail to UT_ClearDefaultReturnValue

Describe alternatives you've considered
None

Additional context
See also #722, the comments should be updated for UT_SetDefaultReturnValue and UT_ClearDefaultReturnValue

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added this to the 6.0.0 milestone Jan 4, 2021
@skliper skliper added the unit-test Tickets related to the OSAL unit testing (functional and/or coverage) label Jan 4, 2021
zanzaben added a commit to zanzaben/osal that referenced this issue Jan 4, 2021
@zanzaben
Fix nasa#724 Refactor UT_ClearForceFail to UT_ClearDefaultReturnValue
a4f0937
This was referenced Jan 4, 2021
Merged
Closed
Closed
Closed
zanzaben added a commit to zanzaben/osal that referenced this issue Jan 12, 2021
@zanzaben
Fix nasa#724 Refactor UT_ClearForceFail to UT_ClearDefaultReturnValue
50079a7
zanzaben added a commit to zanzaben/osal that referenced this issue Jan 13, 2021
@zanzaben
Fix nasa#724 Refactor UT_ClearForceFail to UT_ClearDefaultReturnValue
eb99a65
zanzaben added a commit to zanzaben/osal that referenced this issue Jan 13, 2021
@zanzaben
Fix nasa#724, Refactor UT_ClearForceFail to UT_ClearDefaultReturnValue
a53e27c
@CDKnightNASA CDKnightNASA mentioned this issue Jan 13, 2021
Closed
zanzaben added a commit to zanzaben/osal that referenced this issue Jan 22, 2021
@zanzaben
Fix nasa#724, Refactor UT_ClearForceFail to UT_ClearDefaultReturnValue
3fa2f50
astrogeco added a commit that referenced this issue Jan 22, 2021
@astrogeco
Merge pull request #725 from zanzaben/fix724_Rename_UT_ClearForceFail
436e1ea 
Fix #724, #722, rename ut clear force fail
This was referenced Jan 26, 2021
Merged
Merged
Merged
Merged
jphickey added a commit to jphickey/osal that referenced this issue Aug 10, 2022
@jphickey
Fix nasa#724, implement config-based target builds
05eb82a 
The existing build system built target executables grouped by toolchain
as a proxy for CPU architecture + machine options/flags.  The app binaries
would be built once and copied to any/all targets sharing that toolchain.

The side effect of doing this is that the application needs to be written
in an CPU-agnostic manner, performing its subscriptions and configurations
from runtime table data rather than hardcoded/fixed values.  Unfortunately
most apps are not coded that way, so workarounds were needed.

This changes the top level process to include the "platform" within this
target build logic, effectively treating different platform configs as
entirely different builds, even if they share the same toolchain file.

As a result, binaries will only be shared between targets that explicitly
set the "TGTx_PLATFORM" setting in targets.cmake to the same value.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zanzaben zanzaben

Labels
deprecate unit-test Tickets related to the OSAL unit testing (functional and/or coverage)
Projects
None yet
Milestone
Caelum
3 participants
@zanzaben @skliper @astrogeco