[Pkg-mailman-hackers] Pkg-mailman commit - rev 630 - trunk/debian

[Thijs Kinkhorst]

Author: thijs
Date: 2009-09-25 09:31:03 +0000 (Fri, 25 Sep 2009)
New Revision: 630

Modified:
   trunk/debian/changelog
   trunk/debian/postinst
   trunk/debian/rules
Log:
Remove potentially long running 'find' command in postinst, as
permissions are already set correctly in the deb. Thanks Paul
Slootman, closes: #544046.

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog	2009-05-22 09:20:43 UTC (rev 629)
+++ trunk/debian/changelog	2009-09-25 09:31:03 UTC (rev 630)
@@ -1,3 +1,11 @@
+mailman (1:2.1.12-3) UNRELEASED; urgency=low
+
+  * Remove potentially long running 'find' command in postinst, as
+    permissions are already set correctly in the deb. Thanks Paul
+    Slootman, closes: #544046.
+
+ -- Thijs Kinkhorst <thijs at debian.org>  Fri, 25 Sep 2009 11:30:25 +0200
+
 mailman (1:2.1.12-2) unstable; urgency=low
 
   [ Lionel Elie Mamane ]

Modified: trunk/debian/postinst
===================================================================
--- trunk/debian/postinst	2009-05-22 09:20:43 UTC (rev 629)
+++ trunk/debian/postinst	2009-09-25 09:31:03 UTC (rev 630)
@@ -174,11 +174,6 @@
 	    install -d -o root -g list -m 2775 /var/lock/mailman
 	fi
 	sg list -c "umask 002; /usr/lib/mailman/bin/update;"
-
-	# fix permissions
-	find /var/lib/mailman/ -type d -exec chmod g+s \{\} \;
-	chmod o-rx /var/lib/mailman/archives/private
-	chown www-data /var/lib/mailman/archives/private
     ;;
 
     *)

Modified: trunk/debian/rules
===================================================================
--- trunk/debian/rules	2009-05-22 09:20:43 UTC (rev 629)
+++ trunk/debian/rules	2009-09-25 09:31:03 UTC (rev 630)
@@ -167,7 +167,8 @@
 	chmod -x debian/mailman/usr/lib/$(package)/scripts/paths.py \
 		debian/mailman/usr/lib/$(package)/Mailman/Cgi/*
 
-	chmod o-x debian/mailman/var/lib/$(package)/archives/private
+	chmod o-rx debian/mailman/var/lib/$(package)/archives/private
+	chgrp www-data debian/mailman/var/lib/$(package)/archives/private
 	chmod 0755 debian/mailman/usr/lib/$(package)/cron/*
 	chmod 0644 debian/mailman/usr/lib/$(package)/cron/{crontab.in,paths.py}