On 01/14/2014 10:54 AM, Bernhard Voelker wrote: > On 01/13/2014 09:09 PM, Pádraig Brady wrote: >> Pushing the attached 2 patches in a while. > > Hi Padraig, > > thanks, the refactoring into gcc_shared_ is a good idea. > > But I missed this one: > when selinux is not supported, the new no-ctx.sh test is skipped > with the wrong and misleading "LD_PRELOAD interception failed" diagnostic:
> + test -e preloaded > + skip_ 'LD_PRELOAD interception failed' Oh right. I think this should restrict the test appropriately... commit 3620df245a2211dc441e019845f98b91333bda77 Author: Pádraig Brady <[email protected]> Date: Tue Jan 14 11:30:51 2014 +0000 tests: restrict a recent SELinux test to SELinux systems * tests/cp/no-ctx.sh: Since the test diagnoses whether the intercepted lgetfilecon() calls are actually called or not, restrict the test to systems where that occurs. The test cases are minimal on non SELinux systems and should be well covered by other tests. Reported-by: Bernhard Voelker diff --git a/tests/cp/no-ctx.sh b/tests/cp/no-ctx.sh index 3b5eb82..6851785 100755 --- a/tests/cp/no-ctx.sh +++ b/tests/cp/no-ctx.sh @@ -22,6 +22,7 @@ . "${srcdir=.}/tests/init.sh"; path_prepend_ ./src print_ver_ cp require_gcc_shared_ +requires_selinux_ # Replace each getfilecon and lgetfilecon call with a call to these stubs. cat > k.c <<'EOF' || framework_failure_
