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 more CMake variables to pass to vendor projects #519

Merged
merged 1 commit into from
Mar 13, 2024
Merged

Add more CMake variables to pass to vendor projects #519

merged 1 commit into from
Mar 13, 2024

Conversation

cottsay
Copy link
Contributor

@cottsay cottsay commented Mar 13, 2024

  • CMAKE_FIND_DEBUG_MODE: Similar to CMAKE_VERBOSE_MAKEFILE, this flag causes CMake to generate additional console output.
  • CMAKE_MODULE_PATH: Similar to CMAKE_PREFIX_PATH, this list of directories can be used to help CMake search additional locations for package finding modules.

@cottsay
Add more CMake variables to pass to vendor projects
54d144a 
* CMAKE_FIND_DEBUG_MODE: Similar to CMAKE_VERBOSE_MAKEFILE, this flag
  causes CMake to generate additional console output.
* CMAKE_MODULE_PATH: Similar to CMAKE_PREFIX_PATH, this list of
  directories can be used to help CMake search additional locations for
  package finding modules.

Signed-off-by: Scott K Logan <logans@cottsay.net>
@cottsay cottsay added the enhancement New feature or request label Mar 13, 2024
@cottsay cottsay self-assigned this Mar 13, 2024
@cottsay cottsay requested a review from clalancette as a code owner March 13, 2024 16:12
@cottsay
Copy link
Contributor Author

cottsay commented Mar 13, 2024

CI with every ament_cmake_vendor_package user:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@cottsay cottsay merged commit 9743c9c into rolling Mar 13, 2024
2 of 3 checks passed
@delete-merged-branch delete-merged-branch bot deleted the cottsay/vendor-more-vars branch March 13, 2024 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@clalancette clalancette clalancette approved these changes

Assignees

@cottsay cottsay

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cottsay @clalancette