Re: [Libguestfs] [libnbd PATCH v7 0/9] Rust Bindings for Libnbd
On Thu, Aug 17, 2023 at 09:29:47AM -0500, Eric Blake wrote:
On Thu, Aug 17, 2023 at 04:13:36AM +0000, Tage Johansson wrote:
> It is possible to add a minimum version requirement in Cargo.toml, I guess I
> should do that to make the errors a bit more clear.
It's okay if older platforms can't build Rust bindings, but anything
we can do to make it so that configure detects that gracefully up
front (and the rest of the build still succeeds), rather than
falling apart later during make when the missing feature turns into
a compiler error, is a good thing. And whatever we do, it doesn't
hurt if we amend README to call out the minimum Rust version we are
willing to support.
^ This is the important bit. ./configure should disable the Rust
bindings if they can't be built (for any reason, but including Rust
being too old). This avoids people hitting problems later on. We
don't control the environment where people try to compile libnbd.
Rich.
--
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