Subject: bug#63860: closed (Re: bug#63860: [PATCH] Shepherd: Add Bash completion file.)

From: [email-masked] (GNU bug Tracking System)
Date: Mon, 05 Jun 2023 12:04:03 +0000

Your bug report

#63860: [PATCH] Shepherd: Add Bash completion file.

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
63860: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=63860
GNU Bug Tracking System
Contact [email-masked] with problems

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]}")
             ;;
         *)
             ;;

Subject: [PATCH] Shepherd: Add Bash completion file.
From: Oleg Pykhalov <[email-masked]>
To: [email-masked]
Cc: Oleg Pykhalov <[email-masked]>
Date: Sat, 3 Jun 2023 01:19:23 +0300
List-Id: <debbugs-submit.debbugs.gnu.org>

This patch adds a Bash completion file discussed in the 45004 issue:

awk replaced with grep and sed;
added more Guix related subcommands;
fixed typo with 't' letter in 'for word in ${COMP_WORDS[*]}t'.

Tested with command:

    guix build -f guix.scm

which produced:

    /gnu/store/hr4rqm9wi9i2fpcd75zpqfd04vqavyif-shepherd-0.10.99-git/etc/bash_completion.d/herd

Oleg Pykhalov (1):
  Add Bash completion file.

 Makefile.am              |   4 ++
 configure.ac             |   9 ++++
 etc/completion/bash/herd | 104 +++++++++++++++++++++++++++++++++++++++
 3 files changed, 117 insertions(+)
 create mode 100644 etc/completion/bash/herd

base-commit: b9b15db9e5bda27287706b6727c462df686e7621
-- 
2.38.0