From: Sam James Date: Wed, 14 Sep 2022 14:15:43 +0000 (-0600) Subject: configure.ac: fix bashism X-Git-Tag: v2.0.3~13 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=af221915a97d7fb590b7307e4cfa37dac6e96582;p=thirdparty%2Flibcgroup.git 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) --- diff --git a/configure.ac b/configure.ac index 8031707a..1d6470f8 100644 --- a/configure.ac +++ b/configure.ac @@ -140,7 +140,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