Re: [PATCH libvirt 2/6] include winsock2.h before windows.h

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

 




----- Mensaje original -----
> On 01/25/2012 01:13 PM, Marc-André Lureau wrote:
> > windows.h is included by threads.h.
> > winsock2.h should be included before.
> > 
> > Avoid the following warning:
> > 
> > In file included from ../gnulib/lib/unistd.h:51:0,
> >                  from ../src/util/util.h:30,
> >                  from rpc/virkeepalive.c:29:
> > /usr/x86_64-w64-mingw32/sys-root/mingw/include/winsock2.h:15:2:
> > warning: #warning Please include winsock2.h before windows.h
> > [-Wcpp]
> > ---
> >  src/rpc/virkeepalive.c |    4 ++++
> >  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> NACK.  I agree that things should be fixed, but the fix should live
> in
> the problematic "threads.h" inclusion pattern, not in the downstream
> virkeepalive.c.

I agree, but ff we follow that reasoning, windows.h should include winsock2.h, shouldn't it?

> Does this alternate patch work for you?  (One of these days, I need
> to
> install the mingw64 cross-compiler, so I can test it myself.)

Yes, that works too.

Btw, I don't mind testing patches, but using mingw64 is almost a 'yum install' away :) And you could help to fix the few remaining issues. The most worrying being the gnulib "stat" module, the rest should be in control.

> 
> diff --git i/src/util/threads-win32.h w/src/util/threads-win32.h
> index 8109afd..1b23402 100644
> --- i/src/util/threads-win32.h
> +++ w/src/util/threads-win32.h
> @@ -1,7 +1,7 @@
>  /*
>   * threads-win32.h basic thread synchronization primitives
>   *
> - * Copyright (C) 2009, 2011 Red Hat, Inc.
> + * Copyright (C) 2009, 2011-2012 Red Hat, Inc.
>   *
>   * This library is free software; you can redistribute it and/or
>   * modify it under the terms of the GNU Lesser General Public
> @@ -21,6 +21,9 @@
> 
>  #include "internal.h"
> 
> +#ifdef HAVE_WINSOCK2_H
> +# include <winsock2.h>
> +#endif
>  #include <windows.h>
> 
>  struct virMutex {
> 
> --
> Eric Blake   eblake@xxxxxxxxxx    +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
> 
> 

--
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]