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

Fix #285, Refactor OSAL to avoid inclusion of C files #443

Merged
merged 3 commits into from
May 4, 2020
Merged

Fix #285, Refactor OSAL to avoid inclusion of C files #443

merged 3 commits into from
May 4, 2020

Commits on May 4, 2020

  1. Fix nasa#285, Refactor OSAL to avoid inclusion of C files 

    Use separate source files and CMake-based source selection
to configure OSAL features, rather than using the C preprocessor.

All implementation units now provide a separate header
file for each functional group/subsystem.

This commit covers the OSAL main FSW code and particularly the
"portable" implementation blocks to follow this model.  Includes
minor updates to UT stubs due to header file changes.
    @jphickey
    jphickey committed May 4, 2020
    Configuration menu
    Copy the full SHA
    2edd5bd View commit details
    Browse the repository at this point in the history

  2. Fix nasa#285, Refactor OSAL to avoid inclusion of C files 

    Use separate source files and CMake-based source selection
to configure OSAL features, rather than using the C preprocessor.

This commit covers the OSAL coverage test.  This had been using
inclusion of a C file to allow access to static functions/variables.
This updates all UT code to correspond with the FSW code changes.
All "static" helper functions become non-static so unit test can
invoke directly.  Prototype is in a private header, so
the functions are still not exposed in the API.
    @jphickey
    jphickey committed May 4, 2020
    Configuration menu
    Copy the full SHA
    636bd7c View commit details
    Browse the repository at this point in the history

  3. Update nasa#285, Change CI config to specify permissive mode 

    Permissive mode is required for testing on POSIX when
running as a normal (non-root) user.
    @jphickey
    jphickey committed May 4, 2020
    Configuration menu
    Copy the full SHA
    a9c0205 View commit details
    Browse the repository at this point in the history