Do the pip package dance for colcon as well.

ros2 / ci

ROS 2 CI Infrastructure

http://ci.ros2.org/

Apache License 2.0

48 stars 30 forks source link

Closed clalancette closed 3 months ago

clalancette commented 4 months ago

That is, only install the colcon packages if they were not installed via an earlier mechanism.

We also make sure to only both calling "pip install" if there are any packages to install.

clalancette commented 4 months ago

CI:

clalancette commented 3 months ago

All right. The warnings here were unrelated to this PR; instead, they are related to https://github.com/ros2/rcl/pull/1135 (which was reverted in https://github.com/ros2/rcl/pull/1142), and a change to qt_dotgraph, which should be fixed by https://github.com/ros-visualization/qt_gui_core/pull/290 . With that in mind, I'm going to go ahead and merge this one in.