Re: Call for testers: MPC-0.6 released

[Ralf Wildenhues <Ralf dot Wildenhues at gmx dot de>]

* Kaveh R. Ghazi wrote on Tue, Apr 07, 2009 at 05:32:14PM CEST:
>
> I think we're just missing AIX5.2.  Ralf you did that last time for the 
> svn tarball.  Would you please repeat your testing with mpc-0.6?

Sorry for the delay.  Issues on AIX 5.2:

- This time, I had an old version of mpc installed.  This caused this
failure:

Error: header and library do not match
mpc_get_version: "0.6-dev"
MPC_VERSION_STRING: "0.6"
FAIL: tget_version

Hmm, I don't quite see yet how that could have happened (and I ran "make
install" before analyzing this, so the failure is gone now.


- I still needed to set OBJECT_MODE=64 before running make,
otherwise libmpc is not created.

- These lines in configure are not portable:

   GMP_CC="`$CPP $CPPFLAGS conftest.c 2> /dev/null | $EGREP MPC_OPTION | $SED -e
 's/MPC_OPTION //g' | $SED -e 's/"//g'`"

  GMP_CFLAGS="`$CPP $CPPFLAGS conftest.c 2> /dev/null | $EGREP MPC_OPTION | $SED -e 's/MPC_OPTION //g'| $SED -e 's/"//g'`"

and cause these errors:
  sed: Function s/"//g"`" cannot be parsed.
  ../mpc-0.6/configure[4212]: //g| $SED -e s///g':  not found
  yes CC= CFLAGS=

  ../mpc-0.6/configure[4201]: syntax error at line 4258 : `(' unexpected

Please replace them with these lines, respectively:

  GMP_CC=`$CPP $CPPFLAGS conftest.c 2> /dev/null | $EGREP MPC_OPTION | $SED -e 's/MPC_OPTION //g' | $SED -e 's/"//g'`

  GMP_CFLAGS=`$CPP $CPPFLAGS conftest.c 2> /dev/null | $EGREP MPC_OPTION | $SED -e 's/MPC_OPTION //g'| $SED -e 's/"//g'`

You never need to put the right hand side of a shell assignment in
double quotes; see here for a detailed explanation:
<http://www.gnu.org/software/autoconf/manual/html_node/Shell-Substitutions.html>

(I thought we've already fixed this before?)

Cheers,
Ralf