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

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



On Tue, Aug 05, 2008 at 04:03:41AM -0400, Daniel Veillard wrote:
> On Tue, Aug 05, 2008 at 04:24:08PM +0900, Atsushi SAKAI wrote:
> > Hi,
> > 
> > Currently, compilation for MinGW is broken.
> > Here is the patch.
> > But configure script option for MinGW is changed from previous posting.
> > https://www.redhat.com/archives/libvir-list/2008-June/msg00145.html
> > 
> > Currently, it should do
> > # ./configure --without-xen --without-qemu --without-sasl --without-lxc --without-openvz --without-libvirtd --without-test
> 
>   yes I made compilation with lxc and openvz default now, so changing this
> is normal.
> 
> > currently following 3 make works,
> > make
> > make install
> > make check
> > I do not know why syntax-check is not exist on Makefile on MinGW.
> 
>   I would expect most development to still be done on Linux/Unix, so
> that's not a big deal IMHO
> 
> >  src/domain_conf.c    |    2 ++
> >  src/network_conf.c   |    2 ++
> 
>   basically the changes remove compilations from the XML parsing code.
> On one end I understand why it's not needed (currently) but I wonder why
> this need to be defined out. Seems to me the code should compile on WIN32
> and that's an important point if we ever want to get some native hypervisor
> support there.
>   So what miscompiled there ? Can we fix it ? in the interim having the code
> compiled out as the patch does is fine, but I would like to understand.

Yes, this is completely wrong. The test driver and all the generic network,
and domain  XML code should all work on Windows. The only stuff we should
be disabling is hypervisor drivers, and Linux specific bits of code.

Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|


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