Hello, Am Montag, 23. Oktober 2017, 12:38:33 CEST schrieb Goldwyn Rodrigues: > From: Goldwyn Rodrigues <rgold...@suse.com> > > This is a preparation patch to use for JSON mode of conveying > diff filename. In this patch we move diff generation functions to UI. > In the process, I have cleaned up the code to reduce code and enable > reuse. > > Remove unused function get_profile_diff(). > > Signed-off-by: Goldwyn Rodrigues <rgold...@suse.com>
Acked-by: Christian Boltz <appar...@cboltz.de>
on the condition that someone acks this small patch to avoid breaking
aa-mergeprof.
Hint: Whenever you delete a function, grep -r for it ;-)
=== modified file 'utils/aa-mergeprof'
--- utils/aa-mergeprof 2017-03-02 21:21:53 +0000
+++ utils/aa-mergeprof 2017-10-25 22:01:19 +0000
@@ -128,7 +128,7 @@
apparmor.aa.original_aa[program] =
apparmor.aa.deepcopy(apparmor.aa.aa[program])
#oldprofile =
apparmor.serialize_profile(apparmor.original_aa[program], program, '')
newprofile =
apparmor.aa.serialize_profile(mergeprofiles.user.aa[program], program, '')
-
apparmor.aa.display_changes_with_comments(mergeprofiles.user.filename,
newprofile)
+ aaui.UI_Changes(mergeprofiles.user.filename, newprofile,
comments=True)
elif ans == 'CMD_IGNORE_ENTRY':
break
(I'll commit this together with the main patch.)
Regards,
Christian Boltz
--
> ...womit die Geschichte wieder von vorn losgeht...
Jaha, aber nun ist das eine vollkommen andere Situation: Jetzt hast Du
nämlich eine von Suse generierte kdmrc für ein von Suse geliefertes KDE,
und KDE ist eine vom Installationssupport abgedeckte Komponente.
Also ist das ein meldefähiger und supportberechtigter Bug! :-)
[> Gerald Martin und Kristian Köhntopp in suse-linux]
signature.asc
Description: This is a digitally signed message part.
-- AppArmor mailing list AppArmor@lists.ubuntu.com Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
