From 75de88852acacd665c5b71c19aff67ed7034826b Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Mon, 15 Sep 2008 14:37:49 +0200 Subject: [PATCH] avoid spurious test failure when library is built without ACL support * m4/acl.m4 (USE_ACL): Define as a shell variable, too, for... * modules/acl-tests (Makefile.am) [TESTS_ENVIRONMENT]: Add USE_ACL. * tests/test-file-has-acl.sh: Skip if USE_ACL == 0. * tests/test-copy-acl.sh: Likewise. --- ChangeLog | 8 ++++++++ m4/acl.m4 | 2 ++ modules/acl-tests | 2 +- tests/test-copy-acl.sh | 6 ++++++ tests/test-file-has-acl.sh | 6 ++++++ 5 files changed, 23 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 375d43f57..41b8c5d27 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +2008-09-16 Jim Meyering + + avoid spurious test failure when library is built without ACL support + * m4/acl.m4 (USE_ACL): Define as a shell variable, too, for... + * modules/acl-tests (Makefile.am) [TESTS_ENVIRONMENT]: Add USE_ACL. + * tests/test-file-has-acl.sh: Skip if USE_ACL == 0. + * tests/test-copy-acl.sh: Likewise. + 2008-09-15 Ralf Wildenhues * lib/fstrcmp.c (fstrcmp_bounded): Use a second, less quick upper bound diff --git a/m4/acl.m4 b/m4/acl.m4 index 494ce4beb..08342d03c 100644 --- a/m4/acl.m4 +++ b/m4/acl.m4 @@ -121,6 +121,8 @@ int type = ACL_TYPE_EXTENDED;]], AC_SUBST([LIB_ACL]) AC_DEFINE_UNQUOTED([USE_ACL], [$use_acl], [Define to nonzero if you want access control list support.]) + USE_ACL=$use_acl + AC_SUBST([USE_ACL]) ]) # gl_ACL_GET_FILE(IF-WORKS, IF-NOT) diff --git a/modules/acl-tests b/modules/acl-tests index 04e004cfd..0aacd4141 100644 --- a/modules/acl-tests +++ b/modules/acl-tests @@ -17,7 +17,7 @@ configure.ac: Makefile.am: TESTS += test-file-has-acl.sh test-set-mode-acl.sh test-copy-acl.sh -TESTS_ENVIRONMENT += EXEEXT='@EXEEXT@' +TESTS_ENVIRONMENT += EXEEXT='@EXEEXT@' USE_ACL=$(USE_ACL) check_PROGRAMS += test-file-has-acl test-set-mode-acl test-copy-acl test-sameacls test_file_has_acl_LDADD = $(LDADD) $(LIB_ACL) test_set_mode_acl_LDADD = $(LDADD) $(LIB_ACL) @LIBINTL@ diff --git a/tests/test-copy-acl.sh b/tests/test-copy-acl.sh index 6d5970268..9e20e77cd 100755 --- a/tests/test-copy-acl.sh +++ b/tests/test-copy-acl.sh @@ -3,6 +3,12 @@ # Show all commands when run with environment variable VERBOSE=yes. test -z "$VERBOSE" || set -x +test "$USE_ACL" = 0 && + { + echo "Skipping test: insufficient ACL support" + exit 77 + } + # func_tmpdir # creates a temporary directory. # Sets variable diff --git a/tests/test-file-has-acl.sh b/tests/test-file-has-acl.sh index e2dbe9d08..978396082 100755 --- a/tests/test-file-has-acl.sh +++ b/tests/test-file-has-acl.sh @@ -3,6 +3,12 @@ # Show all commands when run with environment variable VERBOSE=yes. test -z "$VERBOSE" || set -x +test "$USE_ACL" = 0 && + { + echo "Skipping test: insufficient ACL support" + exit 77 + } + # func_tmpdir # creates a temporary directory. # Sets variable -- 2.11.0