Problem with command substitution in mksh shell after upgrading to Cygwin 1.7.26

PRIKHODKO, GEORGE gp1865@att.com
Wed Dec 4 20:05:00 GMT 2013


Hi,

The fix did help. Thank you.

Thanks,
George 

> -----Original Message-----
> Of Corinna Vinschen
> Sent: Wednesday, December 04, 2013 6:17 AM
> Subject: Re: Problem with command substitution in mksh shell after upgrading to
> Cygwin 1.7.26
> 
> On Dec  4 10:13, Corinna Vinschen wrote:
> > On Dec  4 03:22, PRIKHODKO, GEORGE wrote:
> > > Hi,
> > >
> > > After upgrading to Cygwin to 1.7.26 the commands:
> > >
> > > print $(getIP)
> > > print $(ipconfig | grep "IPv4 Address" | sort | head -1 | cut -d ":" -f 2 \
> > >     | sed -e "s/ *\\([0-9.]*\\).*/\\1/")
> > >
> > > in the following script
> > >
> > > #!/usr/bin/ksh
> > >
> > > cv=$(uname -r)
> > > cv=${cv%\(*}
> > > cygcheck -s -v -r > cygcheck_$cv.out
> > >
> > > exec > test_$cv.log 2>&1
> > > set -xv
> > >
> > > getIP() {
> > >     ipconfig | grep "IPv4 Address" | sort | head -1 | cut -d ":" -f 2 \
> > >         | sed -e "s/ *\\([0-9.]*\\).*/\\1/"
> > > }
> > > #
> > > # the following four commands should print IP address 4 times
> > > #
> > > getIP
> > >
> > > print $(getIP)
> > >
> > > ipconfig | grep "IPv4 Address" | sort | head -1 | cut -d ":" -f 2 \
> > >     | sed -e "s/ *\\([0-9.]*\\).*/\\1/"
> > >
> > > print $(ipconfig | grep "IPv4 Address" | sort | head -1 | cut -d ":" -f 2 \
> > >     | sed -e "s/ *\\([0-9.]*\\).*/\\1/")
> > >
> > > exit
> > >
> > > started to produce incorrect results.
> > > test_1.7.26.log shows instead of IP address a whole output of ipconfig is
> printed for these lines.
> > >
> > > /usr/bin/ksh, used in the script, is a symbolic link to /usr/bin/mksh
> > >
> > > Please, let me know if you need more info to research the problem.
> >
> > No, thank you.  I could track down the culprit of the problem.
> > I don't have a fix yet, but it will be fixed in 1.7.27, which
> > will be due really soon.
> 
> The fix was already in CVS.  Please try the latest snapshot from
> http://cygwin.com/snapshots/, it should already contain the patch.
> 
> 
> Thanks,
> Corinna
> 
> --
> Corinna Vinschen                  Please, send mails regarding Cygwin to
> Cygwin Maintainer                 cygwin AT cygwin DOT com
> Red Hat


More information about the Cygwin mailing list