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 #686, Use errno in clock_getres error reporting #687

Merged
merged 1 commit into from
Dec 18, 2020

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Dec 14, 2020

Describe the contribution
Fix #686 - Now uses errno instead of status return from clock_getres with strerror reporting

Testing performed
Built and ran unit tests on linux, which doesn't mean much since it doesn't include this code in functional or coverage testing.
Requesting retest from @ericgilligan-nasa

Expected behavior changes
Fixed error message

System(s) tested on

  • Hardware: cFS Dev Server
  • OS: Ubuntu 18.04
  • Versions: Bundle main + this commit

Additional context
None.

Third party code
None.

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

@skliper skliper added bug CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Dec 14, 2020
@skliper skliper added this to the 6.0.0 milestone Dec 14, 2020
@astrogeco
Copy link
Contributor

CCB 2020-12-16 APPROVED

@astrogeco astrogeco added CCB-20201216 and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Dec 18, 2020
@astrogeco astrogeco changed the base branch from main to integration-candidate December 18, 2020 14:29
@astrogeco astrogeco merged commit 8f9db2c into nasa:integration-candidate Dec 18, 2020
astrogeco added a commit to nasa/cFS that referenced this pull request Dec 18, 2020
@skliper skliper deleted the fix686-use_errno branch February 1, 2021 21:36
jphickey pushed a commit to jphickey/osal that referenced this pull request Aug 10, 2022
jphickey pushed a commit to jphickey/osal that referenced this pull request Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OSAL not reporting correct error string if clock_getres() call in OS_Posix_TimeBaseAPI_Impl_Init fails
2 participants