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

RE: [Xen-devel] ioemu build failure with GCC > 4.3



I think the patch also needs "--disable-gcc-check"

Thanks
Ross

-----Original Message-----
From: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
[mailto:xen-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of Ian Jackson
Sent: Thursday, July 10, 2008 5:50 AM
To: Daniel P. Berrange; Keir Fraser; He, Qing;
xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: Re: [Xen-devel] ioemu build failure with GCC > 4.3

iwj writes ("Re: [Xen-devel] ioemu build failure with GCC > 4.3"):
> Daniel P. Berrange writes ("Re: [Xen-devel] ioemu build failure with
GCC > 4.3"):
> > None of the entire slirp directory looks like a good idea. This code
> > isn't even used in Xen is it ? My vote  would be disabling
compilation
> > of entire slirp directory for Xen. 
> 
> This is the right answer, yes.

I've done this now.

Ian.

commit 29c20f126f298cf2162169b4bbb8206860f11240
Author: Ian Jackson <iwj@xxxxxxxxxxxxxxxxxxxxxxxx>
Date:   Thu Jul 10 10:48:51 2008 +0100

    Disable slirp.
    
    The slirp system is not really a very nice idea, especially not in a
    Xen context.  Also, the code misuses inline and does not compile on
    GCC >4.3ish.
    
    Thanks to Qing He <qing.he@xxxxxxxxx> for the report.

diff --git a/xen-setup b/xen-setup
index c00122c..755b9ec 100755
--- a/xen-setup
+++ b/xen-setup
@@ -9,7 +9,7 @@ rm -f config-host.mak
 
 if test -f config-host.h; then mv config-host.h config-host.h~; fi
 
-./configure --disable-curses "$@" --prefix=/usr
+./configure --disable-curses --disable-slirp "$@" --prefix=/usr
 
 target=i386-dm
 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.