On 10/25/2013 03:45 PM, Daniel J Walsh wrote: > > This patch looks good to me. acked. This reverts the fix I just applied, and causes make test to fail in sepolgen for me. Do you see a different result? Also, cc me. Current selinux master: cd selinux git checkout master git pull cd sepolgen make test $ make test make -C tests test make[1]: Entering directory `/home/sds/selinux/sepolgen/tests' python run-tests.py ........................................... ---------------------------------------------------------------------- Ran 43 tests in 0.811s OK make[1]: Leaving directory `/home/sds/selinux/sepolgen/tests' With this patch: make -C tests test make[1]: Entering directory `/home/sds/selinux/sepolgen/tests' python run-tests.py ............................F.............. ====================================================================== FAIL: test_append (test_matching.TestMatchList) ---------------------------------------------------------------------- Traceback (most recent call last): File "/home/sds/selinux/sepolgen/tests/test_matching.py", line 54, in test_append self.assertEqual(len(ml), 2) AssertionError: 1 != 2 ---------------------------------------------------------------------- Ran 43 tests in 0.796s FAILED (failures=1) make[1]: *** [test] Error 1 make[1]: Leaving directory `/home/sds/selinux/sepolgen/tests' make: *** [test] Error 2 -- This message was distributed to subscribers of the selinux mailing list. If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with the words "unsubscribe selinux" without quotes as the message.