Re: Patch: Make HEAD compile with standard C compiler

On Wed, 8 Feb 2006, Daniel Berlin wrote:

> On Wed, 2006-02-08 at 18:18 -0800, Greg Stein wrote:
> > On Wed, Feb 08, 2006 at 03:38:42PM -0500, Daniel Berlin wrote:
> > > On Wed, 2006-02-08 at 17:00 +0000, Julian Foad wrote:
> > >...
> > > I'm used to bootstrapping gcc, where we have -Werror, -Wunused, etc on
> > > by default, so it catches this stuff for me :(
> > >
> > > I'm not sure why we don't do the same. We could rip the configure magic
> > > that detects whether it is supported or not by the compiler directly
> > > from gcc's configure.
> >

I think -Werror is problematic because different compiler versions warn
for different things. I've had problems with warnings in APR (or Apache)
headers, which I don't control. Making warnings break others built might
be too much.

Thanks,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Feb 9 11:09:50 2006