[FIXED JENKINS-5784]

Applied the change from Olav Reinert.
jenkinsci · Mar 22, 2012 · 8fb2f51 · 8fb2f51
1 parent 2b5a24c
commit 8fb2f51
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/changelog.html b/changelog.html
@@ -55,6 +55,9 @@
 <!-- Record your changes in the trunk here. -->
 <div id="trunk" style="display:none"><!--=TRUNK-BEGIN=-->
 <ul class=image>
+  <li class=bug>
+    Fixed a log rotation portability problem on RedHat RPM package.
+    (<a href="https://issues.jenkins-ci.org/browse/JENKINS-5784">issue 5784</a>)  
   <li class=bug>
     Computer.getHostName() returns null when it is not.
     (<a href="https://issues.jenkins-ci.org/browse/JENKINS-13185">issue 13185</a>)  

diff --git a/rpm/SOURCES/jenkins.logrotate b/rpm/SOURCES/jenkins.logrotate
@@ -8,6 +8,9 @@
     missingok
     create 644
     postrotate
-     kill -s SIGALRM `cat /var/run/jenkins.pid`
+      if [ -s /var/run/jenkins.pid ]; then
+        JPID=`cat /var/run/jenkins.pid`
+        test -n "`find /proc/$JPID -maxdepth 0 -user jenkins 2>/dev/null`" && /bin/kill -s ALRM $JPID || :
+      fi
     endscript
 }