Re: namref dicussion continued: [PATCH] export -r reference

2016-05-08 Thread Dan Douglas
On Sun, May 8, 2016 at 6:03 PM, Chet Ramey wrote: > On 5/5/16 6:02 PM, Dan Douglas wrote: >> On Thu, May 5, 2016 at 2:46 AM, Piotr Grzybowski wrote: >>> ok, so Dan wants this patch. >> >> Yes I think your bash patch is probably an improvement at least in >> this particular case. >> >>> Maybe you

Re: IFS as the control variable in a for loop

2016-05-08 Thread Chet Ramey
On 5/2/16 3:06 PM, Grisha Levit wrote: > When in a non-interactive shell, IFS, when modified as the control variable > in a for loop does not change splitting behavior. > $ bash -c 'for IFS in .; do printf "%s\n" "$IFS" "$*"; done' bash x x > . > x x > > $ bash -ic 'for IFS in .; do printf "%s\n"

Re: History modifier :p ignored prior to another history reference

2016-05-08 Thread Chet Ramey
On 5/5/16 2:08 PM, Piotr Grzybowski wrote: > Hey, > > confirmed, this is what happens by construction. > this one is going to be tricky, expansion happens on the shell line, way > before it comes to execution. I do not see reparsing again at execute_command > even if we had means, which we don

Re: History modifier :p ignored prior to another history reference

2016-05-08 Thread Chet Ramey
On 5/3/16 12:15 PM, Dean Stanton wrote: > History modifier :p is not honored when it appears prior to another history > reference on the same command line. csh and tcsh appear to make the :p modifier apply to every history expansion on the line if it appears anywhere in the line. I'll take a loo

Re: set isearch-terminators "\r" in .inputrc causes malloc assert fails/death

2016-05-08 Thread Chet Ramey
On 5/4/16 1:43 PM, Britton Kerin wrote: > I tried with export INPUTRC=test_inputrc, where test_inputrc contains > just this: > > set isearch-terminators "\r" > > this causes bash to sometimes spit out things like this: I can't reproduce this with bash-4.3.42 on RHEL 7, and there's independent co

Re: namref dicussion continued: [PATCH] export -r reference

2016-05-08 Thread Chet Ramey
On 5/4/16 12:30 PM, Piotr Grzybowski wrote: > Hey, > > after another discussion with Grisha, and following one of his remarks, I > kindly request your opinion on the attached patch (like: bad, not needed, > already done, weak as it stands, harmful, etc.). It addresses the exporting > of identi

Re: namref dicussion continued: [PATCH] export -r reference

2016-05-08 Thread Chet Ramey
On 5/5/16 6:02 PM, Dan Douglas wrote: > On Thu, May 5, 2016 at 2:46 AM, Piotr Grzybowski wrote: >> ok, so Dan wants this patch. > > Yes I think your bash patch is probably an improvement at least in > this particular case. > >> Maybe you can comment on wether the patches are valid. > > The pos

Re: set isearch-terminators "\r" in .inputrc causes malloc assert fails/death

2016-05-08 Thread Piotr Grzybowski
On 5 May 2016, at 13:48, Eduardo A. Bustamante López wrote: > I'm able to replicate with the master branch. [..] yes, it appeared on the master with: commit ac50fbac377e32b98d2de396f016ea81e8ee9961 Date: Wed Feb 26 09:36:43 2014 -0500 Bash-4.3 distribution sources and documentation at t