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

Support catkin locate --this #552

Closed
beetleskin opened this issue May 22, 2019 · 8 comments
Closed

Support catkin locate --this #552

beetleskin opened this issue May 22, 2019 · 8 comments

Comments

@beetleskin
Copy link

beetleskin commented May 22, 2019

For various bash commands, it would be nice to locate the package of the current working directory. My current workaround is

catkin locate `catkin list --this -u`

but the missing --this always felt wrong, since most other verbs support it.

@mikepurvis
Copy link
Member

A PR to add this functionality would be welcome!

@beetleskin
Copy link
Author

I could give it a shot, would you point me to the right piece of code to put this in?

@jarvisschultz
Copy link
Contributor

Note, this feature was introduced in 09e2b12, and should work on any version >=0.4.0

@beetleskin
Copy link
Author

That's not true: the commit added catkin list --this, not catkin locate --this. I'm asking for the latter.

@jarvisschultz
Copy link
Contributor

Doh.... sorry. Totally misread the original comment

jarvisschultz added a commit to jarvisschultz/catkin_tools that referenced this issue May 1, 2020
@jarvisschultz
Copy link
Contributor

Felt bad I gave dumb advice, so just submitted a PR.

@beetleskin
Copy link
Author

haha, nice, thanks :)

@timonegk
Copy link
Member

This was fixed in #605.

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