Re: FYI: 333-gary-refactor-LTDL_INIT.patch

[Ralf Wildenhues]

Hi Gary,

* Gary V. Vaughan wrote on Tue, Jan 08, 2008 at 05:23:24AM CET:
[...]
>>>  Index: ChangeLog
>>>  from  Gary V. Vaughan  <address@hidden>
[...]
>>>     * libltdl/m4/libtool.m4 (LT_INIT): Declare that this macro must be
>>>     expanded before LTDL_INIT, and be sure to parse caller options.

This is contrary to what I think we recommended in earlier 1.9X times
(but I may be misremembering), it is contrary to the code in
subproject.at (plain oversight?), and it is for example done the other
way round in GraphicsMagick, but OTOH not in
branch-1-5/libltdl/configure.ac.

So I ask to please clarify once and for all the order in which things
should be (including a notice in NEWS), the rationale for it, and please
to make it clear in the manual whether LTDL_INIT requires that LT_INIT
also be called or not (some examples given in the manual carry LTDL_INIT
but do not mention LT_INIT at all).

I have some pending patch for which I need to know the order.

Thank you,
Ralf

Index: tests/subproject.at
===================================================================
RCS file: /cvsroot/libtool/libtool/tests/subproject.at,v
retrieving revision 1.13
diff -u -r1.13 subproject.at
--- tests/subproject.at 8 Jan 2008 05:07:41 -0000       1.13
+++ tests/subproject.at 12 Jan 2008 14:14:48 -0000
@@ -33,8 +33,8 @@
 AC_CONFIG_AUX_DIR([sub/ltdl/config])
 AC_CONFIG_MACRO_DIR([sub/ltdl/m4])
 AM_INIT_AUTOMAKE([foreign])
-LT_WITH_LTDL
 LT_INIT
+LT_WITH_LTDL
 AC_CONFIG_FILES([Makefile])
 AC_OUTPUT
 ]])