ros-visualization / rqt_graph

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

Install executable rqt_graph #49

Closed jacobperron closed 4 years ago

jacobperron commented 4 years ago

This fixes a regression introduced in #42.

Now the application can be started with 'rqt_graph' or with 'ros2 run'

ivanpauno commented 4 years ago

Maybe, trying something like this https://stackoverflow.com/questions/54290710/how-to-properly-get-console-scripts-install-scripts-directory

jacobperron commented 4 years ago

This should probably target a to-be-created dashing-devel branch.

To clarify, is the reason because Crystal is EOL, or something else?

dirk-thomas commented 4 years ago

To clarify, is the reason because Crystal is EOL, or something else?

Yeah, since it is a significant change I would not risk applying it to an EOL distro.

jacobperron commented 4 years ago

I've targeted a new dashing-devel branch and ran CI for sanity.

dirk-thomas commented 4 years ago

I've targeted a new dashing-devel branch and ran CI for sanity.

@jacobperron Please reference all related PRs / commits here to ensure the branch name was updated in all locations.

jacobperron commented 4 years ago

Related PRs for ROS 2 Rolling:

jacobperron commented 4 years ago

Related PRs for remaining distros: