Re: [PATCH] Add nofirstword completion option

[Luca Boccassi]

On Tue, 2018-06-12 at 15:18 -0400, Chet Ramey wrote:
> On 6/12/18 3:15 PM, Luca Boccassi wrote:
> 
> > In case it can be somewhat useful, once 5.0-beta with the new
> > option is
> > out I'll ask QA at $work to run regressions tests on it and report
> > back
> > in case any issues are found. They have a lot of tests for the
> > command
> > line interface (not just related to completion), so perhaps it
> > could
> > help a little with testing the new version.
> 
> That would be great, thanks.

Hi Chet,

Found another small issue: the INITIALWORD Iflag in complete.def is not
initialised in the compopt_builtin function, which means it sometimes
evaluates to true and spuriously triggers a loookup for the INITIALWORD
completion even though it's not set, printing an error, for example
trying to complete "sh":

$ sh-bash: compopt: _InitialWorD_: no completion specification

The fix is quite simple and it's inlined below.

Thanks!

-- 
Kind regards,
Luca Boccassi

--- a/builtins/complete.def
+++ b/builtins/complete.def
@@ -831,7 +831,7 @@ compopt_builtin (list)
   WORD_LIST *l, *wl;
   COMPSPEC *cs;
 
-  opts_on = opts_off = Eflag = Dflag = 0;
+  opts_on = opts_off = Eflag = Dflag = Iflag= 0;
   ret = EXECUTION_SUCCESS;
 
   reset_internal_getopt ();
-- 
2.17.1

signature.asc
Description: This is a digitally signed message part