packages: cronie/cronie.spec - use install(1) to do all in once, but it's r...

[glen]

Author: glen                         Date: Sun Apr 18 15:04:57 2010 GMT
Module: packages                      Tag: HEAD
---- Log message:
- use install(1) to do all in once, but it's racy?

---- Files affected:
packages/cronie:
   cronie.spec (1.30 -> 1.31) 

---- Diffs:

================================================================
Index: packages/cronie/cronie.spec
diff -u packages/cronie/cronie.spec:1.30 packages/cronie/cronie.spec:1.31
--- packages/cronie/cronie.spec:1.30	Tue Mar 16 15:28:27 2010
+++ packages/cronie/cronie.spec	Sun Apr 18 17:04:49 2010
@@ -140,10 +140,7 @@
 
 %post
 if [ ! -f /var/log/cron ]; then
-	umask 027
-	touch /var/log/cron
-	chgrp crontab /var/log/cron
-	chmod 660 /var/log/cron
+	install -m 660 -g crontab /dev/null /var/log/cron
 fi
 /sbin/chkconfig --add crond
 %service crond restart "Cron Daemon"
@@ -207,6 +204,9 @@
 All persons listed below can be reached at <cvs_login>@pld-linux.org
 
 $Log$
+Revision 1.31  2010/04/18 15:04:49  glen
+- use install(1) to do all in once, but it's racy?
+
 Revision 1.30  2010/03/16 14:28:27  megabajt
 - added CVE reference in rev 1.26
 
================================================================

---- CVS-web:
    http://cvs.pld-linux.org/cgi-bin/cvsweb.cgi/packages/cronie/cronie.spec?r1=1.30&r2=1.31&f=u