Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 377713 - www-servers/nginx configcheck improvements
Summary: www-servers/nginx configcheck improvements
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal trivial (vote)
Assignee: Benedikt Böhm (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-04 10:26 UTC by Johan Bergström
Modified: 2011-09-07 08:34 UTC (History)
3 users (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 Johan Bergström 2011-08-04 10:26:37 UTC
Hello,
am I the only one tired of seeing this?

# /etc/init.d/nginx restart
 * Caching service dependencies ...                                       [ ok ]
 * Checking nginx' configuration ...
nginx: the configuration file /etc/nginx/nginx.conf syntax is ok
nginx: configuration file /etc/nginx/nginx.conf test is successful        [ ok ]
 * Stopping nginx ...                                                     [ ok ]
 * Checking nginx' configuration ...
nginx: the configuration file /etc/nginx/nginx.conf syntax is ok
nginx: configuration file /etc/nginx/nginx.conf test is successful        [ ok ]
 * Starting nginx ...                                                     [ ok ]

adding -q to flags (nginx -c <foo> -t -q) will instead output this:

# /etc/init.d/nginx restart
 * Caching service dependencies ...                                       [ ok ]
 * Checking nginx' configuration ...                                      [ ok ]
 * Stopping nginx ...                                                     [ ok ]
 * Checking nginx' configuration ...                                      [ ok ]
 * Starting nginx ...                                                     [ ok ]

I'm a bit puzzled by checking config twice on a restart; but I guess thats what reload is for. On the other hand, i see that at least init.d/sshd does this:

    if [ "${RC_CMD}" = "restart" ] ; then
        checkconfig || return 1
    fi

I wouldn't mind seeing this implemented as well.

Thanks for listening!
Comment 1 Benedikt Böhm (RETIRED) gentoo-dev 2011-09-07 08:34:38 UTC
fixed, thanks