search

ros-visualization / rqt_graph

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

Remove rqt_graph script. #66

Open clalancette opened 3 years ago

clalancette commented 3 years ago

Instead, only install the console script, which means it will work on both Unix and Windows.

Signed-off-by: Chris Lalancette clalancette@openrobotics.org

If we go with this PR, there are a few consequences:

  1. Running rqt_graph on Linux will no longer work. This is a change from earlier distributions, and what we had previously discussed, but it does bring rqt_graph in line with almost all of the other rqt plugins.
  2. We'll have to update the documentation, since at least https://docs.ros.org/en/rolling/Tutorials/Launch-Files/Creating-Launch-Files.html and https://docs.ros.org/en/rolling/Tutorials/Topics/Understanding-ROS2-Topics.html use the 'bare' style.
  3. We should update the release notes to explicitly call this change out.
clalancette commented 3 years ago

@ros-pull-request-builder retest this please

clalancette commented 3 years ago

@ros-pull-request-builder retest this please

clalancette commented 3 years ago

@ros-pull-request-builder retest this please

ahcorde commented 2 weeks ago

is this PR still relevant @clalancette ?