Re: Command substitution in here-documents

2018-11-19 Thread Herbert Xu
On Fri, Aug 17, 2018 at 09:08:35PM +0800, Herbert Xu wrote: > Ron Yorston wrote: > > I've been continuing to worry about this. > > > > In the old code (prior to commit 7c245aa) the TNL token that was > > detected in list() had actually been read. The current code uses > > peektoken(). > > > >

Re: Command substitution in here-documents

2018-08-16 Thread Ron Yorston
I've been continuing to worry about this. In the old code (prior to commit 7c245aa) the TNL token that was detected in list() had actually been read. The current code uses peektoken(). When parseheredoc() is called in interactive mode the last call to readtoken() is in the code to set the

Re: Command substitution in here-documents

2018-08-13 Thread Martijn Dekker
Op 13-08-18 om 18:30 schreef Ron Yorston: > Simon Ser wrote: >> On August 13, 2018 4:22 PM, Martijn Dekker wrote: >>> - The latest release is 0.5.10.2. I can't reproduce the bug at all in >>> 0.5.10, 0.5.10.1 or 0.5.10.2. >> >> This is interesting. I can reproduce with the latest commit in

Re: Command substitution in here-documents

2018-08-13 Thread Ron Yorston
Simon Ser wrote: >On August 13, 2018 4:22 PM, Martijn Dekker wrote: >> - The latest release is 0.5.10.2. I can't reproduce the bug at all in >> 0.5.10, 0.5.10.1 or 0.5.10.2. > >This is interesting. I can reproduce with the latest commit in master. I'm with Simon, the bug is present in

Re: Command substitution in here-documents

2018-08-13 Thread Simon Ser
Hi, Thanks both for your replies. On August 13, 2018 4:22 PM, Martijn Dekker wrote: > - The latest release is 0.5.10.2. I can't reproduce the bug at all in > 0.5.10, 0.5.10.1 or 0.5.10.2. This is interesting. I can reproduce with the latest commit in master. > - I can only reproduce

Re: Command substitution in here-documents

2018-08-13 Thread Martijn Dekker
Op 11-08-18 om 12:08 schreef Ron Yorston: > - Fixing this (other than using the blunt instrument of reverting the > faulty commit) is beyond my pay grade. Someone with a better > understanding of the code will need to take a look. My observations: - The latest release is 0.5.10.2. I can't

Re: Command substitution in here-documents

2018-08-11 Thread Ron Yorston
Simon Ser wrote: >I'm trying to use command substitution in here-documents. Here's my script: > > cat < `echo hi` > EOF > >This seems not to work in dash 0.5.9.1. It fails with this error: > > dash: 1: > : not found 'git bisect' points to commit 7c245aa8ed

Command substitution in here-documents

2018-08-10 Thread Simon Ser
Hi all, I'm trying to use command substitution in here-documents. Here's my script: cat < : not found According to the spec: > If no part of word is quoted, all lines of the here-document shall be > expanded for parameter expansion, command substitution, and arithmetic > expa