On Sun, Mar 15, 2015 at 10:20:16PM +0100, Christian Boltz wrote: > Changes in set_profile_flags(): > - rewrite set_profile_flags to use parse_profile_start_line() and > write_header(). > - replace the silent failure for non-existing files with a proper > exception (using lazy programming - the check is done by removing the > "if os.path.isfile()" check, open_file_read then raises the > exception ;-) > - comment out regex_hat_flag and the code that was supposed to handle > hat flags, which were totally broken. We'll need another patch to fix > it, and we also need to decide if we want to do that because it > introduces a behaviour change (currently, aa-complain etc. don't > change hat flags). > > The tests for set_profile_flags() are also updated: > - prepend a space to comments because write_header always adds a space > between '{' and the comment > - remove a test with superfluous quotes that are no longer kept (that's > just a profile cleanup, so dropping that test is the easiest way) > - update test_set_flags_10 and test_set_flags_12 to use the correct > profile name > - enable the tests for invalid (empty) flags > - update the test for a non-existing file > > Note: test_set_flags_10, test_set_flags_12 and test_set_flags_nochange_09 > will fail with this patch applied. The next patch will fix that. > [ 21-rewrite-set_profile_flags.diff ]
Alright, so with the python 2/3 division difference squared away, Acked-by: Steve Beattie <st...@nxnw.org>. Thanks. -- Steve Beattie <sbeat...@ubuntu.com> http://NxNW.org/~steve/
signature.asc
Description: Digital signature
-- AppArmor mailing list AppArmor@lists.ubuntu.com Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor