Hello, Am Mittwoch, 26. November 2014 schrieb Steve Beattie: > While trying to help Christian diagnose some problems I was having > with his capabilities classes patches, I wrote the following patches. > > The first was discovered while writing the second: the assignment for > setting segments['include'] = True is wrong, it occurs inside the > 'if not segments['include'] and True in segments.values():' block, > whereas it needs to always get set outside of that if test.
Acked-by: Christian Boltz <[email protected]> > The second pulls out all the common processing for writing out each > of the prior segments that need to be written before writing the > current segment into a function called 'write_prior_segments', > reducing a bunch of ugly duplication. Nice cleanup! Acked-by: Christian Boltz <[email protected]> even if I can foresee a merge conflict with the [3/3] part of my capability rule class patchset ;-) Please commit those patches soon so that I can send out a new version of the capability rule class patchset. Regards, Christian Boltz -- I will hand over the release job to Mr. Bash, he is more reliable :/ [Adrian Schroeter in opensuse] -- AppArmor mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
