-
Notifications
You must be signed in to change notification settings - Fork 145
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
Unable to include headers from workspace packages that do not install the headers, when install is enabled #427
Comments
I would expect the exact opposite. If you do the |
|
@gavanderhoorn @wjwwood |
O wait, of course it works: I assumed you were then building other packages on-top of the installed workspace. If the headers weren't present in the underlying workspace, things wouldn't have worked. But that is not the case here. |
I understand your use case, but it's not something we intended to support. AFAIK, the idea has always been that the only "right" way to share resources between two packages is by installing them. The fact that you can access them when using a merged devel space (which I'm sorry, as I expect this isn't what you wanted to hear, but I'd currently classify this as "wont-fix". |
System Info
Build / Run Issue
Headers from packages that export their include folder, but do not install it, cannot be found when install is enabled, and using linked or isolated develspaces. Using a merged develspace still works.
catkin_make
catkin_make install
catkin_make_isolated --merge
catkin build
(linked develspace)catkin build
(isolated develspace)catkin build
(merged develspace)catkin build -p1
(linked develspace)catkin build -p1
(isolated develspace)catkin build -p1
(merged develspace)Expected Behavior
Headers from packages that export their include directory should be accessible to dependent packages within the workspace when the workspace is configured to install, even if the package does not install the headers.
Actual Behavior
The project fails to build, because the secondary
Log gist: https://gist.github.com/ajthompson/e6f958a979de86254eb29e914966a0bf
Steps to Reproduce the Issue
The workspace provided uses two simple packages - the first package, test, provides a simple header file defining a macro in include/test/test.h, and exports the include directory in the CMakeLists.txt. The second package, test2, has a build dependency for test, and contains a single C++ source file, consisting of a main function that returns the macro defined in test.h.
The text was updated successfully, but these errors were encountered: