All of lore.kernel.org
 help / color / mirror / Atom feed
From: Blue Swirl <blauwirbel@gmail.com>
To: "Andreas Färber" <andreas.faerber@web.de>
Cc: Bug 636315 <636315@bugs.launchpad.net>, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [Bug 636315] [NEW] configure and build errors on Solaris 10 due to /bin/sh usage
Date: Sun, 12 Sep 2010 21:05:25 +0000	[thread overview]
Message-ID: <AANLkTi=yRVCDiyGuYYZgVL5L6P-GO=S1-zvSyiqLnynm@mail.gmail.com> (raw)
In-Reply-To: <577C53A8-9AA7-4D71-98EF-A75F93A4F74B@web.de>

On Sun, Sep 12, 2010 at 5:58 PM, Andreas Färber <andreas.faerber@web.de> wrote:
> Am 12.09.2010 um 19:47 schrieb Blue Swirl:
>
>> On Sun, Sep 12, 2010 at 5:35 PM, Andreas Färber <andreas.faerber@web.de>
>> wrote:
>>>
>>> Am 12.09.2010 um 19:22 schrieb Blue Swirl:
>>>
>>>> What is the output of "sh ./tracetool --nop --check-backend"?
>>>
>>> ./tracetool: syntax error at line 51: `$' unexpected
>>
>> Does this patch fix the problem?
>>
>> diff --git a/tracetool b/tracetool
>> index 534cc70..c7582bf 100755
>> --- a/tracetool
>> +++ b/tracetool
>> @@ -48,7 +48,8 @@ get_argnames()
>> {
>>    local nfields field name
>>    nfields=0
>> -    for field in $(get_args "$1"); do
>> +    args=get_args "$1"
>> +    for field in "$args"; do
>
> This part yes. (I took the liberty of adding args to the local line above)
>
>>        nfields=$((nfields + 1))
>
> Next error is here:
> ./tracetool: syntax error at line 53: `nfields=$' unexpected

That looks like fully standards compliant, so Solaris' /bin/sh is not.
Can you try what happens with /usr/xpg4/bin/sh?

  reply	other threads:[~2010-09-12 21:05 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-12 11:26 [Qemu-devel] [Bug 636315] [NEW] configure and build errors on Solaris 10 due to /bin/sh usage Andreas Färber
2010-09-12 17:22 ` Blue Swirl
2010-09-12 17:35   ` Andreas Färber
2010-09-12 17:47     ` Blue Swirl
2010-09-12 17:58       ` Andreas Färber
2010-09-12 21:05         ` Blue Swirl [this message]
2010-09-12 22:02           ` Andreas Färber
2010-09-14 16:34             ` blueswirl
2010-09-14 20:37               ` Andreas Färber
2010-09-14 20:53                 ` Blue Swirl
2010-09-17 21:14                   ` Andreas Färber
2010-09-13  8:49           ` Michael Tokarev
2010-09-13 21:34             ` Andreas Färber
2010-09-14 16:41             ` Blue Swirl
2010-09-14 18:18               ` Michael Tokarev
2016-10-21 11:35 ` [Qemu-devel] [Bug 636315] " Thomas Huth

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='AANLkTi=yRVCDiyGuYYZgVL5L6P-GO=S1-zvSyiqLnynm@mail.gmail.com' \
    --to=blauwirbel@gmail.com \
    --cc=636315@bugs.launchpad.net \
    --cc=andreas.faerber@web.de \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.