From 53e85bfee776296820165b0fb26a27c75b16de5a Mon Sep 17 00:00:00 2001 From: Hilko Bengen Date: Fri, 28 Feb 2020 14:16:06 +0100 Subject: [PATCH] Check for mke2fs in sbin dirs, configure path into linuxdisk plugin Gbp-Pq: Name 0004-Check-for-mke2fs-in-sbin-dirs-configure-path-into-li.patch --- configure.ac | 7 +++++-- plugins/linuxdisk/filesystem.c | 2 +- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index 0e42ced..140345e 100644 --- a/configure.ac +++ b/configure.ac @@ -670,12 +670,15 @@ 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). +AC_PATH_PROG([MKE2FS], [mke2fs], [no], [$PATH:/usr/local/sbin:/usr/sbin:/sbin]) +AC_MSG_CHECKING([for $MKE2FS supporting the -d option]) mke2fs_with_d=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"], [ + AS_IF([$MKE2FS -V >/dev/null 2>&1], [ + AS_IF([LANG=C $MKE2FS -d 2>&1 | grep -sq "option requires an argument"], [ mke2fs_with_d=yes + AC_DEFINE_UNQUOTED([MKE2FS], ["$MKE2FS"], [path to mke2fs binary]) ]) ]) AC_MSG_RESULT([$mke2fs_with_d]) diff --git a/plugins/linuxdisk/filesystem.c b/plugins/linuxdisk/filesystem.c index a9f7064..e0b02b1 100644 --- a/plugins/linuxdisk/filesystem.c +++ b/plugins/linuxdisk/filesystem.c @@ -219,7 +219,7 @@ mke2fs (const char *filename) return -1; } - fprintf (fp, "mke2fs -q -F -t %s ", type); + fprintf (fp, MKE2FS " -q -F -t %s ", type); if (label) { fprintf (fp, "-L "); shell_quote (label, fp); -- 2.30.2