Nginx upgrade breaks php5-fpm

Bug #736225 reported by Marco Romeny
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Nginx
Fix Released
Undecided
Michael Lustfield
Debian
Fix Released
Unknown

Bug Description

I notice that the ppa ( only checked 0.8.54 ) uses /usr/share/nginx/www as the webroot instead of /var/www which confuses at least php5-fpm. Any good reason for the change? Of course, it would be great if php5-fpm wouldn't just assume it, or that it owned the directory itself, but /usr/share/nginx/www seemed unusual...

I noticed when upgrading from ubuntu's stock 0.7.* to 0.8.4 ( the upgrade isn't clean -- errors out until I remove 0.7.* and install 0.8.* ) and php5-fpm refuses to serve anymore due to the now missing /var/www

Sorry to reopen this as a bug - I do believe it's fairly significant and will impact quite a few people. The 'answer' I got to the previously filed question "/usr/share/nginx/www is good" is not really an answer - merely an opinion. Blue is a nice color.

Revision history for this message
Michael Lustfield (michaellustfield) wrote :

"Blue is a nice color?" Hm?

Debian policy states that /var/www should not be used. If php5-fpm expects that directory to exist, then it should handle that, not Nginx. I'll add a fix for this in the php5 package that is supplied by our PPA.

Because of part 4 in Debian Policy 11.5 I will not add this back to Nginx.

Revision history for this message
Michael Lustfield (michaellustfield) wrote :

I added the directory to nginx-common.dirs. It's uploaded to launchpad and will be building and published soon (depending on the queue). It takes about an hour of build time once it gets through the queue...

Changed in nginx:
status: New → Fix Released
Revision history for this message
Marco Romeny (marco-mimecom) wrote :

I like Red too!

Glag the bug is fixed. To mee it's strange that /var/www isn't supposed to be used. I would believe that data that can be used by similar packages should stay fixed in one place. Apache, lighttpd, nginx all have as their main function to serve html-files, so it's make sense for /var/www to be a common place for such files in a default environment.

Truth be told, I have almost always set up my own directory for the sake of doing virtual hosting etc, but it would be nice for people who don't dig around config-files to have a standard directory whatever webserver they elect to use...

Revision history for this message
Michael Lustfield (michaellustfield) wrote :

hmm... build broke... I'll look into this when I toss out the next version of Nignx.

Changed in nginx:
status: Fix Released → In Progress
assignee: nobody → Michael Lustfield (mtecknology)
Revision history for this message
Michael Lustfield (michaellustfield) wrote :

Alrighty, I have the php5 package building again. Bumped up to a newer version as well.

Revision history for this message
Michael Lustfield (michaellustfield) wrote :

Apparently there's some ugliness in the php5 source that is causing issues with ./configure checks. I tried a dirty ugly hack.... We'll see if this works.

Revision history for this message
Marco Romeny (marco-mimecom) wrote :

php5 has always been dirty business, but it gets the job done! like the mob kinda... Much appreciate the work though! Somehow I never feel comfortable compiling and packing myself -- always feel certain I'd do it wrong, and even if I didn't complainers like me would just come with opinions!

Revision history for this message
Michael Lustfield (michaellustfield) wrote :

In the php5 package that the Nginx team is uploading, a fix was added by adding '/var/www/' to 'php5-common.dirs'. It _should_ probably go into php5-fpm.dirs but I didn't know if any of the other php5 packages depend on a directory that they don't supply.

The version we have has the fix applied but I will forward the bug upstream and link this bug report to it so you can follow.

Changed in nginx:
status: In Progress → Fix Released
Changed in debian:
status: Unknown → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.