Re: [Mingw-w64-public] [PATCH 1/4] Move IDL generation option from --enable-sdk=idl to --enable-idl

[Rafaël C. <raf...@gm...>]

Le 2012-02-01 07:41, Earnie Boyd a écrit :
> On Tue, Jan 31, 2012 at 9:12 PM, Rafaël Carré <fun...@pu...> wrote:
>> Le 2012-01-31 01:57, JonY a écrit :
>>> On 1/31/2012 11:07, Rafaël Carré wrote:
>>>> ---
>>>>  mingw-w64-headers/configure.ac |   19 +++++++++++--------
>>>>  1 files changed, 11 insertions(+), 8 deletions(-)
>>>>
>>>
>>> What is the benefit of it being split up?
>>
>> Kai wanted to split up at list IDL to keep it disabled.
>>
>>> If possible keep the options
>>> as is, just enable them by default, eg --enable-sdks=all by default.
>>
>> I think --enable-sdks="ddk,directx" is a strange way to use configure.
>>
> 
> No, it is acceptable as per your output below.
> 
>> --disable-foo --enable-bar is the standard way to use autoconf, and this
>> makes the configure.ac code smaller.
>>
>> ./configure --help says:
>>
>> Optional Features:
>>
>>  --disable-FEATURE       do not include FEATURE (same as
>> --enable-FEATURE=no)
>>  --enable-FEATURE[=ARG]  include FEATURE [ARG=yes]
>>
> 
> --enable-FEATURE[=ARG] and you're giving an ARG of "ddk,directx".
> Even GCC uses an ARG other than "yes".
> 
>   --enable-cloog-backend[=BACKEND]
>                           set the CLooG BACKEND used to either isl, ppl or
>                           ppl-legacy (default)
>   --enable-stage1-languages[=all]
>                           choose additional languages to build during stage1.
>                           Mostly useful for compiler development
> 
> 

Yes it makes sense, in fact I am just not used to it :)