Re: [PATCH] option detection failures

[Christian Cornelssen]

Hello,

On Tue, 29 Oct 2002, Christian Cornelssen wrote:

> It is indeed the same problem and the same fix.  Obviously, your
> patch has not been committed to the mainline as of 2002/10/25.  My
> patch is identical in that aspect.  I am still wondering which CVS
> branches are actively developed.

Well, Chuck's patch came in Oct 27 while my patch was based on the
Oct-25 revision of `libtool.m4'.  So I might have missed a commit.
But a "cvs update" from now has shown that no changes to `libtool.m4'
have been committed in the meantime.  This frees me from having to
"diff" again :-)

On Tue, 29 Oct 2002, Bob Friesenhahn wrote:

> As of the latest release, the libtool 1.4.X line is no longer
> maintained.  CVS MAIN is the only branch which is actively developed,
> in preparation for releasing 1.5.  All patches submitted should be
> against CVS head.

So I am on the right branch?  That relieves me.

I have tried to produce one of those horrible ChangeLog entries.  Here
is the result:

2002-10-29  Christian Cornelssen  <address@hidden>

        * libtool.m4 (AC_LIBTOOL_COMPILER_OPTION,
        AC_LIBTOOL_PROG_CC_C_O): Changed method for compilation with
        custom flags to be language-independent.  Thereby removed
        CFLAGS/GCJFLAGS mixup bug.  Imitate Autoconf-style logging.
        Log entries no longer discarded by compilation failures.
        (AC_LIBTOOL_PROG_CC_C_O): Replaced echo by printf to account
        for "\n"s (also reported Oct 27 by Charles Wilson).
        (_LT_AC_LANG_C_CONFIG, _LT_AC_LANG_CXX_CONFIG,
        _LT_AC_LANG_GCJ_CONFIG): Tuned settings of
        lt_simple_compile_test_code and lt_simple_link_test_code
        to avoid warnings that could break some option detections.
        (AC_LIBTOOL_PROG_COMPILER_NO_RTTI): Removed
        redundant "-c conftest.$ac_ext" in the argument to
        AC_LIBTOOL_COMPILER_OPTION.
        (AC_LIBTOOL_PROG_LD_SHLIBS): Replaced C-specific conftest
        source code with $lt_simple_compile_test_code, to support
        the "-lc" test with other languages.

Some of this may not be required for a ChangeLog entry, so you can
probably cut it down further.

Best regards,

Christian Cornelssen