GNU bug report logs - #63860
[PATCH] Shepherd: Add Bash completion file.

Previous Next

Package: guix-patches;

Reported by: Oleg Pykhalov <go.wigust <at> gmail.com>

Date: Fri, 2 Jun 2023 22:20:01 UTC

Severity: normal

Tags: patch

Done: Ludovic Courtès <ludo <at> gnu.org>

Bug is archived. No further changes may be made.

Full log

Message #23 received at 63860 <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: Oleg Pykhalov <go.wigust <at> gmail.com>
Cc: 63860 <at> debbugs.gnu.org
Subject: Re: [PATCH v2] Add Bash completion file.
Date: Sat, 03 Jun 2023 23:42:05 +0200

Am Samstag, dem 03.06.2023 um 22:28 +0300 schrieb Oleg Pykhalov:
> Liliana Marie Prikler <liliana.prikler <at> gmail.com> writes:
> 
> […]
> 
> > > +{
> > > +    local command="${COMP_WORDS[1]}"
> > > +    case "$command" in
> > > +        discover)
> > > +            complitions="guix-daemon"
> > > +            mapfile -t COMPREPLY < <(compgen -W "$complitions" -
> > > -
> > > "${COMP_WORDS[$COMP_CWORD]}")
> > > +            ;;
> > Is there a reason to call the variable "complitions" rather than
> > "completions"
> 
> No reason, it's a typo probably from autocompletion of my editor. :-)
> 
> > or using an immediate value?
> 
> Do you mean to use the following?
> --8<---------------cut here---------------start------------->8---
>     case "$command" in
>         discover)
>             mapfile -t COMPREPLY < <(compgen -W guix-daemon --
> "${COMP_WORDS[$COMP_CWORD]}")
>             ;;
>     ...
> --8<---------------cut here---------------end--------------->8---
> 
> yes, this will work.
I'd still quote "guix-daemon", but yeah.

Cheers
This bug report was last modified 1 year and 352 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.