Open MPI logo

Open MPI Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |   all Development mailing list

From: Ralf Wildenhues (Ralf.Wildenhues_at_[hidden])
Date: 2007-08-06 16:48:13


Hello Jeff,

* Jeff Squyres wrote on Mon, Aug 06, 2007 at 04:27:59PM CEST:
> On Aug 5, 2007, at 3:41 PM, Ralf Wildenhues wrote:
>
> >> WHY: https://svn.open-mpi.org/trac/ompi/ticket/982 is fixed by newer
> >> Libtool snapshots (e.g., 1.2444 2007/04/10 is what I have installed
> >> at Cisco).
[...]
> > Asking because I don't think the bug was consciously fixed in Libtool;
> > only a test was added to expose the issue. I'll put it on my list of
> > things to look at.
[...]
> FWIW, note that we are applying this patch to the generated
> aclocal.m4 (in all versions -- it appears to apply cleanly with a
> little fuzz on the exact line numbering):

Ahh, yes, that was the patch that fixed the problem (rather than the
Autoconf upgrade), I remember now. Thanks for searching!

> --- aclocal.m4.old 2007-04-20 15:18:48.000000000 -0700
> +++ aclocal.m4 2007-04-20 15:18:59.000000000 -0700
> @@ -5311,7 +5311,7 @@
> # Commands to make compiler produce verbose output that lists
> # what "hidden" libraries, object files and flags are used when
> # linking a shared library.
> - output_verbose_link_cmd='$CC -shared $CFLAGS -v conftest.
> $objext 2>&1 | $GREP "\-L"'
> + output_verbose_link_cmd='$CC -shared $CFLAGS -v conftest.
> $objext 2>&1 | $GREP "\-L" | tail -n 1'
>
> else
> GXX=no
>
> This fixes the problem for us (we stole it from a libtool mailing
> list post from a long time ago). If this could be applied to the
> Libtool development trunk, that would be great... :-)

The patch has two issues. First a simple one, it should be
  sed -n '$p'
instead of `tail -n 1', for portability. Second, and more importantly,
I remember to have tested the patch on some but not all compilers that I
know do pretend to be g++ at times (icpc, pathCC?, pgCC?). I hope none
of them (nor g++ either) get the idea of splitting long output lines of
`$CXX -v' with backslash-newline.

Anyway, I'll put on my list throwing out another test round for the
patch.

Cheers,
Ralf