[Libosinfo] [osinfo-db-tools PATCH v2 1/2] configure: Fix EXECUTE_TESTS condition

Fabiano Fidêncio fidencio at redhat.com
Tue Apr 23 11:12:15 UTC 2019


When b380daf430 fixed the non-portable shell test syntax, a new `test`
should have been added so both conditions are properly evaluated.

Example:
[test "x$foo" = "xyes" && test "x$bar = "xyes"] instead of
[test "x$foo" = "yes" -a "x$bar" = "xyes"].

Signed-off-by: Fabiano Fidêncio <fidencio at redhat.com>
---
 configure.ac | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configure.ac b/configure.ac
index f3139f2..25a1a27 100644
--- a/configure.ac
+++ b/configure.ac
@@ -32,7 +32,7 @@ else
     HAVE_PYTEST="no"
     HAVE_REQUESTS="no"
 fi
-AM_CONDITIONAL([EXECUTE_TESTS], [test "x$HAVE_PYTEST" == "xyes" && "x$HAVE_REQUESTS" = "xyes"])
+AM_CONDITIONAL([EXECUTE_TESTS], [test "x$HAVE_PYTEST" == "xyes" && test "x$HAVE_REQUESTS" = "xyes"])
 
 LT_INIT([shared disable-static win32-dll])
 
-- 
2.21.0




More information about the Libosinfo mailing list