search

ros-visualization / rqt_graph

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

Noetic removal of scripts argument got rid of global executable #44

Closed sloretz closed 4 years ago

sloretz commented 4 years ago

It looks like I made a mistake in https://github.com/ros-visualization/rqt_graph/pull/43 by removing the scripts argument from the setup.py. I mistakenly thought it was installing to the package bin, but it actually installs to the global bin path. Meaning before #43 it would have been possible to either run rosrun rqt_graph rqt_graph or just plain rqt_graph, but after #43 only the first one works.

PRs incoming

ros-discourse commented 4 years ago

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

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

dirk-thomas commented 4 years ago

All four PRs have been merged and released into Noetic.