Richard W.M. Jones
2020-Jun-19 19:59 UTC
[Libguestfs] [PATCH nbdkit] build: Rename configure --without-linuxdisk -> --disable-linuxdisk
When I was reviewing this I forgot the slightly ridiculous rules that
autoconf uses for --enable/disable vs --with/without:
https://www.gnu.org/software/autoconf/manual/autoconf-2.66/html_node/Package-Options.html
https://www.gnu.org/software/autoconf/manual/autoconf-2.66/html_node/External-Software.html
Since this is disabling an intrinsic feature of the software (a
plugin) it should use --disable.
Updates: commit 2a3efc5836df3d55d304cc4b3ea2800b5e93831a
---
configure.ac | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/configure.ac b/configure.ac
index e6ca09de..376b1538 100644
--- a/configure.ac
+++ b/configure.ac
@@ -483,17 +483,17 @@ AS_IF([test "$GNUTLS_LIBS" != ""],[
LIBS="$old_LIBS"
])
-AC_ARG_WITH([linuxdisk],
- [AS_HELP_STRING([--without-linuxdisk],
+AC_ARG_ENABLE([linuxdisk],
+ [AS_HELP_STRING([--disable-linuxdisk],
[disable linuxdisk plugin @<:@default=check@:>@])],
[],
- [with_linuxdisk=check])
+ [enable_linuxdisk=check])
dnl Check for mke2fs -d (used by linuxdisk plugin). There are two
dnl possible outcomes that we care about: (1) We have mke2fs and
dnl it supports the -d option. (2) We either don't have mke2fs
dnl or it's too old to support the -d option (eg. on RHEL 7).
mke2fs_with_d=no
-AS_IF([test "$with_linuxdisk" != "no"], [
+AS_IF([test "enable_linuxdisk" != "no"], [
AC_MSG_CHECKING([for mke2fs supporting the -d option])
AS_IF([mke2fs -V >/dev/null 2>&1], [
AS_IF([LANG=C mke2fs -d 2>&1 | grep -sq "option requires an
argument"], [
--
2.25.0
Richard W.M. Jones
2020-Jun-19 20:06 UTC
Re: [Libguestfs] [PATCH nbdkit] build: Rename configure --without-linuxdisk -> --disable-linuxdisk
Sorry this patch is nonsense, ignore it. V2 coming. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com virt-df lists disk usage of guests without needing to install any software inside the virtual machine. Supports Linux and Windows. http://people.redhat.com/~rjones/virt-df/
Apparently Analagous Threads
- [PATCH 3/3] nbdkit: add a --without-linuxdisk configure argument
- [PATCH nbdkit v2 0/3] build: Rename configure --without-linuxdisk -> --disable-linuxdisk
- [PATCH 1/3] nbdkit: fix build of the SSH plugin on FreeBSD
- [PATCH nbdkit v3 4/4] Add linuxdisk plugin.
- Re: [PATCH nbdkit v3 4/4] Add linuxdisk plugin.