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

Commit 12872fd7d573e1 disabled ability to pass directories to list verb #571

Closed
5 tasks
jarvisschultz opened this issue Oct 18, 2019 · 2 comments
Closed
5 tasks

Comments

@jarvisschultz
Copy link
Contributor

jarvisschultz commented Oct 18, 2019

System Info

  • Operating System: 4.15.0-65-generic #74~16.04.1-Ubuntu SMP Wed Sep 18 09:51:44 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
  • Python Version: Python 2.7.12
  • Version of catkin_tools: catkin_tools 0.4.5
  • ROS Distro: kinetic

Build / Run Issue

  • Works with catkin_make
  • Works with catkin_make_isolated --merge
  • Works with catkin build
  • Works with catkin build -p1
  • I did not read this

Expected Behavior

Numerous examples in the documentation reference the ability to get a list of packages in a directory (e.g. catkin list -u path/to/directory). For example, the cheat sheet has several examples, and so does the header for the list verb. This functionality was dropped in this commit that added the ability to run catkin list --this. This seems like quite useful functionality. Why was it dropped? If we wanted to add it back should we look to a PR, a custom verb, something else?

@mikepurvis
Copy link
Member

@jarvisschultz Did you ever do any work on updating the sphinx docs per #572 (comment)?

@jarvisschultz
Copy link
Contributor Author

Finally found a few minutes to deal with this. See PR #604

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

3 participants