From 0a535d4988183bb708f7df86c03476a288e16da0 Mon Sep 17 00:00:00 2001 From: Sam James Date: Wed, 14 Sep 2022 08:16:36 -0600 Subject: [PATCH] configure.ac: fix bashism configure scripts need to be runnable with a POSIX-compliant /bin/sh. On many (but not all!) systems, /bin/sh is provided by Bash, so errors like this aren't spotted. Notably Debian defaults to /bin/sh provided by dash which doesn't tolerate such bashisms as '=='. This retains compatibility with bash. Fixes configure warnings/errors like: ``` checking whether to build static libraries... no ./configure: 14089: test: xno: unexpected operator checking for x86_64-pc-linux-gnu-g++... x86_64-pc-linux-gnu-g++ ``` Signed-off-by: Sam James Reviewed-by: Kamalesh Babulal Signed-off-by: Tom Hromatka (cherry picked from commit 4fc7dc1036676901d555fe296cca661a783e405b) --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 36a9d15b..58eb40b5 100644 --- a/configure.ac +++ b/configure.ac @@ -137,7 +137,7 @@ AC_ARG_ENABLE([opaque-hierarchy], AC_ARG_ENABLE([tests], [AS_HELP_STRING([--enable-tests],[compile libcgroup tests [default=yes]])], [ - if test "x$enableval" == xno; then + if test "x$enableval" = xno; then with_tests=false else with_tests=true -- 2.47.2