From: Sam James <sam@gentoo.org>
Date: Wed, 14 Sep 2022 14:11:58 +0000 (-0600)
Subject: configure.ac: fix bashism
X-Git-Tag: v3.1.0~309
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=4fc7dc1036676901d555fe296cca661a783e405b;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 <sam@gentoo.org>
Reviewed-by: Kamalesh Babulal <kamalesh.babulal@oracle.com>
Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
---

diff --git a/configure.ac b/configure.ac
index c0a7fe3c..671b4fec 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