On 12/17/2011 10:22 PM, Gary V. Vaughan wrote:
libtool: minimise forks per invocation under bash.
* build-aux/general.m4sh (lt_HAVE_PLUSEQ_OP, lt_HAVE_ARITH_OP)
(lt_HAVE_XSI_OPS): Set these without forking a test script when
running under bash, to avoid a few unnecessary
On 12/18/2011 12:46 AM, Gary V. Vaughan wrote:
Pushed as obvious.
Dash shipped with Ubutu-11.10 as /bin/sh, among others, still
has a crippled echo builtin that mis-handles backslashes.
That's an unfair characterization. Rather, dash ships an echo builtin
that complies with POSIX by default
On 12/18/2011 06:33 AM, Gary V. Vaughan wrote:
Can anyone think of something better than just removing the whole
lt_HAVE_PLUSEQ_OP
clause from the patch quoted above, and letting the shell figure it by
itself later
on?
Adding an extra eval seems to do the trick:
Yes - hiding behind eval
On 12/18/2011 06:49 AM, Gary V. Vaughan wrote:
* build-aub/general.m4sh (lt_HAVE_PLUSEQ_OP): Instead of using
$((..)) arithmetic, which causes an error on dash, use a case
based bash version check.
Dash understands $(( )). What it doesn't understand is ${BASH_VERSINFO[0]}.
Solaris /bin/sh
Hi Eric.
On 12/19/2011 02:44 PM, Eric Blake wrote:
On 12/17/2011 10:22 PM, Gary V. Vaughan wrote:
We should try to minimise forks, especially on Windows where they are
+# unreasonably slow, so skip the feature probes when bash is being used:
+if test set = ${BASH_VERSION+set}; then
+: