On Tue, Oct 06, 2020 at 03:29:33PM +0200, Martin Kletzander wrote: > On Thu, Sep 17, 2020 at 01:39:56PM +0100, Richard W.M. Jones wrote: > >As discussed in the emails today, this is the third version addressing > >most points from the v1/v2 review. > > > >You will need to pair this with the changes in libguestfs-common from > >this series: > > > >https://www.redhat.com/archives/libguestfs/2020-September/msg00050.html > > > > I'm going to look at this, but I do not have a machine to try this on, so I > will > not be able to do any functional testing (i.e. the review will be very much > incomplete). But from my limited testing I can tell you it breaks the build > for > me with the following errors:
I have --disable-golang ... Enabling it shows the exact same error. It's actually (another) bug caused by moving <guestfs.h> from lib/ to include/ so that the golang bindings are using the installed copy of guestfs.h (which lacks this newly added binding) rather than the local copy. I'll push a fix in a bit. Rich. > Making all in golang > make[2]: Entering directory '/home/nert/dev/libguestfs/golang' > ../run go install libguestfs.org/guestfs > # libguestfs.org/guestfs > src/libguestfs.org/guestfs/guestfs.go:3964:34: could not determine kind of > name for C.GUESTFS_CRYPTSETUP_OPEN_CRYPTTYPE_BITMASK > src/libguestfs.org/guestfs/guestfs.go:3960:34: could not determine kind of > name for C.GUESTFS_CRYPTSETUP_OPEN_READONLY_BITMASK > src/libguestfs.org/guestfs/guestfs.go:3925:10: could not determine kind of > name for C.guestfs_cryptsetup_close > src/libguestfs.org/guestfs/guestfs.go:3970:10: could not determine kind of > name for C.guestfs_cryptsetup_open_argv > make[2]: *** [Makefile:2509: pkg/linux_amd64/libguestfs.org/guestfs.a] Error 2 > make[2]: Leaving directory '/home/nert/dev/libguestfs/golang' > > so just so you know before I get to looking at the code. > > Have a nice day, > Martin > > >Rich. > > > > > >_______________________________________________ > >Libguestfs mailing list > >Libguestfs@redhat.com > >https://www.redhat.com/mailman/listinfo/libguestfs -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com Fedora Windows cross-compiler. Compile Windows programs, test, and build Windows installers. Over 100 libraries supported. http://fedoraproject.org/wiki/MinGW _______________________________________________ Libguestfs mailing list Libguestfs@redhat.com https://www.redhat.com/mailman/listinfo/libguestfs