Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 260880

Summary: app-emulation/xen-tools: uses -Werror during build
Product: Gentoo Linux Reporter: Diego Elio Pettenò (RETIRED) <flameeyes>
Component: New packagesAssignee: Gentoo Xen Devs <xen>
Status: RESOLVED WORKSFORME    
Severity: normal CC: binki, esigra
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
URL: http://blog.flameeyes.eu/2009/02/25/future-proof-your-code-dont-use-werror
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 260867    
Attachments: removes all werrors

Description Diego Elio Pettenò (RETIRED) gentoo-dev 2009-03-02 14:04:51 UTC
Hello,

you're receiving this canned bug because the package in summary uses the flag -Werror during build, which transforms warnings in errors, and is prone to break software when new GCC releases are added to Gentoo.

Please make sure your package does not use -Werror during build, so that it can be more future-proof for the new GCC releases.

Thanks,
Diego
Comment 1 Nathan Phillip Brink (binki) (RETIRED) gentoo-dev 2009-08-19 17:14:36 UTC
Created attachment 201727 [details, diff]
removes all werrors

I believe that this patch should be applied against the xen package and the xen-tools package. It applies cleanly against xen-tools-3.4.1, I haven't had time to check it against anything else.

I think that tools/ioemu-qemu-xen/configure will not enable -Werror by default. In fact, that configure file is an example of how xen should handle -Werror.

The reason to try to remove -Werror from packages is to prevent potentially avoidable Gentoo bugs from being filed. The following is an example of such a bug that would be filed: http://bugzilla.xensource.com/bugzilla/show_bug.cgi?id=1427 . While this bug is probably something that should be fixed in the xen source, most of these warnings can be safely ignored.
Comment 2 Alexey Shvetsov archtester gentoo-dev 2011-03-26 11:40:40 UTC
Xen 4.1 in tree. Please test with it and reopen if it doesnt work