Re: [Libguestfs] [PATCH 0/5] Patch to build hivex lib on Windows

On Tue, Sep 06, 2011 at 11:31:11PM +0200, Gillen Daniel wrote: I've applied an equivalent patch: http://git.annexia.org/?p=hivex.git;a=commitdiff;h=085707078097f6def993b2... I tested it using the Fedora Windows cross-compiler. Please check it still works in your environment. Some comments: Lots of small patches are easier to review. However each patch still needs to be an atomic, consistent commit, which wasn't the case here. 'git format-patch' or 'git send-email' makes it simpler both for you to send, and for me to apply, patches. We had some discussion on IRC about how to reduce the impact on the main code further. There were two proposals: either to implement a hashtable in the Windows mmap code to map <something> to hive handles. Or to do what we did (which is a bit of a hack): #define mmap(...) mmap (h, ...) This isn't the best solution, but it'll do til we think of something better. Files not listed in *_SOURCES so make dist would have failed. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones New in Fedora 11: Fedora Windows cross-compiler. Compile Windows programs, test, and build Windows installers. Over 70 libraries supprt'd http://fedoraproject.org/wiki/MinGW http://www.annexia.org/fedora_mingw