On 2/1/2012 10:12, Rafaël Carré 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. >
OK, sure. IDL regen looks unrelated to installing additional headers. >> 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. > > --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] > > > Attached patch applies on top of first patch and just change the > default, but I still prefer those other ones (separate --enable-ddk / > --enable-directx) > > IMHO, keep the "all" in the AS_HELP_STRING output, since it is a valid argument.
signature.asc
Description: OpenPGP digital signature
------------------------------------------------------------------------------ Keep Your Developer Skills Current with LearnDevNow! The most comprehensive online learning library for Microsoft developers is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3, Metro Style Apps, more. Free future releases when you subscribe now! http://p.sf.net/sfu/learndevnow-d2d
_______________________________________________ Mingw-w64-public mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
