ROBOTIS-GIT / dynamixel-workbench

ROS packages for Dynamixel controllers, msgs, single_manager, toolbox, tutorials
http://emanual.robotis.com/docs/en/software/dynamixel/dynamixel_workbench/
Apache License 2.0
111 stars 178 forks source link

broken install rule in dynamixel_workbench_toolbox #112

Closed tfoote closed 6 years ago

tfoote commented 6 years ago

It looks to be trying to install from the wrong path here: https://github.com/ROBOTIS-GIT/dynamixel-workbench/blob/master/dynamixel_workbench_toolbox/CMakeLists.txt#L60

This means the 0.2.0 release fails to build to debian packages. http://build.ros.org/view/Kbin_uxv8_uXv8/job/Kbin_uxv8_uXv8__dynamixel_workbench_toolbox__ubuntu_xenial_arm64__binary/121/console

00:20:40.528 Install the project...
00:20:40.538 /usr/bin/cmake -P cmake_install.cmake
00:20:40.655 -- Install configuration: "None"
00:20:40.662 -- Installing: /tmp/binarydeb/ros-kinetic-dynamixel-workbench-toolbox-0.2.0/debian/ros-kinetic-dynamixel-workbench-toolbox/opt/ros/kinetic/lib/pkgconfig/dynamixel_workbench_toolbox.pc
00:20:40.666 -- Installing: /tmp/binarydeb/ros-kinetic-dynamixel-workbench-toolbox-0.2.0/debian/ros-kinetic-dynamixel-workbench-toolbox/opt/ros/kinetic/share/dynamixel_workbench_toolbox/cmake/dynamixel_workbench_toolboxConfig.cmake
00:20:40.667 -- Installing: /tmp/binarydeb/ros-kinetic-dynamixel-workbench-toolbox-0.2.0/debian/ros-kinetic-dynamixel-workbench-toolbox/opt/ros/kinetic/share/dynamixel_workbench_toolbox/cmake/dynamixel_workbench_toolboxConfig-version.cmake
00:20:40.668 -- Installing: /tmp/binarydeb/ros-kinetic-dynamixel-workbench-toolbox-0.2.0/debian/ros-kinetic-dynamixel-workbench-toolbox/opt/ros/kinetic/share/dynamixel_workbench_toolbox/package.xml
00:20:40.670 -- Installing: /tmp/binarydeb/ros-kinetic-dynamixel-workbench-toolbox-0.2.0/debian/ros-kinetic-dynamixel-workbench-toolbox/opt/ros/kinetic/lib/libdynamixel_workbench_toolbox.so
00:20:40.677 -- Set runtime path of "/tmp/binarydeb/ros-kinetic-dynamixel-workbench-toolbox-0.2.0/debian/ros-kinetic-dynamixel-workbench-toolbox/opt/ros/kinetic/lib/libdynamixel_workbench_toolbox.so" to ""
00:20:40.681 CMake Error at cmake_install.cmake:71 (file):
00:20:40.681   file INSTALL cannot find
00:20:40.681   "/tmp/binarydeb/ros-kinetic-dynamixel-workbench-toolbox-0.2.0/DynamixelWorkbench/include/dynamixel_workbench".
00:20:40.681 
robotpilot commented 6 years ago

Thank you so much. Now, I fixed it and up PR. Thanks,

2018-02-21 8:13 GMT+09:00 Tully Foote notifications@github.com:

It looks to be trying to install from the wrong path here: https://github.com/ROBOTIS-GIT/dynamixel-workbench/blob/ master/dynamixel_workbench_toolbox/CMakeLists.txt#L60

This means the 0.2.0 release fails to build to debian packages. http://build.ros.org/view/Kbin_uxv8_uXv8/job/Kbin_uxv8_ uXv8dynamixel_workbench_toolboxubuntu_xenial_arm64__binary/121/console

00:20:40.528 Install the project... 00:20:40.538 /usr/bin/cmake -P cmake_install.cmake 00:20:40.655 -- Install configuration: "None" 00:20:40.662 -- Installing: /tmp/binarydeb/ros-kinetic-dynamixel-workbench-toolbox-0.2.0/debian/ros-kinetic-dynamixel-workbench-toolbox/opt/ros/kinetic/lib/pkgconfig/dynamixel_workbench_toolbox.pc 00:20:40.666 -- Installing: /tmp/binarydeb/ros-kinetic-dynamixel-workbench-toolbox-0.2.0/debian/ros-kinetic-dynamixel-workbench-toolbox/opt/ros/kinetic/share/dynamixel_workbench_toolbox/cmake/dynamixel_workbench_toolboxConfig.cmake 00:20:40.667 -- Installing: /tmp/binarydeb/ros-kinetic-dynamixel-workbench-toolbox-0.2.0/debian/ros-kinetic-dynamixel-workbench-toolbox/opt/ros/kinetic/share/dynamixel_workbench_toolbox/cmake/dynamixel_workbench_toolboxConfig-version.cmake 00:20:40.668 -- Installing: /tmp/binarydeb/ros-kinetic-dynamixel-workbench-toolbox-0.2.0/debian/ros-kinetic-dynamixel-workbench-toolbox/opt/ros/kinetic/share/dynamixel_workbench_toolbox/package.xml 00:20:40.670 -- Installing: /tmp/binarydeb/ros-kinetic-dynamixel-workbench-toolbox-0.2.0/debian/ros-kinetic-dynamixel-workbench-toolbox/opt/ros/kinetic/lib/libdynamixel_workbench_toolbox.so 00:20:40.677 -- Set runtime path of "/tmp/binarydeb/ros-kinetic-dynamixel-workbench-toolbox-0.2.0/debian/ros-kinetic-dynamixel-workbench-toolbox/opt/ros/kinetic/lib/libdynamixel_workbench_toolbox.so" to "" 00:20:40.681 CMake Error at cmake_install.cmake:71 (file): 00:20:40.681 file INSTALL cannot find 00:20:40.681 "/tmp/binarydeb/ros-kinetic-dynamixel-workbench-toolbox-0.2.0/DynamixelWorkbench/include/dynamixel_workbench". 00:20:40.681

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ROBOTIS-GIT/dynamixel-workbench/issues/112, or mute the thread https://github.com/notifications/unsubscribe-auth/ACktq8CorkYMp1-nxRsMCaphgW4MfHeHks5tW1GMgaJpZM4SM0vf .

-- //------------------------------------------------------------------------------ Yoonseok Pyo, Ph.D., Senior Research Engineer / Open Source Team / R&D Dept. ROBOTIS CO., LTD. (http://www.robotis.com)

1505 Ace High-End Tower NO. 3, Gasan-dong 145,

GasanDigital 1-ro, GeumCheon-gu, Seoul, Korea 153-787

Follow ROBOTIS on Twitter https://twitter.com/robotis, Facebook https://www.facebook.com/robotis.company, Linkedin https://www.linkedin.com/company/robotis, Github https://github.com/ROBOTIS-GIT Personal Website (http://www.robotpilot.net) //------------------------------------------------------------------------------

tfoote commented 6 years ago

It looks like there's a similar issue still failing to build with the new release: http://build.ros.org/view/Kbin_uxv8_uXv8/job/Kbin_uX64__dynamixel_workbench_toolbox__ubuntu_xenial_amd64__binary/128/console

robotpilot commented 6 years ago

Thanks, @tfoote I will re-PR this package in the next sync. Please drop it. I'm sorry I took your time. I will have my own system and an internal procedure to prevent it, next week.

tfoote commented 6 years ago

It's not a problem. The notifications are mostly for your benefit. I'll go ahead with the sync without waiting.

robotpilot commented 6 years ago

Solved! ref issues: https://github.com/ROBOTIS-GIT/dynamixel-workbench/issues/117