Re: In freeze for libvirt-0.9.8 and availability of rc2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/05/2011 09:41 AM, Daniel P. Berrange wrote:
> On Mon, Dec 05, 2011 at 09:25:40AM -0700, Eric Blake wrote:
>> On 12/04/2011 02:26 AM, Peter Krempa wrote:
>>> I tried building rc2 on my macbook and build was successful, but I've
>>> got a warning on each file built:
>>>
>>> cc1: warning: command line option "-Wabi" is valid for C++/ObjC++ but
>>> not for C
>>> cc1: warning: command line option "-Wdeprecated" is valid for C++/ObjC++
>>> but not for C
>>
>> This is a known (and harmless) issue with the gnulib 'manywarnings'
>> module, and only triggers with older versions of gcc.  It will probably
>> be resolved in gnulib in time for libvirt 0.9.9, but I don't see us
>> doing anything about it for now (if you don't want the warning, then
>> don't use --enable-compile-warnings with ./autogen.sh or ./configure).
> 
> Can we workaround it by adding
> 
> 
>    dontwarn="$dontwarn -Wabi"
>    dontwarn="$dontwarn -Wdeprecated"
> 
> to our m4 config, until such time as we rebase gnulib ?

Sure.  Done as follows, under the trivial rule:

From f17e0e2182ecefd9ba9eb346bab839cd5b59fb9c Mon Sep 17 00:00:00 2001
From: Eric Blake <eblake@xxxxxxxxxx>
Date: Mon, 5 Dec 2011 10:14:55 -0700
Subject: [PATCH] build: reduce warnings from older gcc

Older gcc warns (on every file!) that -Wabi and -Wdeprecated only
make sense on C++ projects.  Newer gcc accepts these warnings for
C, but it is not clear that they can do anything useful, so it
is easier to just drop the warnings altogether.

* m4/virt-compile-warnings.m4 (LIBVIRT_COMPILE_WARNINGS): Silence
-Wabi and -Wdeprecated on older gcc.
Reported by Peter Krempa.
---
 m4/virt-compile-warnings.m4 |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/m4/virt-compile-warnings.m4 b/m4/virt-compile-warnings.m4
index a17e4b7..ba388aa 100644
--- a/m4/virt-compile-warnings.m4
+++ b/m4/virt-compile-warnings.m4
@@ -22,6 +22,8 @@ AC_DEFUN([LIBVIRT_COMPILE_WARNINGS],[

         # Don't care about C++ compiler compat
         dontwarn="$dontwarn -Wc++-compat"
+        dontwarn="$dontwarn -Wabi"
+        dontwarn="$dontwarn -Wdeprecated"
         # Don't care about ancient C standard compat
         dontwarn="$dontwarn -Wtraditional"
         # Don't care about ancient C standard compat
-- 
1.7.7.3

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]