Hello,

Am Sonntag, 3. Juni 2018, 15:58:47 CEST schrieb Christian Boltz:
> My prefered option would be to change (V)iew Changes so that it writes
> the new profile in clean mode instead of least-possible-changes mode.

If you want to see in practise what this means:

--- a/utils/apparmor/aa.py
+++ b/utils/apparmor/aa.py
@@ -1894,7 +1894,8 @@ def save_profiles():
                     oldprofile = get_profile_filename(which)
 
                 try:
-                    newprofile = serialize_profile_from_old_profile(aa[which], 
which, '')
+                    #newprofile = 
serialize_profile_from_old_profile(aa[which], which, '')
+                    newprofile = serialize_profile(aa[which], which, '')
                 except AttributeError:
                     # see 
https://bugs.launchpad.net/ubuntu/+source/apparmor/+bug/1528139
                     newprofile = "###\n###\n### Internal error while 
generating diff, please use '%s' instead\n###\n###\n" % _('View Changes b/w 
(C)lean profiles')

This is a minimal patch (without cleanup etc.), but it's more than good
enough to see what it does ;-)


Regards,

Christian Boltz
-- 
..........why use Windows, if there is a door.............

Attachment: signature.asc
Description: This is a digitally signed message part.

-- 
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to