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 #1289, cmake script modernization #1291

Merged
merged 1 commit into from
Oct 11, 2022
Merged

Fix #1289, cmake script modernization #1291

merged 1 commit into from
Oct 11, 2022

Commits on Sep 29, 2022

  1. Fix nasa#1289, cmake script modernization 

    Use target properties to define interfaces and compiler definitions
rather than referencing global variables or using "known" paths in
other modules.  This better aligns with current practices and creates
a more robust build environment that is less dependent on specific path
names existing in a given module.

This adds an "osal_public_api" interface target that contains the paths
to the public API headers as its INTERFACE_INCLUDE_DIRECTORIES and any
required compiler definitions as its INTERFACE_COMPILE_DEFINITIONS
property.  Applications should use this rather than referring to the
"${OSAL_SOURCE_DIR}/src/os/inc" include path directly.
    @jphickey
    jphickey committed Sep 29, 2022
    Configuration menu
    Copy the full SHA
    66f2d4a View commit details
    Browse the repository at this point in the history