On Aug 3, 2006, at 11:29 PM, Albert Chin wrote:

On Thu, Aug 03, 2006 at 02:50:31PM +0200, Ralf Menzel wrote:
Albert Chin <[EMAIL PROTECTED]> writes:
Good catch. Updated patch attached.

2006-08-01  Albert Chin <[EMAIL PROTECTED]>

        * libtool.m4 (AC_LIBTOOL_POSTDEP_PREDEP) [ solaris ]:
        Don't set $postdeps to "-lCstd -lCrun" if
        "-library=stlport4" set in CXXFLAGS as stlport4 C++
        library incompatible with Cstd C++ library. Use
        '-library=Cstd -library=Crun' instead of '-lCstd -lCrun'.

2006-08-01  Albert Chin <[EMAIL PROTECTED]>

        * libltdl/m4/libtool.m4 (_LT_SYS_HIDDEN_LIBDEPS) [ solaris ]:
        Don't set $postdeps to "-lCstd -lCrun" if
        "-library=stlport4" set in CXXFLAGS as stlport4 C++
        library incompatible with Cstd C++ library. Use
        '-library=Cstd -library=Crun' instead of '-lCstd -lCrun'.

I found some problems with the patches:

1. In the patch for libtool-HEAD the 'for ... do' line is superflous.

2. _LT_AC_TAGVAR doesn't seem to work with libtool-HEAD. I used
   _LT_TAGVAR, which seems to work.

3. In libtool-HEAD the patch addresses the Sun C compiler for Linux
   and not for Solaris, while the patch for libtool-1-5 addresses the
   Solaris compiler and ignores the linux version.

The following patches seem to work for me:

Good catch. I should have tested the new patches! Updated patches
attached.


I just applied Ralf's patches, then saw this. It's okay with me as is, do you insist on the variable name change? I can see the value, but am tired of applying patches for this :)

Peter


Reply via email to