[selinux-policy: 1/8] Fix genman to stop truncating boolean descriptions that do not end with a '.', allow users to specif

Daniel J Walsh dwalsh at fedoraproject.org
Thu Aug 2 15:47:30 UTC 2012


commit 03f80ae03ad92f37667dfff7e8427879e0bae961
Author: Dan Walsh <dwalsh at redhat.com>
Date:   Thu Jul 26 16:11:52 2012 -0400

    Fix genman to stop truncating boolean descriptions that do not end with a '.', allow users to specify a list of domains to produce, fix spelling mistake

 genman.py |    9 +++++++--
 1 files changed, 7 insertions(+), 2 deletions(-)
---
diff --git a/genman.py b/genman.py
index 3b649ca..f4c920d 100755
--- a/genman.py
+++ b/genman.py
@@ -209,7 +209,9 @@ control.
                 if b.endswith("anon_write"):
                     self.anon_list.append(b)
                 else:
-                    desc = seobject.booleans_dict[b][2][0].lower() + seobject.booleans_dict[b][2][1:-1]
+                    desc = seobject.booleans_dict[b][2][0].lower() + seobject.booleans_dict[b][2][1:]
+                    if desc[-1] == ".":
+                        desc = desc[:-1]
                     self.booltext += """
 .PP
 If you want to %s, you must turn on the %s boolean.
@@ -361,7 +363,7 @@ Path%s:
         self.fd.write("""
 
 .PP
-Note: File context can be temporarily modified with the chcon command.  If you want to permanantly change the file context you need to use the 
+Note: File context can be temporarily modified with the chcon command.  If you want to permanently change the file context you need to use the 
 .B semanage fcontext 
 command.  This will modify the SELinux labeling database.  You will need to use
 .B restorecon
@@ -441,5 +443,8 @@ selinux(8), %s(8), semanage(8), restorecon(8), chcon(1)
         if self.booltext != "":        
             self.fd.write(", setsebool(8)")
 
+if len(sys.argv) > 2:
+        domains = sys.argv[2:]
+
 for domainname in domains:
     ManPage(domainname, sys.argv[1])


More information about the scm-commits mailing list