-
Notifications
You must be signed in to change notification settings - Fork 280
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
CMake package registry breaks workspace isolation #1119
Comments
What it seems to 'break' is workspace isolation. Whether it's completely incompatible with Catkin and Colcon I'm not sure. |
I don't know enough about the CMake package registry to make a suggestion here. It sounds reasonable to me though. Even with these flags embedded by default the user should be able to pass different values if desired. Would you like to create a pull request for the proposed change? Probably not only to Just for reference the parallel ticket on |
This is a really subtle, but problematic issue with CMake, especially on Windows. I've experienced unexpected and highly undesirable behaviour from CMake's "magic" use of it's package registry where it's basically used a recently built version of a library in an unrelated workspace. The way I see it, colcon's philosophy is to create a self-contained workspace which is at odds with the CMake package registry. It would be reasonable to disable the package regsitry by default an option to turn restore the default "magic" behaviour. To avoid the issue though, you can use |
We recently ran into issues with packages that leverage the CMake package registry feature. This feature does not seem to be compatible with build tools like catkin, colcon, etc.. Should catkin automatically be passing the CMake disable flags?
The text was updated successfully, but these errors were encountered: