-
Notifications
You must be signed in to change notification settings - Fork 57
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
Use system pybind11 and absl when available #73
Conversation
tree/CMakeLists.txt
Outdated
set(ABSEIL_CMAKE_ARGS | ||
${ABSEIL_CMAKE_ARGS} | ||
"-DCMAKE_OSX_ARCHITECTURES=${CMAKE_OSX_ARCHITECTURES}") | ||
find_package(pybind11 CONFIG) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we require a minimal pybind11 version here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For example, it compiles with 2.10.4
.
find_package(pybind11 2.10.4 REQUIRED CONFIG)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for delay - updated, PTAL.
@superbobry bumping this. I'll fork if needed, but would prefer to use your version since it should already be done. |
Current CMakeLists.txt always downloads and builds `pybind11` and `absl` which conflicts with packaging best practices in some of the distributions. This change uses system libraries (if available) and falls back to the old behavior (if not).
thank you @ndl. |
63f25d4
into
google-deepmind:master
Thanks! |
thank you so much @superbobry! |
BTW - can confirm this works as expected. |
Now we just need a new release |
Current CMakeLists.txt always downloads and builds
pybind11
andabsl
which conflicts with packaging best practices in some of the distributions. This change uses system libraries (if available) and falls back to the old behavior (if not).Note: it's better to view the diffs with white space changes ignored, than it's much more clear than the change is just a couple lines of code.