<div dir="ltr"><div dir="ltr">The rebased version is at 

<a href="https://github.com/gyf304/nbdkit/commits/msys2-port">https://github.com/gyf304/nbdkit/commits/msys2-port</a><div>It is currently buildable with:</div><div>autoreconf -i && ./configure host=x86_64-pc-msys && make -j<br></div><div><br></div><div>A few changes are added:</div><div>* nbdkit_peer_name is added back but will always error.</div><div>* __declspec(dllexport) is added to the headers for PE compatibility.</div><div>* -D_spawnv=spawnv is now removed.</div><div>  I think I didn't run ./configure properly last time.</div><div><br></div><div>Here is the list of features that are tested to compile.</div><div><br></div><div>Optional server features:<br><br>    bash-completion ........................ yes<br>    manual pages ........................... yes<br>    SELinux ................................ no<br>    TLS .................................... no<br><br>Optional plugins:<br><br>    curl ................................... no<br>    example4 ............................... yes<br>    ext2 ................................... no<br>    floppy ................................. no<br>    guestfs ................................ no<br>    gzip ................................... no<br>    iso .................................... no<br>    libvirt ................................ no<br>    nbd .................................... no<br>    ssh .................................... no<br>    tar .................................... yes<br>    vddk ................................... yes<br><br>Languages:<br><br>    lua .................................... no<br>    ocaml .................................. no<br>    perl ................................... yes<br>    python ................................. yes<br>    ruby ................................... no<br>    rust ................................... no<br>    tcl .................................... no<br><br>Optional filters:<br><br>    ext2 ................................... no<br>    xz ..................................... no<br></div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Mar 22, 2020 at 10:28 AM Richard W.M. Jones <<a href="mailto:rjones@redhat.com">rjones@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sat, Mar 21, 2020 at 09:36:15PM -0400, Frank Gu wrote:<br>
> Answering a few questions:<br>
> <br>
> * Unclear why it's necessary to rename nbdkit-common.h -> nbdkit-compat.h<br>
> - It's not a rename, but a new file. nbdkit-compat.h is included for PE DLL<br>
>   compatibility, and is included in nbdkit-common.h<br>
<br>
Oh I see.  The github UI is super-confusing and made it look as if the<br>
file had been renamed.<br>
<br>
Is it possible at some point that you could rebase your work on top of<br>
the current head, and split it into individually reviewable patches?<br>
<br>
Rich.<br>
<br>
-- <br>
Richard Jones, Virtualization Group, Red Hat <a href="http://people.redhat.com/~rjones" rel="noreferrer" target="_blank">http://people.redhat.com/~rjones</a><br>
Read my programming and virtualization blog: <a href="http://rwmj.wordpress.com" rel="noreferrer" target="_blank">http://rwmj.wordpress.com</a><br>
Fedora Windows cross-compiler. Compile Windows programs, test, and<br>
build Windows installers. Over 100 libraries supported.<br>
<a href="http://fedoraproject.org/wiki/MinGW" rel="noreferrer" target="_blank">http://fedoraproject.org/wiki/MinGW</a><br>
<br>
</blockquote></div></div>