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

[Ralf Wildenhues]

* Ralf Wildenhues wrote on Tue, Jan 08, 2008 at 08:29:05PM CET:
> * Gary V. Vaughan wrote on Tue, Jan 08, 2008 at 05:23:24AM CET:
> > On 8 Jan 2008, at 04:40, Ralf Wildenhues wrote:
> >
> >> With this patch, I see in the log of
> >>  make distcheck
> >>
> >> that the libltdl subdirectory is being configured.  This is wrong, as
> >> libltdl in the Libtool package should be in nonrecursive mode rather
> >> than in subproject mode.  This causes distcheck to fail.  Can you fix
> >> this?
> >
> > I noticed this yesterday when I started the alpha release process too.
> > Yes, I'll figure this out before I go any further.
> 
> It is because _LTDL_SETUP calls _LTDL_MODE_DISPATCH which calls
> AC_CONFIG_SUBDIRS.  That's wrong, no?  Is that because implicitly
> subproject-libltdl mode is assumed for the Libtool package?

OK to apply this fix (feeling a bit hackish, but does exactly what's
needed)?

Cheers,
Ralf

2008-01-14  Ralf Wildenhues  <address@hidden>

        * configure.ac: Set libltdl nonrecursive option, to avoid
        configuring libltdl inside the Libtool package.

Index: configure.ac
===================================================================
RCS file: /cvsroot/libtool/libtool/configure.ac,v
retrieving revision 1.79
diff -u -r1.79 configure.ac
--- configure.ac        6 Jan 2008 16:33:30 -0000       1.79
+++ configure.ac        14 Jan 2008 19:24:05 -0000
@@ -181,6 +181,7 @@
 ## Libtool initialisation. ##
 ## ----------------------- ##
 LT_INIT([dlopen win32-dll])
+_LT_SET_OPTION([LTDL_INIT], [nonrecursive])
 _LTDL_SETUP
 
 # Enable all the language support we can