FYI: HEAD: Kill two showstoppers at once

Hi Gary, * Gary V. Vaughan wrote on Wed, Aug 31, 2005 at 02:45:32PM CEST: Ralf Wildenhues wrote: Ah, okay. We just need to move the _LT_SET_OPTIONS up the expansion stack. If it didn't take arguments we could just AC_REQUIRE a wrapper, Isn't that what the optional second argument to

FYI: HEAD: Kill two showstoppers at once

Hi Gary, * Gary V. Vaughan wrote on Tue, Aug 30, 2005 at 07:37:15PM CEST: On 30 Aug 2005, at 17:51, Ralf Wildenhues wrote: * Gary V. Vaughan wrote on Tue, Aug 30, 2005 at 06:31:42PM CEST: Ralf Wildenhues wrote: see updated patch at end. Oops. Thanks! I have applied that one.

Re: HEAD: Kill two showstoppers at once

Slight omission: * Ralf Wildenhues wrote on Tue, Aug 30, 2005 at 06:51:44PM CEST: --- libltdl/m4/ltoptions.m4 30 Aug 2005 14:31:40 - 1.2 +++ libltdl/m4/ltoptions.m4 30 Aug 2005 16:46:52 - @@ -20,9 +20,15 @@ # _LT_SET_OPTION(NAME) -# --

Re: FYI: HEAD: Kill two showstoppers at once

Hallo Ralf! Ralf Wildenhues wrote: I have applied that one. Okay, thanks. Ah, okay. We just need to move the _LT_SET_OPTIONS up the expansion stack. If it didn't take arguments we could just AC_REQUIRE a wrapper, Isn't that what the optional second argument to AC_REQUIRE could be used

Re: HEAD: Kill two showstoppers at once

* Ralf Wildenhues wrote on Tue, Aug 30, 2005 at 01:46:42PM CEST: The ltoptions machinery has one little bug. Two showstoppers are the result of this: AC_DISABLE_STATIC not working, and AC_LIBTOOL_WIN32_DLL not working. (The other old-style option setting macros don't work either, I believe):

Re: HEAD: Kill two showstoppers at once

Oops, sorry. Typo alert: Gary V. Vaughan wrote: # _LT_SET_OPTION(NAME) # -- # Set option NAME, and if there is a matching handler defined, # dispatch to it. Other NAMEs are saved as a flag. m4_define([_LT_SET_OPTION], [m4_define(_LT_MANGLE_OPTION([$1]))dnl

Re: HEAD: Kill two showstoppers at once

Hi Gary, * Gary V. Vaughan wrote on Tue, Aug 30, 2005 at 06:31:42PM CEST: Ralf Wildenhues wrote: Just a slight cleanup of your suggestion. Changes: change m4_fatal to m4_warn, because we want to be forward compatible, Agreed. I also had to add a few newlines, else we'd end up with | #

Re: HEAD: Kill two showstoppers at once

On 30 Aug 2005, at 17:51, Ralf Wildenhues wrote: Hi Gary, Hawdy! * Gary V. Vaughan wrote on Tue, Aug 30, 2005 at 06:31:42PM CEST: Ralf Wildenhues wrote: Just a slight cleanup of your suggestion. Changes: change m4_fatal to m4_warn, because we want to be forward compatible, Agreed. I