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

Add target_include_directories to src/CMakeLists.txt #295

Closed
williamposey opened this issue Sep 11, 2024 · 0 comments · Fixed by #301
Closed

Add target_include_directories to src/CMakeLists.txt #295

williamposey opened this issue Sep 11, 2024 · 0 comments · Fixed by #301
Assignees
@rjbrown2
Labels
cmp NASA GSFC CMP

Comments

@williamposey
Copy link
Contributor

Add the line target_include_directories(crypto PUBLIC ../include) in after the call to add_cfe_app(crypto ${LIB_SRC_FILES}) in src/CMakeLists.txt. This allows for another cFS app to use the cFE CMake function add_cfe_app_dependency in order to include public header files from another app/library that it depends on
@williamposey williamposey added the cmp NASA GSFC CMP label Sep 11, 2024
@rjbrown2 rjbrown2 self-assigned this Sep 11, 2024
rjbrown2 pushed a commit that referenced this issue Sep 11, 2024
@rjbrown6
[#295] Add include directory call
9b211e3
@rjbrown2 rjbrown2 mentioned this issue Sep 11, 2024
Merged
@rjbrown2 rjbrown2 linked a pull request Sep 11, 2024 that will close this issue
Merge 295 Target include Directory #301
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rjbrown2 rjbrown2

Labels
cmp NASA GSFC CMP
Projects
CryptoLib Path Forward
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Merge 295 Target include Directory nasa/CryptoLib
2 participants
@rjbrown2 @williamposey