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

CFE needs to provide CMake-based config file for OSAL #652

Closed
jphickey opened this issue Apr 29, 2020 · 2 comments · Fixed by #654, #667 or #672
Closed

CFE needs to provide CMake-based config file for OSAL #652

jphickey opened this issue Apr 29, 2020 · 2 comments · Fixed by #654, #667 or #672
Milestone

Comments

@jphickey
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Currently the CFE cmake script generates an osconfig.h file for OSAL to compile with.

Describe the solution you'd like
After nasa/osal#285 is fixed (PR nasa/osal#427) the CFE will need to pass OSAL a configuration file in CMake syntax, which then OSAL uses to generate its own osconfig.h.

Additional context
Needed for compatibility

Requester Info
Joseph Hickey, Vantage Systems, Inc.

jphickey added a commit to jphickey/cFE that referenced this issue Apr 29, 2020
Rather than CFE generating an osconfig.h file directly, it just
needs to select file(s) from the defs directory and pass them
to OSAL for the config.
jphickey added a commit to jphickey/cFE that referenced this issue Apr 29, 2020
Rather than CFE generating an osconfig.h file directly, it just
needs to select file(s) from the defs directory and pass them
to OSAL for the config.
jphickey added a commit that referenced this issue Apr 29, 2020
astrogeco added a commit that referenced this issue Apr 30, 2020
jphickey added a commit to jphickey/cFE that referenced this issue Apr 30, 2020
Rather than CFE generating an osconfig.h file directly, it just
needs to select file(s) from the defs directory and pass them
to OSAL for the config.
jphickey added a commit to jphickey/cFE that referenced this issue May 4, 2020
Rather than CFE generating an osconfig.h file directly, it just
needs to select file(s) from the defs directory and pass them
to OSAL for the config.
jphickey added a commit to jphickey/cFE that referenced this issue May 4, 2020
Rather than CFE generating an osconfig.h file directly, it just
needs to select file(s) from the defs directory and pass them
to OSAL for the config.
astrogeco added a commit that referenced this issue May 4, 2020
@jtovias
Copy link
Contributor

jtovias commented May 20, 2020

@jphickey , I'm trying to override OS_MAX_FILE_NAME, how do I update OS_MAX_FILE_NAME in native_osconfig.cmake?

@jtovias
Copy link
Contributor

jtovias commented May 20, 2020

@jphickey figured it out. have to set OSAL_CONFIG_MAX_FILE_NAME.

@skliper skliper added this to the 6.8.0 milestone Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants