Re: [Xen-devel] 3.1.x and 3.2.x releases

["Ryan Scott" <rjscott@xxxxxxxxx>]

On Dec 28, 2007 9:55 AM, John Levon <levon@xxxxxxxxxxxxxxxxx> wrote:

On Fri, Dec 28, 2007 at 01:16:50PM +0000, Keir Fraser wrote:

> Oh, it's because your stdint.h type definitions are macros rather than
> typedefs. I believe the C spec requires them to be typedef names (Section
> 7.18 of the C99 draft spec). It looks like the problem stems from the
> stdint.h supplied with gnulib, included in libvirt-0.4.0. Why does libvirt
> require its own stdint.h?

Note this same bogus header breaks Solaris full stop (I think we have a
patch for it, but I'm not sure)

I did hit this when compiling 0.4.0 on Solaris.  I hacked around this with the following patch, but figured I'd spend more time figuring out a better solution before sending something upstream.  Still, it may be useful to some.

-Ryan

diff --git a/configure b/configure
--- a/configure
+++ b/configure
@@ -36255,7 +36255,7 @@ fi
 CFLAGS="$old_cflags"
 LDFLAGS="$old_ldflags"

-GNUTLS_CFLAGS=
+GNUTLS_CFLAGS=-D_GL_JUST_INCLUDE_SYSTEM_STDINT_H
 GNUTLS_LIBS=
 if test "x$PKG_CONFIG" != "x" ; then

diff --git a/proxy/Makefile.in b/proxy/Makefile.in
--- a/proxy/Makefile.in
+++ b/proxy/Makefile.in
@@ -154,7 +154,7 @@ BITSIZEOF_WINT_T = @BITSIZEOF_WINT_T@
 BRCTL = @BRCTL@
 CC = @CC@
 CCDEPMODE = @CCDEPMODE@
-CFLAGS = @CFLAGS@
+CFLAGS = @CFLAGS@ -D_GL_JUST_INCLUDE_SYSTEM_STDINT_H
 COMPILER_FLAGS = @COMPILER_FLAGS@
 COVERAGE_CFLAGS = @COVERAGE_CFLAGS@
 COVERAGE_LDFLAGS = @COVERAGE_LDFLAGS@

 

regards
john

_______________________________________________
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