Hi, On Wed, Sep 02, 2009 at 12:36:29PM +0800, Alan Coopersmith wrote: > The one additional thought I have is should this take an optional > argument for the default value, so that maintainers can make it be > on by default in their modules on a case-by-case basis? > (Perhaps with arguments of "yes", "no" (the default), and "git" > (yes if $(top_srcdir)/.git exists, otherwise no).)
No. People are going to go 'man my module is literally perfect, I'm going to enable -pedantic -Werror'. Then someone will build with a newer gcc, or find some crap headers from an external project we depend on, or the insane way we have to screw around with our build environment will mean that under some systems, we're going to get warnings. And the build will fail. It's a very useful development tool, don't get me wrong, but we can't ever ship a tarball with -Werror enabled. Cheers, Daniel
pgpzMGQq1NLh0.pgp
Description: PGP signature
_______________________________________________ xorg-devel mailing list [email protected] http://lists.x.org/mailman/listinfo/xorg-devel
