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

Enable argcomplete by default #41

Open
ijnek opened this issue Sep 8, 2022 · 1 comment
Open

Enable argcomplete by default #41

ijnek opened this issue Sep 8, 2022 · 1 comment
Assignees

Comments

@ijnek
Copy link

ijnek commented Sep 8, 2022

I found that instructions about argcomplete were added to the ros2_documentation in ros2/ros2_documentation#2059 last year, but as discussed in that PR, setting up argcomplete shouldn't be part of a beginner tutorial.

As embarassing as this sounds, it took me a long time until I realised argcomplete existed, and that I didn't have to type out the package selection options manually.

As argcomplete actually benefits everyone, is there a way to enable arg completion by default? What are the drawbacks that outweigh the advantages?

@nuclearsandwich
Copy link
Contributor

is there a way to enable arg completion by default?

command completion support requires hooking into a user's shell and I am absolutely not about to make that something which colcon does "behind the scenes" without explicit consent from the user. Depending on the shell, it may be possible for apt packages to install completions in special directories where they would be picked up automatically but there is no such clean facility for a package from pypi and I don't believe that bash supports these.

I could see us introducing a verb like colcon argcomplete install which will idempotently update the content of the user's shell configuration to include setup for colcon's argcompletion but that will simplify setup not negate it.

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

No branches or pull requests

2 participants