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.