Re: 279-gary-LT_CONFIG_LTDL_DIR.diff

2005-10-03 Thread Gary V. Vaughan
Ralf Wildenhues wrote: Hi Gary, Hallo Ralf, (There should be a way for the user to specify "I am also content with an older installed libltdl version"; for example, by allowing to test for a different library function.) That's a whole other feature. I'm on the fence. Let's discuss it more

Re: 279-gary-LT_CONFIG_LTDL_DIR.diff

2005-10-02 Thread Ralf Wildenhues
Hi Gary, * Ralf Wildenhues wrote on Thu, Sep 29, 2005 at 06:39:28PM CEST: > * Gary V. Vaughan wrote on Thu, Sep 29, 2005 at 06:32:32PM CEST: > > Ralf Wildenhues wrote: > > > > This patch is just to introduce LT_CONFIG_LTDL_DIR without regressions. > > The rest of the fixes for LT_WITH_LTDL are ye

Re: 279-gary-LT_CONFIG_LTDL_DIR.diff

2005-09-30 Thread Gary V. Vaughan
Ralf Wildenhues wrote: Hi Gary, Hallo Ralf, Thanks for the re-review ;-) Okay to commit the attached? I really don't want this to be a neverending story, ;) but there are some more issues. Not at all. Here's a possible reason: grep _LTDL_DIR Makefile configure | Makefile:subdirs = _LTD

Re: 279-gary-LT_CONFIG_LTDL_DIR.diff

2005-09-29 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Thu, Sep 29, 2005 at 06:32:32PM CEST: > Ralf Wildenhues wrote: > > >I believe it is fixed with the following patch. OK to apply? > > > > * libtoolize.m4sh (func_included_files): Do not recurse > > non-existent files. > > Looks okay to me. App

Re: 279-gary-LT_CONFIG_LTDL_DIR.diff

2005-09-29 Thread Gary V. Vaughan
Ralf Wildenhues wrote: Hi Gary, Hallo Ralf! I think I can discount the following points... plus an okay for your patch... I'll repost 279 later when I've worked out the remaining known bugs. * Gary V. Vaughan wrote on Thu, Sep 29, 2005 at 11:19:41AM CEST: Ralf Wildenhues wrote: * Gary V.

Re: 279-gary-LT_CONFIG_LTDL_DIR.diff

2005-09-29 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Thu, Sep 29, 2005 at 11:19:41AM CEST: > Ralf Wildenhues wrote: > >* Gary V. Vaughan wrote on Tue, Sep 27, 2005 at 03:36:43PM CEST: > >>I'm not sure I understand how the aclocal.m4 problem came > >>to be: > > > >(The aclocal.m4 problem is orthogonal; let's discus

Re: 279-gary-LT_CONFIG_LTDL_DIR.diff

2005-09-29 Thread Gary V. Vaughan
Ralf Wildenhues wrote: Hi Gary, Hallo Ralf, Thanks for the review. New version of the patch attached. * Gary V. Vaughan wrote on Tue, Sep 27, 2005 at 03:36:43PM CEST: I'm not sure I understand how the aclocal.m4 problem came to be: (The aclocal.m4 problem is orthogonal; let's discuss it

Re: 279-gary-LT_CONFIG_LTDL_DIR.diff

2005-09-28 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Tue, Sep 27, 2005 at 03:36:43PM CEST: > > Thanks for the review. This fixes the inline LT_CONFIG_LTDL_DIR problem > you report, but I'm not sure I understand how the aclocal.m4 problem came > to be: This patch looks much better, but still has a bug, see below

279-gary-LT_CONFIG_LTDL_DIR.diff

2005-09-27 Thread Gary V. Vaughan
Hallo Ralf, Thanks for the review. This fixes the inline LT_CONFIG_LTDL_DIR problem you report, but I'm not sure I understand how the aclocal.m4 problem came to be: Ralf Wildenhues wrote: > ++ cat aclocal.m4 ./foo/libltdl/configure.ac How did configure_ac get set to ./foo/libltdl/configure.ac?

Re: 279-gary-LT_CONFIG_LTDL_DIR.diff

2005-09-26 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Fri, Sep 23, 2005 at 02:26:24PM CEST: > Here is the first part of the patch series to fix the LT_WITH_INIT > interfaces. Okay to commit? No, unfortunately. Observed bug below. > This changeset brings some sanity to tracking the directory name for > libltdl s

Re: 279-gary-LT_CONFIG_LTDL_DIR.diff

2005-09-23 Thread Gary V. Vaughan
Ralf Wildenhues wrote: Hi Gary, Hallo Ralf, * Gary V. Vaughan wrote on Fri, Sep 23, 2005 at 02:26:24PM CEST: Here is the first part of the patch series to fix the LT_WITH_INIT interfaces. Okay to commit? This changeset brings some sanity to tracking the directory name for libltdl sources.

Re: 279-gary-LT_CONFIG_LTDL_DIR.diff

2005-09-23 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Fri, Sep 23, 2005 at 02:26:24PM CEST: > Here is the first part of the patch series to fix the LT_WITH_INIT > interfaces. Okay to commit? > > This changeset brings some sanity to tracking the directory name for > libltdl sources. Old syntax still works, but th

Re: 279-gary-LT_CONFIG_LTDL_DIR.diff

2005-09-23 Thread Gary V. Vaughan
ChangeLog: Index: libtool--devo--1.0/ChangeLog from Gary V. Vaughan <[EMAIL PROTECTED]> * libltdl/m4/ltdl.m4: Bump serial as we are changing the interface. (LT_CONFIG_LTDL_DIR): New macro to centralise setting the subdirectory used for libltdl. (LTDL_CONV

279-gary-LT_CONFIG_LTDL_DIR.diff

2005-09-23 Thread Gary V. Vaughan
Here is the first part of the patch series to fix the LT_WITH_INIT interfaces. Okay to commit? This changeset brings some sanity to tracking the directory name for libltdl sources. Old syntax still works, but the documentation now talks only about using the new LT_CONFIG_LTDL_DIR. If any of the