Date: Tue, 22 Feb 2011 21:13:40 +0000 (UTC) From: Doug Barton <dougb@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/etc/periodic/daily 310.accounting src/etc/rc.d accounting Message-ID: <201102222113.p1MLDvpP013082@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
dougb 2011-02-22 21:13:40 UTC
FreeBSD src repository
Modified files:
etc/periodic/daily 310.accounting
etc/rc.d accounting
Log:
SVN rev 218961 on 2011-02-22 21:13:40Z by dougb
Update how accounting log files are rotated.
The old version had a race between the time that the old file was
cp'ed to acct.0 and the time that 'sa -s' was run that prevented
the commands that occurred in the meantime from being backed up.
It's also arguable that the old version was inefficient in using
cp which can be a problem on a space-constrained system.
This version avoids both problems, albeit it's considerably more
complicated. The advantage of putting the log rotation in the rc.d
script is that it can handle the _enable and _file questions without
having to do gymnastics to discover either value in the periodic script.
As a side effect of reviewing the rc.d script I cleaned it up a bit.
Revision Changes Path
1.10 +6 -3 src/etc/periodic/daily/310.accounting
1.12 +32 -8 src/etc/rc.d/accounting
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201102222113.p1MLDvpP013082>