[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#37342: debian-policy: [PROPOSED] move to logrotate



>     bazsi>      Here is a good example for a logrotate config file
>     bazsi>      (for more information see logrotate(8)):
> 
>     bazsi>         /var/log/apache/* {
>     bazsi>            [...]
>     bazsi>            kill -HUP `cat /var/run/apache.pid`
>     bazsi>            endscript
>     bazsi>         }
> 
>     bazsi>      Which rotates all files under `/var/log/apache', saves
>     bazsi>      12 compressed generations, and sends a HUP signal at
>     bazsi>      the end of rotation.
> 
>  Shouldn't that use `/etc/init.d/apache reload' instead?  Most things,
>  as far as I know, will work that way, sed -e 's/apache/$DAEMON/'.  I
>  think it would be good to display the /etc/init.d/* method in this
>  policy item, as a way of documenting that feature of the sysvinit
>  scripts, which provide a standard interface to reloading any daemon.

hmmm... I agree with you here. That example was intended to show the usage
of logrotate, not meant to show the way of restarting a daemon. 

So the line "kill -HUP `cat /var/run/apache.pid`" should read:

/etc/init.d/apache restart

-- 
Bazsi
PGP key: http://www.balabit.hu/pgpkey.txt, or finger bazsi@balabit.hu


Reply to: