[libvirt] [PATCH] fix MinGW compilation(200808)

Atsushi SAKAI sakaia at jp.fujitsu.com
Mon Aug 11 10:57:46 UTC 2008


Hi, Jim and Dan

Thank you for commenting.

I compiled with attached patch successfully.

 src/domain_conf.c    |    1 +
 src/domain_conf.h    |    2 ++
 src/network_conf.c   |    1 +
 src/util.c           |    4 ++--
 src/virsh.c          |    3 +++
 tests/testutilsxen.c |    2 ++
 6 files changed, 11 insertions(+), 2 deletions(-)


Thanks
Atsushi SAKAI




Jim Meyering <jim at meyering.net> wrote:

> Atsushi SAKAI <sakaia at jp.fujitsu.com> wrote:
> > I am looking around the code about WEXITSTATUS.
> > It seems that following kind of code needs on gnulib.
> > http://kerneltrap.org/mailarchive/git/2008/2/27/1003054
> > What do you think?
> 
> Hi Atsushi,
> 
> Yes, git provides MinGW support, and I see it's compat/mingw.h
> includes this line:
> 
> #define WEXITSTATUS(x) ((x) & 0xff)
> 
> Could it be as easy as adding this?
> 
>   #ifndef WEXITSTATUS
>   # define WEXITSTATUS(x) ((x) & 0xff)
>   #endif
> 
> gnulib provides heavy-weight (different API, and GPL'd) support for
> running subprocesses via its execute module, but it targets more
> than just MinGW.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix_mingw_compile_20080811.patch
Type: application/octet-stream
Size: 3215 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20080811/18ba03a9/attachment-0001.obj>


More information about the libvir-list mailing list