yanegomi at gmail.com
Tue Jan 3 01:19:13 UTC 2012
On Mon, Jan 2, 2012 at 2:51 PM, Doug Barton <dougb at freebsd.org> wrote:
> On 01/02/2012 14:49, Garrett Cooper wrote:
>> On Mon, Jan 2, 2012 at 2:45 PM, Doug Barton <dougb at freebsd.org> wrote:
>>> On 01/02/2012 14:14, Garrett Cooper wrote:
>>>> How does this look for starters? The attached patch's goal is to
>>>> provide a generic, rc(5)-like infrastructure that would quiet down the
>>>> periodic emails for 120.clean-preserve .
>>> The periodic scripts are badly in need of attention, so effort in that
>>> area is much appreciated.
>>> Regarding your patch, rather than copying functions from rc.subr, why
>>> not just source it? Yes, you will get more than you need, but I think
>>> that the virtue of not having to maintain the same code in 2 places far
>>> outweighs that minor drawback.
>> That works too, assuming that rc.subr isn't too rc(5) centric.
> Well of course it's rc-centric, but that's not the point. :) If you're
> going to be using the exact same code from rc.subr, you might as well
> just source it. The things that you'll get by doing that which are only
> relevant to rc you just ignore.
>> Thanks for the feedback!
> Glad to help.
Here's a patch (untested apart from sh -n, but I'm going to toss
it into a VM to watch the sparks fly), based on stable/9 that makes
periodic(5) a bit more like rc(5).
Apart from that it resolves some inconsistencies with
800.zfs-scrub (the defaults were in the script and not
/etc/defaults/periodic.conf), removes duplicate rc=0 declarations,
consolidates and generalizes the "catmsgs" function, catches more
errors, and some other good stuff.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 59030 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/freebsd-current/attachments/20120103/87754bf8/periodic-rototill-0001.obj
More information about the freebsd-current