On 12/19/2012 12:35 PM, Daniel P. Berrange wrote:
On Wed, Dec 19, 2012 at 12:24:39PM -0700, Eric Blake wrote:
> Commit 8b8fcdea introduced a check for broken gcc -Wlogical-op,
> but did not guard the check against non-gcc compilers, which might
> lead to spurious failures when another compiler encounters an
> unknown pragma. Additionally, all of our compiler warning logic
> should belong in a single file, and use cache variables to allow
> overriding the decision at configure time if necessary.
>
> * configure.ac (BROKEN_GCC_WLOGICALOP): Move...
> * m4/virt-compile-warnings.m4 (LIBVIRT_COMPILE_WARNINGS): ...here,
> and update to modern autoconf idioms.
> ---
>
> Fixes the issues mentioned here:
>
https://www.redhat.com/archives/libvir-list/2012-December/msg01197.html
>
https://www.redhat.com/archives/libvir-list/2012-December/msg01211.html
>
> configure.ac | 23 -----------------------
> m4/virt-compile-warnings.m4 | 22 ++++++++++++++++++++++
> 2 files changed, 22 insertions(+), 23 deletions(-)
ACK
Thanks; pushed this and the buf.c cleanup.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library