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

(rqt_dep) Add a way to filter by test/build/run/depend #265

Closed
vrabaud opened this issue Aug 6, 2014 · 5 comments
Closed

(rqt_dep) Add a way to filter by test/build/run/depend #265

vrabaud opened this issue Aug 6, 2014 · 5 comments

Comments

@vrabaud
Copy link
Contributor

vrabaud commented Aug 6, 2014

Probably 3 tick boxes around. Any better design ? I could have a look at it, I am sure it is trivial to add.

@vrabaud vrabaud changed the title (rosdep) Add a way to filter by test/build/run/depend (rqt_dep) Add a way to filter by test/build/run/depend Aug 6, 2014
@ablasdel
Copy link
Contributor

ablasdel commented Aug 6, 2014

This could get tricky since there are now 2 supported package.xml formats:

http://farnsworth.csres.utexas.edu/docs/catkin/html/howto/format2/index.html

@dirk-thomas
Copy link
Contributor

@ablasdel The API supports both package format versions.

@vrabaud But I think the plugin uses the rospkg API which can't distinguish dependency types.

It is also be unclear if you really want to consider only a single type. Since usually when using direct build dependencies that implies to also use recursive run dependencies...

@vrabaud
Copy link
Contributor Author

vrabaud commented Aug 7, 2014

@dirk-thomas , it makes sense. My motivation was that this depends takes into account test depends which I do not care about when I just want to build something. Maybe just an option to add test depends then ?

@dirk-thomas
Copy link
Contributor

Optionally adding test dependencies would be possible. But only for catkin packages.

@dirk-thomas
Copy link
Contributor

This issue was moved to ros-visualization/rqt_dep#1

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

No branches or pull requests

3 participants