Thanks for including it in 1.4.4-1, but you didn't include this from my patch:

>               if [ $retvar -ne 0 ]
>               then
> +                     log_end_msg $retvar
>                       exit $retvar
>               fi

When reloading nginx when there is invalid configuration now results in:

root@host:~# /etc/init.d/nginx reload
[ ok ] Reloading nginx configuration: nginx.
root@host:~# echo "invalid config" >> /etc/nginx/nginx.conf
root@host:~# /etc/init.d/nginx reload
[....] Reloading nginx configuration: nginxroot@host:~# 

Including this line from my patch, it results in a better failure message:

root@host:~# /etc/init.d/nginx reload
[ ok ] Reloading nginx configuration: nginx.
root@host:~# echo "invalid config" >> /etc/nginx/nginx.conf 
root@host:~# /etc/init.d/nginx reload
[FAIL] Reloading nginx configuration: nginx failed!
root@host:~# 

Attachment: signature.asc
Description: Digital signature

Reply via email to