Hello,

if a *Ruleset is empty, let __repr__() print/return

    <FooRuleset (empty) />

instead of

    <FooRuleset>
</FooRuleset>


I propose this patch for trunk and 2.10.


[ 42-improve-repr-empty-ruleset.diff ]

=== modified file ./utils/apparmor/rule/__init__.py
--- utils/apparmor/rule/__init__.py     2015-12-21 00:42:28.521222690 +0100
+++ utils/apparmor/rule/__init__.py     2015-12-21 22:57:03.421574167 +0100
@@ -291,7 +291,10 @@
 
     def __repr__(self):
         classname = self.__class__.__name__
-        return '<%s>\n' % classname + '\n'.join(self.get_raw(1)) + '</%s>' % 
classname
+        if self.rules:
+            return '<%s>\n' % classname + '\n'.join(self.get_raw(1)) + '</%s>' 
% classname
+        else:
+            return '<%s (empty) />' % classname
 
     def add(self, rule):
         '''add a rule object'''


Regards,

Christian Boltz
-- 
> Kann mir jemand sagen, wie unter der neuen SuSE der inetd gestartet
> wird? [...]  Ich such mir hier einen Affen...
hmm, hier? Wo ist hier? Und wenn Du einen findest, was dann?
Was willst Du mit einem Affen? Welchen denn? Wieso hier?
Wenn Du einen findest, kannst ihn ja mal fragen...

Vielleicht hilfts Dir, wenn Du mal in /etc/init.d /etc/syconfig
/etc/rc.config.d usw. suchst. Achso ja, Affen hab ich da noch keinen
gesehen;)         [> Andreas Meyer und Bernd Obermayr in suse-linux]

Attachment: 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

Reply via email to