[mailman] fix #838580 - check if directories are 0755, there is no need to need

[Jan Kaluža]

commit 5d3c3e491e6dff5de6d49dc2d4ad059c5769d96b
Author: Jan Kaluza <hanzz.k at gmail.com>
Date:   Tue Jan 22 10:26:59 2013 +0100

    fix #838580 - check if directories are 0755, there is no need to need
    
      02775 on all Mailman directories

 mailman-2.1.15-check_perms.patch |   22 ++++++++++++++++++++++
 mailman.spec                     |    8 +++++++-
 2 files changed, 29 insertions(+), 1 deletions(-)
---
diff --git a/mailman-2.1.15-check_perms.patch b/mailman-2.1.15-check_perms.patch
new file mode 100644
index 0000000..3baa92f
--- /dev/null
+++ b/mailman-2.1.15-check_perms.patch
@@ -0,0 +1,22 @@
+diff --git a/bin/check_perms b/bin/check_perms
+index 5ef3306..d50518a 100755
+--- a/bin/check_perms
++++ b/bin/check_perms
+@@ -70,7 +70,7 @@ class State:
+ 
+ STATE = State()
+ 
+-DIRPERMS = S_ISGID | S_IRWXU | S_IRWXG | S_IROTH | S_IXOTH
++DIRPERMS = S_IRWXU | S_IRGRP | S_IROTH | S_IXOTH
+ QFILEPERMS = S_ISGID | S_IRWXU | S_IRWXG
+ PYFILEPERMS = S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH
+ ARTICLEFILEPERMS = S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP
+@@ -195,7 +195,7 @@ def checkall():
+             continue
+         if (mode & DIRPERMS) <> DIRPERMS:
+             STATE.ERRORS += 1
+-            print C_('directory must be at least 02775: %(d)s'),
++            print C_('directory must be at least 0755: %(d)s'),
+             if STATE.FIX:
+                 print C_('(fixing)')
+                 os.chmod(d, mode | DIRPERMS)
diff --git a/mailman.spec b/mailman.spec
index 8074c36..75df44c 100644
--- a/mailman.spec
+++ b/mailman.spec
@@ -1,7 +1,7 @@
 Summary: Mailing list manager with built in Web access
 Name: mailman
 Version: 2.1.15
-Release: 7%{?dist}
+Release: 8%{?dist}
 Epoch: 3
 Group: Applications/Internet
 Source0: ftp://ftp.gnu.org/pub/gnu/mailman/mailman-%{version}.tgz
@@ -36,6 +36,7 @@ Patch18: mailman-2.1.12-initcleanup.patch
 # the service is now off by default
 Patch20: mailman-2.1.12-init-not-on.patch
 Patch21: mailman-2.1.13-env-python.patch
+Patch22: mailman-2.1.15-check_perms.patch
 
 License: GPLv2+
 URL: http://www.list.org/
@@ -127,6 +128,7 @@ additional installation steps, these are described in:
 %patch18 -p1 -b .initcleanup
 %patch20 -p1
 %patch21 -p1
+%patch22 -p1
 
 #cp $RPM_SOURCE_DIR/mailman.INSTALL.REDHAT.in INSTALL.REDHAT.in
 cp %{SOURCE5} INSTALL.REDHAT.in
@@ -532,6 +534,10 @@ exit 0
 %dir %attr(775,root,%{mmgroup}) %{lockdir}
 
 %changelog
+* Tue Jan 22 2013 Jan Kaluza <jkaluza at redhat.com> - 3:2.1.15-8
+- fix #838580 - check if directories are 0755, there is no need to need
+  02775 on all Mailman directories
+
 * Thu Dec 06 2012 Jan Kaluza <jkaluza at redhat.com> - 3:2.1.15-7
 - remove fuzz=3 and rebase patches