search

ros-visualization / rqt_graph

http://wiki.ros.org/rqt_graph
26 stars 27 forks source link

[Noetic] Use catkin_install_python() #43

Closed sloretz closed 4 years ago

sloretz commented 4 years ago

This uses catkin_install_python() instead of a mix of install and the setup.py scripts argument to make sure the shebang gets rewritten.

It fixes a bug in the current debian package I encountered here: http://wiki.ros.org/ROS/Tutorials/UnderstandingTopics#Using_rqt_graph

$ rosrun rqt_graph rqt_graph 
/usr/bin/env: ‘python’: No such file or directory

@dirk-thomas got time to make another Noetic release with this fix? If not, with your permission I can make one.

ros-discourse commented 4 years ago

This pull request has been mentioned on ROS Discourse. There might be relevant details there:

https://discourse.ros.org/t/ros-noetic-call-for-testing/14203/1

ros-discourse commented 4 years ago

This pull request has been mentioned on ROS Discourse. There might be relevant details there:

https://discourse.ros.org/t/ros-noetic-ninjemys-release/14262/14