Subject: Re: bug#63860: [PATCH] Shepherd: Add Bash completion file.
From: Ludovic Courtès <[email-masked]>
To: Oleg Pykhalov <[email-masked]>
Cc: [email-masked], Liliana Marie Prikler <[email-masked]>
Date: Mon, 05 Jun 2023 14:03:32 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

Hello,

Oleg Pykhalov <[email-masked]> skribis:

> * etc/completion/bash/herd: New file.
> * Makefile.am (dist_bashcompletion_DATA): New variable.
> * configure.ac: Add --with-bash-completion-dir.

Excellent!  Applied with the changes below.

Thank you & thanks Liliana for reviewing!

Ludo’.

diff --git a/NEWS b/NEWS
index d6c2fc0..b644adb 100644
--- a/NEWS
+++ b/NEWS
@@ -13,6 +13,11 @@ Please send Shepherd bug reports to [email-masked].
 
 * Changes in 0.10.2
 
+** New Bash completion
+
+A Bash completion file is now installed, providing tab completion for the
+‘herd’ command.
+
 ** ‘herd’ shows a hint when the service and action are likely swapped
 
 The hint is printed for instance when typing ‘herd foobar start’.
diff --git a/etc/completion/bash/herd b/etc/completion/bash/herd
index f006b86..74e7a24 100644
--- a/etc/completion/bash/herd
+++ b/etc/completion/bash/herd
@@ -44,8 +44,8 @@ _herd_complete_3()
     local command="${COMP_WORDS[2]}"
     case "$command" in
         guix-daemon)
-            complitions="on off"
-            mapfile -t COMPREPLY < <(compgen -W "$complitions" -- "${COMP_WORDS[$COMP_CWORD]}")
+            completions="on off"
+            mapfile -t COMPREPLY < <(compgen -W "$completions" -- "${COMP_WORDS[$COMP_CWORD]}")
             ;;
         *)
             ;;