-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Edit readme with install instructions #36
Comments
Could you detail more on this? Is it a runtime or an instillation issue in particular? Any logs or stdout/stderr to share? Fixing the root of this may be more useful than redirecting users to any one alternative instillation method. |
I was unable to find it in This is my command output. I was able to find
|
I think you'd need to install the apt repo for ros first. Looks to be working in docker:
@cottsay , has
|
Yes, I think on Ubuntu 20.04, That's not a major issue since Humble is on 22.04 and other ROS 2 distros are EOL, and colcon is by-and-large a ROS 2 build tool. I was wondering, because the project I am working on uses colcon on ROS 1, and therefore I am working in 20.04. Still, I was happy to see that there was a work-around for me to install it via pip. |
Looks like kinetic and newer has at least some colcon packages: https://packages.ubuntu.com/search?keywords=colcon&searchon=names&suite=all§ion=all There are instructions for how to install debs without ROS repositories: https://colcon.readthedocs.io/en/released/user/installation.html#outside-the-ros-project |
I also came in here to see if somebody pointed that out. Dont think i ever saw a readme without any installation instructions, even if its a single installation command |
Hello devs,
This is an awesome extension to the base colcon functionality. I understand there are two ways to install, with
apt
asapt install python3-colcon-clean
and withpip install colcon-clean
.The
apt
method unfortunately does not work on Ubuntu 20.04, but fortunately thepip
version works.I was wondering if this could be reflected in the readme of this repo, as this information was not super-obvious. Thanks!
The text was updated successfully, but these errors were encountered: