-
Notifications
You must be signed in to change notification settings - Fork 40
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
Make sure to export the include directory for resource_retriever. #22
Conversation
Otherwise, it can't properly be used as a library. Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Can you please export the package libraries, interfaces and the include directories before the dependencies? (c.f. ongoing set of PRs for fixing the order of exports) |
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Sure, the latest commit moves it around to the same order as some of the other PRs. |
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.
lgtm
Thanks for the reviews. I'm fairly certain that the CI failures are unrelated to this PR (the macOS ones look like a bug in rclcpp, while the Windows one looks like a known issue: ros2/build_farmer#86). Therefore, I'm going to merge this. |
…20969) Increasing version of package(s) in repository `resource_retriever` to `2.1.0-1`: - upstream repository: https://github.com/ros/resource_retriever.git - release repository: https://github.com/ros2-gbp/resource_retriever-release.git - distro file: `dashing/distribution.yaml` - bloom version: `0.8.0` - previous version for package: `null` ## libcurl_vendor ``` * depend on curl (mapping to curl, libcurl4-openssl-dev) for packaging (#25 <ros/resource_retriever#25>) * add missing dependency on pkg-config (#19 <ros/resource_retriever#19>) * [libcurl_vendor] convert to ament and setup env hooks for library paths (#14 <ros/resource_retriever#14>) * Contributors: Dirk Thomas, Mikael Arguedas, William Woodall ``` ## resource_retriever ``` * Make sure to export the include directory for resource_retriever. (#22 <ros/resource_retriever#22>) * Contributors: Chris Lalancette ```
…os#20969) Increasing version of package(s) in repository `resource_retriever` to `2.1.0-1`: - upstream repository: https://github.com/ros/resource_retriever.git - release repository: https://github.com/ros2-gbp/resource_retriever-release.git - distro file: `dashing/distribution.yaml` - bloom version: `0.8.0` - previous version for package: `null` ## libcurl_vendor ``` * depend on curl (mapping to curl, libcurl4-openssl-dev) for packaging (ros#25 <ros/resource_retriever#25>) * add missing dependency on pkg-config (ros#19 <ros/resource_retriever#19>) * [libcurl_vendor] convert to ament and setup env hooks for library paths (ros#14 <ros/resource_retriever#14>) * Contributors: Dirk Thomas, Mikael Arguedas, William Woodall ``` ## resource_retriever ``` * Make sure to export the include directory for resource_retriever. (ros#22 <ros/resource_retriever#22>) * Contributors: Chris Lalancette ```
Otherwise, it can't properly be used as a library.
fixes #21
Signed-off-by: Chris Lalancette clalancette@openrobotics.org