Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 247746 - www-servers/nginx - init.d script modification
Summary: www-servers/nginx - init.d script modification
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Konstantin Arkhipov (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-11-20 14:22 UTC by Oleg Gawriloff
Modified: 2010-01-03 19:53 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Oleg Gawriloff 2008-11-20 14:22:49 UTC
In /etc/init.d/nginx we see:
depend() {
        need net
        use dns logger 
}
In case where some paths on network systems this causes problems when netmount stop issued.
So this must be rewritten as it done in /etc/init.d/apache:
depend() {
        need net
        use dns logger netmount
}



Reproducible: Always
Comment 1 Sebastian Bauer 2009-06-01 22:09:33 UTC
init script supposed to have:
    kill -USR1 `cat /var/run/nginx.pid` &>/dev/null
not:
    kill -HUP `cat /var/run/nginx.pid` &>/dev/null

in reload section because when you send HUP nginx doesnt reopen logs and you can loose some

tested with app-admin/logrotate and /etc/init.d/nginx reload > /dev/null 2>&1 || true inside rotate definition
Comment 2 Dirkjan Ochtman (RETIRED) gentoo-dev 2010-01-03 19:53:45 UTC
I added the netmount depend in nginx-0.7.64-r2.

We don't want to change reload to use USR1 because it *only* reopens the logs, it doesn't actually reload the config (which seems more important there).