[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Patch to make configure /bin/sh compatible
From: |
Michael McConnell |
Subject: |
Re: [Qemu-devel] Patch to make configure /bin/sh compatible |
Date: |
Fri, 14 Apr 2006 18:38:27 +0100 (BST) |
On Fri, 14 Apr 2006, Thiemo Seufer wrote:
> Ben Taylor wrote:
> > This is a patch to make configure truly /bin/sh compatible,
> > as well as changing some logic to remove "echo -n" for
> > output to files, as "echo -n" is not consistent among
> > environments.
> >
> > This is against the latest cvs branch, and I appreciate
> > any feedback.
>
> [snip]
> > @@ -294,7 +295,7 @@
> > target_list="i386-user arm-user armeb-user sparc-user ppc-user
> > mips-user mipsel-user $target_list"
> > fi
> > else
> > - target_list=$(echo "$target_list" | sed -e 's/,/ /g')
> > + target_list=`echo "$target_list" | sed -e 's/,/ /g'`
>
> A standard-conforming /bin/sh should be capable of handling this.
I've seen various /bin/sh (and ksh) versions that will accept the `....` form
but not the $(...) form. It's bitten me a few times too.
-- Michael "Soruk" McConnell
Eridani Star System
MailStripper - http://mailstripper.eridani.co.uk/ - SMTP spam filter
Mail Me Anywhere - http://www.MailMeAnywhere.com/ - Mobile email