On Thu, Feb 13, 2014 at 04:54:09PM +0200, Or Goshen wrote:
Hi,
I attached the changes I made to a vanilla libguestfs-1.22.6 in order to
make it work in mingw/win32.
Added is also the patch required to make QEMU compatible (add a command to
QMP that lists the supported devices (the regilat way you do it print it to
stderr, which is difficult to redirect in win32)).
A couple of points first:
- You need to start using git.
- You need to develop against the git head, not the old 1.22 branch.
I took the tarball and turned it into a git commit, which I then
cherry picked on top of head. (Except for 'bootstrap' which seems to
be a completely rewritten file -- so I created it as
'bootstrap.win32').
This is attached so you can take it from here. As this was mostly an
automatic process -- git did the hard work, I just resolved a few
things that git could not resolve -- you need to check carefully that
this patch still works.
About the patch itself:
The patch looks fairly sensible. There's some tidying up that could
be done, and it would be nice to confine more of the changes so they
don't affect the main code. But there's nothing here that can't be
pushed upstream in some form, after a proper review and more testing.
The qemu change needs to separately be submitted to the qemu mailing
list, along with an explanation for why that is necessary.
Thanks,
Rich.
--
Richard Jones, Virtualization Group, Red Hat
http://people.redhat.com/~rjones
Read my programming blog:
http://rwmj.wordpress.com
Fedora now supports 80 OCaml packages (the OPEN alternative to F#)