Re: svn commit: r18040 - in trunk/subversion: libsvn_delta libsvn_fs_fs libsvn_ra_dav libsvn_ra_svn libsvn_repos libsvn_subr libsvn_wc svn tests/libsvn_delta

On Wed, 18 Jan 2006, Philip Martin wrote:

> Daniel Rall <dlr@collab.net> writes:
>
> > * configure.in
> > (OVERRIDE_CFLAGS): Add new substitution used when the compiler (CC)
> > is gcc, tacking "-Wno-uninitialized" onto the end of the
> > compilation flags. This additional substitution is necessary
> > because GCC 4.0.1 takes argument ordering of such flags into
> > consideration, and the flags may need to be positioned after
> > EXTRA_CFLAGS to take affect.
>
> Why should -Wno-uninitialized override EXTRA_CFLAGS? I thought that
> EXTRA_CFLAGS was *intended* to override the other CFLAGS.

Because the command-line positioning of gcc's compilation flags
impacts their interpretation, '-Wall -Werror -Wno-uninitialized'
works, while '-Wno-uninitialized -Wall -Werror' fails (I assume
because -Wall clobbers -Wno-uninitialized).

-- 
Daniel Rall