[PPA Regression] Update caused a regression of lucid init scripts trying to place a pid file in /run; Regression causing logrotate to break in lucid

[Thomas Ward]

This is a regression of https://bugs.launchpad.net/nginx/+bug/1072641 caused by the PPA update (in which the fix for that bug may not have been included in the update). This regression only affects the Lucid packages in the stable PPA (https://launchpad.net/~nginx/+archive/stable/).

[Thomas Ward (teward)]

A fix has been uploaded to the PPA for Lucid to repair this issue. This fix will need verification, if someone can verify that this regression no longer exists with the fix that I uploaded, I will mark this Fix Confirmed.

[Rowan Wookey (rwky)]

I've tested with 1.2.7-0ubuntu0ppa2~lucid and it's still broken.

The problem is the init scripts are looking for the pid in /var/run
Nginx is creating the pid in /run
On lucid /run isn't symlinked to /var/run

So the init scripts for lucid need to use /run or nginx needs to use /var/run for it's pid.

Note the logrotate script also checks /var/run so if you change the init scripts logrotate will need updating too.

[Thomas Ward (teward)]

Oops i missed a line from my initial diff... my bad, fix committed, please test it once it builds (I forgot to change the nginx.conf file to update where it creates the pid)

[Rowan Wookey (rwky)]

OK it's fixed!