-
Notifications
You must be signed in to change notification settings - Fork 227
Add Boost.Geometry to dependencies #441
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
base: develop
Are you sure you want to change the base?
Conversation
|
@grafikrobot , did I add it correctly? Or is BGL the first library to depend on Boost.Geometry and test this part of the build system? |
Yes.
Don't know. |
OK, so this is a bug in Boost.Geometry? |
|
@pdimov do you maintain Boost.CRC? Do you know if this problem is with Geometry or CRC? |
|
It's not a problem in CRC. It's caused by Geometry listing CRC as a dependency in its CMakeLists.txt here: but it doesn't actually depend on CRC. |
|
@pdimov thanks for answering. But... why does that cause this error, just because it doesn't actually use it? |
|
Because only the actual dependencies are installed by depinst, so libs/crc is empty and the target Boost::crc doesn't exist. |
|
Thanks for the explanation. PR to fix Boost.Geometry: boostorg/geometry#1437 |
No description provided.