ros-tooling / cross_compile

A tool to build ROS and ROS2 workspaces for various targets
Apache License 2.0
188 stars 60 forks source link

Don't depend on ament Python linters via a ROS2 installation #109

Closed emersonknapp closed 4 years ago

emersonknapp commented 4 years ago

Description

In order to remove this tool from the ROS2 build ecosystem, we can't depend on the ament linters via a ROS2 installation. Instead of relying on them that way, install via pip VCS option

Related Issues

Part of Epic https://github.com/ros-tooling/cross_compile/issues/102

Completion Criteria

cross_compile does not depend on any ament packages being installed via apt

Implementation Notes/Suggestions

https://discourse.ros.org/t/ament-lint-for-pure-python-packages/12247/2 based on this conversation we should be able to just install the ament_lint packages with pip via its vcs options, e.g.

pip3 install -e "git+https://github.com/ament/ament_lint.git@0.8.1#egg=ament_lint&subdirectory=ament_lint"