Skip to content
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

confusion: does colcon or rosdep install packages from package.xml? #4297

Open
VladimirFokow opened this issue Apr 18, 2024 · 3 comments
Open
Assignees

Comments

@VladimirFokow
Copy link
Contributor

VladimirFokow commented Apr 18, 2024

I know rosdep uses package.xml to install packages. But does colcon also use it?

Reading the "Beginner" tutorials, this quote (in section "6 Customize package.xml") is confusing to me.
It says that "colcon" is searching package.xml:

This is where your package.xml would list its dependencies on other packages, for colcon to search for


Maybe replace "colcon" with "rosdep"?
Or does colcon also use it - in which case, replace "colcon" with "rosdep and colcon"?

@clalancette
Copy link
Contributor

I believe it is correct as-is.

rosdep is the tool that indeed walks the package.xml files and installs dependencies.

colcon is the tool that builds a workspace full of packages. In order to do this, it has to know the topological ordering between them, so it also looks at the dependencies listed in package.xml to figure out that ordering.

If you have suggestions on how to improve the wording, we are of course happy to entertain them.

@VladimirFokow
Copy link
Contributor Author

Thank you for clarification!

I would replace "colcon" with "rosdep and colcon", just to be more clear.
But if there are reasons not to, I'm okay with closing this issue

@MichaelOrlov
Copy link
Contributor

@cottsay A friendly ping to follow up on this issue.

@cottsay cottsay self-assigned this Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants