Re: [PATCH] Add nofirstword completion option

2018-06-12 Thread Luca Boccassi
On Tue, 2018-06-12 at 14:54 -0400, Chet Ramey wrote: > On 6/12/18 1:28 PM, Luca Boccassi wrote: > > On Fri, 2018-06-08 at 11:03 -0400, Chet Ramey wrote: > > > On 6/8/18 10:40 AM, Luca Boccassi wrote: > > > > > > > Any chance you had a sec to look at the diff? Would love some > > > > feedback! > >

Re: [PATCH] Add nofirstword completion option

2018-06-12 Thread Chet Ramey
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 t

Re: [PATCH] Add nofirstword completion option

2018-06-12 Thread Chet Ramey
On 6/12/18 1:28 PM, Luca Boccassi wrote: > On Fri, 2018-06-08 at 11:03 -0400, Chet Ramey wrote: >> On 6/8/18 10:40 AM, Luca Boccassi wrote: >> >>> Any chance you had a sec to look at the diff? Would love some >>> feedback! >> >> I'm going to try and look at it this weekend. It's been a busy couple

Re: [PATCH] Add nofirstword completion option

2018-06-12 Thread Luca Boccassi
On Fri, 2018-06-08 at 11:03 -0400, Chet Ramey wrote: > On 6/8/18 10:40 AM, Luca Boccassi wrote: > > > Any chance you had a sec to look at the diff? Would love some > > feedback! > > I'm going to try and look at it this weekend. It's been a busy couple > of > months. Hi Chet, I've seen that the

Re: Case of set -e not being in effect in a subshell.

2018-06-12 Thread Bartłomiej Palmowski
After so many years of using bash I assumed that there is nothing special to set -e that I wouldn't know so I didn't bother to look it up in the docs, my bad. Thanks for the answers! 2018-06-11 16:09 GMT+01:00 Chet Ramey : > On 6/11/18 6:12 AM, Bartłomiej Palmowski wrote: > > > > > The issue is: