Martin Furter wrote:
>
>
> On Sun, 22 Nov 2009, Rainer Jung wrote:
>
>> On 22.11.2009 18:18, Arfrever Frehtes Taifersar Arahesis wrote:
>>> 2009-11-22 18:06:40 Martin Furter napisał(a):
>>>>
>>>> On Sun, 22 Nov 2009, Arfrever Frehtes Taifersar Arahesis wrote:
>>>>
>>>>> 2009-11-22 06:39:05 Martin Furter napisał(a):
>>>>>>
>>>>>> On Sun, 22 Nov 2009, Arfrever Frehtes Taifersar Arahesis wrote:
>>>>>>
>>>>>>> 2009-11-22 03:43:44 Martin Furter napisał(a):
>>>>>>>>
>>>>>>>> Little bugfix...
>>>>>>>>
>>>>>>>> Thanks,
>>>>>>>> Martin
>>>>>>>>
>>>>>>>>
>>>>>>>> [[[
>>>>>>>> Change the shebang line from /bin/sh to /bin/bash because the
>>>>>>>> script
>>>>>>>> contains bashims which do not work with /bin/sh on various UNICES.
>>>>>>>
>>>>>>> This script doesn't have any bashisms. What exactly causes
>>>>>>> problems?
>>>>>>
>>>>>> $(...) is a bashism
>>>>>
>>>>> No. $(command) is documented in POSIX:
>>>>> http://www.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_06_03
>>>>>
>>>>
>>>> Fuck POSIX.
>>>>
>>>> $ /bin/sh -c 'echo $(date)'
>>>> /bin/sh: syntax error at line 1: `(' unexpected
>>>> $ uname -a
>>>> SunOS enterprise 5.9 Generic_118558-22 sun4u sparc SUNW,Ultra-250
>>>
>>> Please report a bug to developers of SunOS shell.
>>
>> Solaris contains a posix compatible shell as /usr/xpg4/bin/sh. It's
>> mentioned in the sh man page and also in "man -s 5 standards".
>>
>> A common workaround is to set CONFIG_SHELL=/bin/ksh before doing
>> configure on Solaris.
>
> Good to know, thanks.
>
> But 1.4 compiled fine without any problems at all. So it's clearly a
> regression and must be fixed in subversion.
Still, running the script throught bash is not the righ solution,
because you add a dependency on bash that wasn't there before. Whilst
almost no-one will notice on Linux, lots of people will notice on
solaris, which IIRC doesn't have bash installed by default.

Hint: do not use $(command), use `command` instead. The quoting rules
are slightly different, but it's guaranteed to work on any flavour of
Unix shell.

-- Brane

Reply via email to