[libvirt] [PATCH] Enable all warnings permanently & default to -Werror for GIT builds
Eric Blake
eblake at redhat.com
Tue Mar 27 15:59:28 UTC 2012
On 03/27/2012 09:50 AM, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange" <berrange at redhat.com>
>
> Given that we auto-detect whether each -Wxxxx flag is supported by
> GCC, and we are warning-free and use automake silent rules, there
> is no compelling reason to allow compile warnings to be disabled.
>
> Replace the --enable-compile-warnings flag with a simpler
> --enable-werror flag, which defaults to 'yes' if building
> from GIT, or 'no' if building from tar.gz
>
> This helps ensure that everyone writing patches for libvirt will
> take care to fix their warning problems before submitting for
> review
>
> * autobuild.sh: Force -Werror
> * configure.ac: Update for LIBVIRT_COMPILE_WARNINGS macro change
> * m4/virt-compile-warnings.m4: Permanently enable all warnings,
> auto-enable Werror for GIT builds
> ---
> autobuild.sh | 4 +-
> configure.ac | 4 +-
> m4/virt-compile-warnings.m4 | 258 +++++++++++++++++++++----------------------
> 3 files changed, 130 insertions(+), 136 deletions(-)
ACK. Most of this was an indentation change; the 'git diff -b' view
makes it easier to review.
--
Eric Blake eblake at redhat.com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120327/96cf309d/attachment-0001.sig>
More information about the libvir-list
mailing list