Re: [SCM] GNU Libtool branch, master, updated. v2.4.2-141-g4099c12

2011-12-19 Thread Eric Blake
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

Re: FYI: [PATCH] bootstrap: adopt autoconf echo normalization code.

2011-12-19 Thread Eric Blake
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

Re: FYI [PATCH] libtool: minimise forks per invocation under bash.

2011-12-19 Thread Eric Blake
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

Re: FYI: [PATCH] libtool: make fork minimisation compatible with dash and zsh.

2011-12-19 Thread Eric Blake
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

Re: [SCM] GNU Libtool branch, master, updated. v2.4.2-141-g4099c12

2011-12-19 Thread Stefano Lattarini
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 +: