-
Notifications
You must be signed in to change notification settings - Fork 217
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
osal Integration candidate: 2021-01-19 #767
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Use a single 64-bit tick counter as OS_time_t, rather than a split 32 bit seconds + 32 bit microseconds counter. This benefits in several ways: - increases the timing precision by 10x (0.1us ticks) - increases the representable range by 400x (+/-14000 yrs) - simplifies addition/subtraction (no carry over) - avoids "year 2038" bug w/32-bit timestamps
Users should call OS_FileSysStatVolume() and read the respective output members of the structure as indicated.
Fix #699, format printf correctly
Fix #718, deprecate OS_fsBlocksFree and OS_fsBytesFree
Fix #743, Create OSAL Security Policy Markdown
Fix #746, Add UtDebug message from OS_printf stub
Fix #577, Document nested tests not supported
Got a segfault after adding #747
See https://github.com/nasa/cFS/pull/177/checks?check_run_id=1742875628 |
Update ReadMe
astrogeco
added a commit
to nasa/cFS
that referenced
this pull request
Jan 27, 2021
astrogeco
added a commit
to nasa/cFS
that referenced
this pull request
Feb 3, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe the contribution
Fix #746, Add UtDebug message from OS_printf stub
Fix #692, Resolve typos in unit test assert OR statements
Fix #577, Document nested tests not supported
Fix #753, Remove UT_CheckForOpenSockets
Fix #229, add mqueue functional test
Fix #757, Readd extern C in osapi.h to support C++ use
Fix #692, display message when not implemented error occurs
Fix #591, add test teardown failure to test summary
Fix #699, format printf correctly
Fix #685, change file create to use read write
Fix #748, add additional casts
Fix #743, Create Security Policy
Fix #429, update OS_time_t definition to 64 ticks
Fix #718, deprecate OS_fsBlocksFree and OS_fsBytesFree
Fix #761, Terminate string in TestReadWriteLseek
Fix #724, Refactor UT_ClearForceFail to UT_ClearDefaultReturnValue
Fix #722, update UT_SetDefaultReturnValue comments
Fix #229, mqueue test uses sequence of numbers
Fix #760, Install modules and clean up files for unit tests
Testing performed
See https://github.com/nasa/cFS/pull/177/checks
Expected behavior changes
PR #748
PR #759
PR #696
PR #747
PR #751
PR #763
PR #758
PR #725
PR #694
PR #754
PR #749
PR #739
PR #695
PR #735
PR #764
System(s) tested on
Ubuntu 18.04
Additional context
Part of nasa/cFS#177
Third party code
None
Contributor Info - All information REQUIRED for consideration of pull request
@jphickey
@zanzaben
@skliper
@ArielSAdamsNASA