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 understands neither, though, so fixing this is
definitely necessary.

> +if test set = "${BASH_VERSION+set}${ZSH_VERSION}"; then
>      : ${lt_HAVE_ARITH_OP="yes"}
>      : ${lt_HAVE_XSI_OPS="yes"}

If you wanted, you could combine those two into a single statement:

: ${lt_HAVE_ARITH_OP=yes} ${lt_HAVE_XSI_OPS=yes}

-- 
Eric Blake   ebl...@redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to