-
Notifications
You must be signed in to change notification settings - Fork 0
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
Use cmake configured completion file #6
Conversation
This also install the `gz*.completion` file. Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
Should that be reflected in the nightly debs that were built above? I downloaded and queried and I'm seeing:
|
In a local docker container I believe I am seeing the correct results though:
|
I think the regular nightlies built from the |
Sorry, I didn't build the debs with the correct |
We were using the
ign.bash_completion.sh
file from the source, not after it was configured by cmake. In gazebosim/gz-tools#103, I needed to use a cmake configured file in order to sourcegz1.completion
from wherever it's installed. This is so that the bash completion files for other ign/gz libraries can be sourced automatically.Also in that PR, I'm using cmake to install the
etc/ign.bash_completion.sh
to/usr/share/bash-completion/completions/ign
, so theignition-tools.install
file can be simplified (no need for renames). It also makes it possible for users to build fromign-tools
from source and still get tab completion.In addition, this PR installs
/usr/share/gz/gz1.completion
, which wasn't being installed before.The resulting deb file has the following contents
See gazebosim/gz-tools#103 for quick instructions for building the deb file locally.
Note
Not sure if this has been addressed before, but AFAIK, installing to
/usr/share/bash-completion/completions/ign
will prevent side-by-side installation of ign-tools. I guess for the near term it won't be a problem becauseign-tools2
would be usinggz
instead ofign
, but could be a problem if we haveign-tools3
at some point.