[Libguestfs] [libnbd PATCH] Fix building with for ocaml < 4.06.0

Martin Kletzander mkletzan at redhat.com
Wed Jun 12 21:25:38 UTC 2019


On Wed, Jun 12, 2019 at 08:02:06AM -0500, Eric Blake wrote:
>On 6/12/19 6:27 AM, Martin Kletzander wrote:
>> Signed-off-by: Martin Kletzander <mkletzan at redhat.com>
>> ---
>>
>> Notes:
>>     This is the simplest solution suggested to me by Rich and it works.
>>
>>     Most of autotools are not friends with me, so the other approach I tried, with
>>     requiring at least OCaml 4.05.0, having this fix in only for OCaml < 4.06.0 and
>>     properly detecting and reporting that, was a bit ugly.
>
>I prefer the lightweight approach - all the more configure.ac does is
>add the ability to do #ifdef-ery probing on things that don't originally
>come with enough #ifdef witnesses; but here, the absence of Bytes_val is
>all the more that you would probe for in configure.ac, making it not
>worth the effort to add yet another macro just to work around the problem.
>
>Which platform are you compiling on, and what OCaml version are you
>using?  That said, I don't see any reason to not commit this patch.
>

Gentoo, OCaml 4.05.0 in the system, but 4.07.0 through opam.  I noticed I cannot
build it when I added it as a package and wanted to install it into the system.

>--
>Eric Blake, Principal Software Engineer
>Red Hat, Inc.           +1-919-301-3226
>Virtualization:  qemu.org | libvirt.org
>



-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libguestfs/attachments/20190612/80b712c9/attachment.sig>


More information about the Libguestfs mailing list