[linux-lvm] [PATCH 1/1] build: force to use bash (was: Re: bashism in tools/Makefile.in)
Christian Hesse
list at eworm.de
Mon Jan 6 12:54:37 UTC 2020
From: Christian Hesse <mail at eworm.de>
> Unfortunately whole lvm2 project is mostly 'bash' based and all developers
> mostly use bash - so we would welcome patches if they would provide
> equivalently working code without bashism...
>
> Fixing couple things while the whole rest of project is still requiring bash
> doesn't make it any better - ATM all scripts/tests/Makefiles mostly expect
> presence of bash.
Fine with me. So how about forcing the build system to use bash?
-- >8 --
From: Christian Hesse <mail at eworm.de>
Subject: build: force to use bash
The build system expects bash, so force to use it. This mitigates issues
where other shells are fed with bashism.
Signed-off-by: Christian Hesse <mail at eworm.de>
---
configure | 15 +++++++++++++++
configure.ac | 15 +++++++++++++++
2 files changed, 30 insertions(+)
diff --git a/configure b/configure
index 6dd7edac3..190d13c77 100755
--- a/configure
+++ b/configure
@@ -2960,6 +2960,21 @@ ac_configure="$SHELL $ac_aux_dir/configure" # Please don't use this var.
+################################################################################
+if test -z "$BASH_VERSION$CONFIG_SHELL"
+then
+ CONFIG_SHELL=bash
+ export CONFIG_SHELL
+ if $CONFIG_SHELL -c "exit 0"
+ then
+ exec $CONFIG_SHELL $0 "$@"
+ else
+ { $as_echo "$as_me:${as_lineno-$LINENO}: The 'bash' shell is needed to build lvm2" >&5
+$as_echo "$as_me: The 'bash' shell is needed to build lvm2" >&6;}
+ as_fn_error $? "'bash' not found" "$LINENO" 5
+ fi
+fi
+
################################################################################
# Make sure we can run config.sub.
$SHELL "$ac_aux_dir/config.sub" sun4 >/dev/null 2>&1 ||
diff --git a/configure.ac b/configure.ac
index 74ca20191..f8d0abdfb 100644
--- a/configure.ac
+++ b/configure.ac
@@ -23,6 +23,21 @@ AC_CONFIG_HEADERS([include/configure.h])
dnl -- Setup the directory where autoconf has auxilary files
AC_CONFIG_AUX_DIR(autoconf)
+################################################################################
+dnl -- We need to run the build system with bash.
+if test -z "$BASH_VERSION$CONFIG_SHELL"
+then
+ CONFIG_SHELL=bash
+ export CONFIG_SHELL
+ if $CONFIG_SHELL -c "exit 0"
+ then
+ exec $CONFIG_SHELL $0 "$@"
+ else
+ AC_MSG_NOTICE([The 'bash' shell is needed to build lvm2])
+ AC_MSG_ERROR(['bash' not found])
+ fi
+fi
+
################################################################################
dnl -- Get system type
AC_CANONICAL_TARGET([])
More information about the linux-lvm
mailing list