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

fix library_abs set to NOTFOUND; issue #15 #17

Closed
wants to merge 2 commits into from

Conversation

amar-laksh
Copy link

Here we go, Sorry for the earlier one.

Copy link

@davidhuziji davidhuziji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tried with "library_abs". It works.

@@ -30,6 +30,7 @@ endif()
# message(STATUS "CURL_INCLUDE_DIRS: ${CURL_INCLUDE_DIRS}")

set(libcurl_vendor_LIBRARIES)
set(libcurl_abs ${CURL_LIBRARY_DIRS})

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @amar-laksh, I guess it should be library_abs instead of libcurl_abs.

@vilvo
Copy link

vilvo commented Mar 15, 2018

set(library_abs ${CURL_LIBRARY_DIRS}) fixes the issue.

@sloretz
Copy link
Contributor

sloretz commented Apr 13, 2018

ROS2 CI

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@sloretz
Copy link
Contributor

sloretz commented Apr 13, 2018

Hi @amar-laksh , Thanks for the pull request (and thanks @davidhuziji and @vilvo for testing).

While this does get a cross-compile build of resource_retriever to work, it breaks normal builds of rviz2. Would you mind investigating? (jenkins build)

[ 83%] Building CXX object CMakeFiles/rviz_default_plugins_object.dir/src/rviz_default_plugins/robot/robot_link.cpp.o
/home/rosbuild/ci_scripts/ws/src/ros2/rviz/rviz_default_plugins/src/rviz_default_plugins/robot/robot_link.cpp:66:42: fatal error: resource_retriever/retriever.h: No such file or directory
compilation terminated.
CMakeFiles/rviz_default_plugins_object.dir/build.make:1046: recipe for target 'CMakeFiles/rviz_default_plugins_object.dir/src/rviz_default_plugins/robot/robot_link.cpp.o' failed

Possibly related, during a cross compile build I notice this output during resource_retriever. Somewhere along the way resource_retriever is trying to link to the content of ${CURL_LIBRARY_DIRS}.

WARNING: Target "resource_retriever" requests linking to directory "/other/tmp/r2_cross/ros2_ws/build/libcurl_vendor/libcurl_install/lib"
.  Targets may link only to libraries.  CMake is dropping the item.

@clalancette
Copy link
Contributor

This hasn't had a response in a long time, so I'm going to close it out. Please feel free to reopen and answer @sloretz 's review if you still want to move forward with this.

@clalancette clalancette closed this Oct 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants