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

[drake_cmake_external] Strenthen CI by removing unnecessary prereqs #347

Merged

Conversation

jwnimmer-tri
Copy link
Contributor

@jwnimmer-tri jwnimmer-tri commented Dec 20, 2024

This will help ensure that the CMake changes in RobotLocomotion/drake#22338 do not break downstream users, by ensuring that the only copies of {eigen, fmt, spdlog} on the CI machine are the custom ones built by CMake and found with find_package -- if the CMake plumbing doesn't work, then the example will fail to compile.

Towards RobotLocomotion/drake#20731.


This change is Reviewable

@jwnimmer-tri
Copy link
Contributor Author

+@rpoyner-tri for both reviews, please.

Copy link
Contributor

@rpoyner-tri rpoyner-tri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: :shipit: complete! all discussions resolved, platform LGTM from [rpoyner-tri] (waiting on @jwnimmer-tri)

@rpoyner-tri rpoyner-tri merged commit 3e1ba13 into RobotLocomotion:main Dec 21, 2024
9 checks passed
@jwnimmer-tri jwnimmer-tri deleted the cmake_external-narrow-deps branch December 21, 2024 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants