Bug#445203: debian-policy: 10.8. Log files: /etc/logrotate.d/<package> preferred
Raphael Geissert <geissert@debian.org> writes:
> While we are at it please let's use invoke-rc.d, which is what should be
> used. The current example could actually be considered as a violation
> of section 9.3.3.2 (if a logrotate file is considered as a "package
> maintainer script,") so please take this email as an objection.
> If invoke-rc.d is not used and the init script follows Policy's
> requirements for 'force-reload' and 'restart,' the service could be
> started in a runlevel where it has been explicitly disabled by the
> administrator. This does not address the case where a service has been
> stopped but is not disabled for the current runlevel, but that's an
> issue with invoke-rc.d.
Good point. Here's an updated patch.
diff --git a/policy.sgml b/policy.sgml
index bad28af..9399b1a 100644
--- a/policy.sgml
+++ b/policy.sgml
@@ -7868,11 +7868,13 @@ ln -fs ../sbin/sendmail debian/tmp/usr/bin/runq
</p>
<p>
- Log files must be rotated occasionally so that they don't
- grow indefinitely; the best way to do this is to drop a log
- rotation configuration file into the directory
- <file>/etc/logrotate.d</file> and use the facilities provided by
- logrotate.<footnote>
+ Log files must be rotated occasionally so that they don't grow
+ indefinitely. The best way to do this is to install a log
+ rotation configuration file in the
+ directory <file>/etc/logrotate.d</file>, normally
+ named <file>/etc/logrotate.d/<var>package</var></file>, and use
+ the facilities provided by <prgn>logrotate</prgn>.
+ <footnote>
<p>
The traditional approach to log files has been to set up
<em>ad hoc</em> log rotation schemes using simple shell
@@ -7897,25 +7899,28 @@ ln -fs ../sbin/sendmail debian/tmp/usr/bin/runq
section="8">):
<example compact="compact">
/var/log/foo/*.log {
-rotate 12
-weekly
-compress
-postrotate
-/etc/init.d/foo force-reload
-endscript
+ rotate 12
+ weekly
+ compress
+ missingok
+ postrotate
+ invoke-rc.d foo force-reload
+ endscript
}
</example>
This rotates all files under <file>/var/log/foo</file>, saves 12
compressed generations, and forces the daemon to reload its
- configuration information after the log rotation.
+ configuration information after the log rotation. It skips this
+ log rotation (via <tt>missingok</tt>) if no such log file is
+ present, which avoids errors if the package is removed but not
+ purged.
</p>
<p>
- Log files should be removed when the package is
- purged (but not when it is only removed). This should be
- done by the <prgn>postrm</prgn> script when it is called
- with the argument <tt>purge</tt> (see <ref
- id="removedetails">).
+ Log files should be removed when the package is purged (but not
+ when it is only removed). This should be done by
+ the <prgn>postrm</prgn> script when it is called with the
+ argument <tt>purge</tt> (see <ref id="removedetails">).
</p>
</sect>
--
Russ Allbery (rra@debian.org) <http://www.eyrie.org/~eagle/>
Reply to: