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

list: Clarify --depends-on behavior #212

Closed
jbohren opened this issue Jun 16, 2015 · 3 comments · Fixed by #293
Closed

list: Clarify --depends-on behavior #212

jbohren opened this issue Jun 16, 2015 · 3 comments · Fixed by #293

Comments

@jbohren
Copy link
Contributor

jbohren commented Jun 16, 2015

@wjwwood Currently, catkin list --depends-on only shows the direct dependencies of each package. Is this the correct behavior, or should it be recursive?

@wjwwood
Copy link
Member

wjwwood commented Jun 16, 2015

That's a good question. My gut reaction is that the pair --depends-on and --depends-on-recursive are clearer than the pair --depends-on-1 and --depends-on, which are direct dependencies and recursive dependencies respectively. However, I've seen both patterns used in similar command line options.

@jbohren
Copy link
Contributor Author

jbohren commented Jun 16, 2015

I also like "recursive" since the meaning is clearer.

@jbohren
Copy link
Contributor Author

jbohren commented Feb 13, 2016

This should be fixed in #293.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants