Tags: patch

On Wed, 04 May 2016 06:06:30 -0400 Sandro Tosi <mo...@debian.org> wrote:
Package: awscli
Severity: normal

Hello,
awscli comes with bash/zsh completion scripts, but they are not activated by
default; please enable them.

I think /usr/share/bash-completion/completions/aws_bash_completer needs only to be renamed "aws", to work with the dynamic completion loading in /usr/share/bash-completion/bash_completion:

diff --git a/debian/rules b/debian/rules
index 418b78e..2c0fa06 100755
--- a/debian/rules
+++ b/debian/rules
@@ -10,7 +10,7 @@
 override_dh_auto_install:
        python3 setup.py install --root=debian/awscli --install-layout=deb
        mv $(CURDIR)/debian/awscli/usr/bin/aws_bash_completer \
-          $(CURDIR)/debian/awscli/usr/share/bash-completion/completions
+          $(CURDIR)/debian/awscli/usr/share/bash-completion/completions/aws
        mv $(CURDIR)/debian/awscli/usr/bin/aws_zsh_completer.sh \
           $(CURDIR)/debian/awscli/usr/share/zsh/vendor-completions/_awscli
        rm $(CURDIR)/debian/awscli/usr/bin/aws.cmd

Reply via email to