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

Include directory not exported #5

Closed
rasmusan opened this issue Mar 27, 2019 · 3 comments
Closed

Include directory not exported #5

rasmusan opened this issue Mar 27, 2019 · 3 comments
Labels
bug Something isn't working

Comments

@rasmusan
Copy link
Contributor

I've had some problems compiling the crystal branch of ROS2 from source with log4cxx for Ubuntu 16.04. When compiling with RCL_LOGGING_IMPLEMENTATION set to rcl_logging_log4cxx, I get the following warning for many packages:

Package 'rcl_logging_log4cxx' exports the include directory
'/home/rsa/ws/ros2/underlay_desktop/install/rcl_logging_log4cxx/share/rcl_logging_log4cxx/cmake/../../../include'
which doesn't exist

The build finally fails on:

CMake Error in CMakeLists.txt:
Imported target "rviz_common::rviz_common" includes non-existent path
"/home/rsa/ws/ros2/underlay_desktop/install/rcl_logging_log4cxx/include"
in its INTERFACE_INCLUDE_DIRECTORIES.

I've fixed it by adding to CMakeLists of rcl_logging_log4cxx:

install(DIRECTORY include/${PROJECT_NAME}
  DESTINATION include/${PROJECT_NAME}
)

After this, everything compiles and works. Is this the "correct" solution? If yes, I can create a pull request.

@dirk-thomas
Copy link
Member

Thanks for reporting this.

I've fixed it by adding to CMakeLists of rcl_logging_log4cxx:
After this, everything compiles and works. Is this the "correct" solution?

Yes, that is the correct fix. Please go ahead and create a pull request.

@rasmusan
Copy link
Contributor Author

Ok, I have created the pull request now: #7

@rasmusan
Copy link
Contributor Author

rasmusan commented May 2, 2019

Pull request is merged. Closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants