Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 25 Sep 2002 17:51:04 +0200
From:      Roman Neuhauser <neuhauser@bellavista.cz>
To:        Bruce M Simpson <bms@spc.org>, freebsd-hackers@freebsd.org
Subject:   Re: periodic(8)-produced diffs
Message-ID:  <20020925155104.GR30361@freepuppy.bellavista.cz>
In-Reply-To: <20020923151059.GJ23343@spc.org>
References:  <20020923150536.GB30361@freepuppy.bellavista.cz> <20020923151059.GJ23343@spc.org>

next in thread | previous in thread | raw e-mail | index | archive | help
# bms@spc.org / 2002-09-23 16:10:59 +0100:
> On Mon, Sep 23, 2002 at 05:05:36PM +0200, Roman Neuhauser wrote:
> [... re periodic diffs]
> > And, what would the preferred interface be? Most of periodic.conf
> > knobs are bools, but I'm not sure
> > diff_{context,traditional,unified}_format="{YES,NO}"
> > is better than
> > diff_format="{context,traditional,unified}"
> 
> No reason why rc.conf knobs *have* to be bools, what ultimately
> matters is the case...esac which parses them. I'd avoid relying on a
> default value in /etc/defaults/rc.conf for the sake of robustness.
> 
> I prefer unified diffs, personally.

    thanks for your input. another thing I'm not sure about: should this
    knob be limited to just setting the diff format? there's also the
    possibility of going the rc.conf ${program}_flags path.

    i guess the question is whether the interface should be limited to a
    safe set of settings, or if the users should be able to shoot
    themselves in the foot in any way they want.
    
    quite a few diff switches don't make any sense in the actions
    performed by the periodic scripts, but IMO it's root's duty to
    choose a reasonable set.

    i think i'll try the $diff_program_options route, and post the
    patch later today or tomorrow.

-- 
begin 666 nonexistent.vbs
FreeBSD 4.7-RC
5:43PM up 8 days, 58 mins, 18 users, load averages: 0.06, 0.12, 0.07
end

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-hackers" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20020925155104.GR30361>