On Thu, Oct 31, 2013 at 11:43:25AM -0700, Steve Beattie wrote: > The parser sanity test make target does not directly depend on the make > targets that generate the tests consumed by the sanity test, leading to > runs that did not verify all the test cases when make check is invoked > with parallelism (e.g. make check -j4). This patch against trunk fixes > the issue (it's a subset of the '[patch 6/8] parser - use new caching > test script' patch from my last patch submission).
And here's the same fix for the 2.8 branch (which doesn't have the dbus patches applied, so is slightly different) Signed-off-by: Steve Beattie <[email protected]> --- parser/tst/Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Index: b/parser/tst/Makefile =================================================================== --- a/parser/tst/Makefile +++ b/parser/tst/Makefile @@ -13,7 +13,7 @@ endif all: tests .PHONY: tests error_output gen_xtrans parser_sanity caching minimize -tests: error_output gen_xtrans parser_sanity caching minimize +tests: error_output parser_sanity caching minimize GEN_TRANS_DIRS=simple_tests/generated_x/ simple_tests/generated_perms_leading/ simple_tests/generated_perms_safe/ @@ -35,7 +35,7 @@ error_output: $(PARSER) grep -q "AppArmor parser error for errors/multi_include.sd in errors/multi_include.sd at line 12: Could not open 'failure'" @echo "Error Output: PASS" -parser_sanity: $(PARSER) +parser_sanity: $(PARSER) gen_xtrans $(Q)${PROVE} ${PROVE_ARG} ${TESTS} caching: $(PARSER) -- Steve Beattie <[email protected]> http://NxNW.org/~steve/
signature.asc
Description: Digital signature
-- AppArmor mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
