Patchwork logrotate: fix /etc/crontab entry -- run as root

login
mail settings
Submitter Jesse Gilles
Date Jan. 19, 2011, 7:13 p.m.
Message ID <1295464393-31400-1-git-send-email-jgilles@multitech.com>
Download mbox | patch
Permalink /patch/322/
State Accepted
Headers show

Comments

Jesse Gilles - Jan. 19, 2011, 7:13 p.m.
* /etc/crontab entry was missing username field, so it never ran

Signed-off-by: Jesse Gilles <jgilles@multitech.com>
---
 recipes/logrotate/logrotate_3.7.1.bb |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Patch

diff --git a/recipes/logrotate/logrotate_3.7.1.bb b/recipes/logrotate/logrotate_3.7.1.bb
index 850d76a..c084dcc 100644
--- a/recipes/logrotate/logrotate_3.7.1.bb
+++ b/recipes/logrotate/logrotate_3.7.1.bb
@@ -5,7 +5,7 @@  DEPENDS = "popt"
 RDEPENDS_${PN} = "cron"
 SECTION = "admin"
 LICENSE = "GPL"
-PR = "r3"
+PR = "r4"
 
 SRC_URI = "http://ftp.debian.org/debian/pool/main/l/logrotate/logrotate_${PV}.orig.tar.gz \
 	   file://logrotate-3.7.1.patch \
@@ -28,7 +28,7 @@  do_install () {
 
 pkg_postinst () {
     # Add the logrotate line to /etc/crontab
-    grep -q "${base_sbindir}/logrotate" ${sysconfdir}/crontab || echo "*/5 * * * *   ${base_sbindir}/logrotate ${sysconfdir}/logrotate.conf" >> ${sysconfdir}/crontab
+    grep -q "${base_sbindir}/logrotate" ${sysconfdir}/crontab || echo "*/5 * * * * root ${base_sbindir}/logrotate ${sysconfdir}/logrotate.conf" >> ${sysconfdir}/crontab
 }
 
 pkg_postrm() {