Date: Mon, 2 Jan 2012 00:21:08 -0500 From: Ben Kaduk <minimarmot@gmail.com> To: Ulrich Spoerlein <uqs@freebsd.org>, Ben Kaduk <minimarmot@gmail.com>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r228990 - in head/usr.sbin: IPXrouted adduser bluetooth/btpand bluetooth/sdpd bootparamd/bootparamd bsnmpd/modules/snmp_bridge bsnmpd/modules/snmp_hostres bsnmpd/modules/snmp_wlan bsnmp... Message-ID: <CAK2BMK7W2SC0j7-TDSVaiCkzn5nk7tDC14Sy-txRxCmrUWps=g@mail.gmail.com> In-Reply-To: <20120101173342.GN83814@acme.spoerlein.net> References: <201112301058.pBUAwFsw010478@svn.freebsd.org> <CAK2BMK6p3VJXezt%2Bpa7-0MbP4=Uv79i=D=cxmDrpAwUK2Mw2vQ@mail.gmail.com> <20120101173342.GN83814@acme.spoerlein.net>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, Jan 1, 2012 at 12:33 PM, Ulrich Spoerlein <uqs@freebsd.org> wrote:
> On Fri, 2011-12-30 at 14:43:22 -0500, Ben Kaduk wrote:
>> On Fri, Dec 30, 2011 at 5:58 AM, Ulrich Spoerlein <uqs@freebsd.org> wrot=
e:
>> > Modified: head/usr.sbin/cron/doc/CHANGES
>> > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D
>> > --- head/usr.sbin/cron/doc/CHANGES =A0 =A0 =A0Fri Dec 30 10:45:00 2011=
=A0 =A0 =A0 =A0(r228989)
>> > +++ head/usr.sbin/cron/doc/CHANGES =A0 =A0 =A0Fri Dec 30 10:58:14 2011=
=A0 =A0 =A0 =A0(r228990)
>> > @@ -1,3 +1,6 @@
>> > +$FreeBSD$
>> > +--------
>> > +
>> > =A0Vixie Cron =A0 =A0 =A0 =A0 =A0 =A0 Changes from V2 to V3
>> > =A0Paul Vixie
>> > =A029-Dec-1993
>> > @@ -20,7 +23,7 @@ be reread whenever it changes.
>> >
>> > =A0I also added a "-e" option to crontab(1). =A0Nine people also sent =
me diffs
>> > =A0to add this option, but I had already implemented it on my own. =A0=
I actually
>> > -released an interrim version (V2.2, I think) for limited testing, and=
got a
>> > +released an interim version (V2.2, I think) for limited testing, and =
got a
>>
>> Should we really be in the business of modifying Vixie's changelog?
>
> I'm replying here, but same goes for your comments on the other revision
> (thanks for all those, btw).
>
> Warning, rant ahead:
I can handle a rant :)
>
> Our current SVN repo is in an inconsistent state thanks to the dubious
> introduction of svn keywords. There are files in there, converted over
> via cvs2svn and never touched since, but our presubmit checks want to
> either see a $FreeBSD$ tag in modified files, or have fbsd:nokeywords
> set. The latter is kinda a no-brainer for everything under contrib/ but
> I'm struggling with what to do exactly for files that are non-contrib
> and may or may not be non-code. Should the keywords be set on $subdir/*
> or can I punch holes in the web of keywords as I see fit? (e.g.
> $subdir/{README,CHANGES,foo.txt})
>
> What's the point then anyway? SVN doesn't track files, but changesets.
> Gah!
I guess I did assume that the $FreeBSD$ keyword was being added
because of the presubmit checks, but didn't actually say so. Sorry.
It definitely is a grey area in which I don't presume to have an
opinion, and I sympathize with your frustration.
>
> Oh wait, this ain't about the $FreeBSD$ keyword, but the typo-fix? I'm
> sure that Vixie won't mind and do believe that correctness of spelling
> is a worthy goal as we always tout the quality of our documentation.
>
I can accept that argument -- I mostly wanted to make sure this wasn't
just a mechanical "run aspell over everything" or similar (though a
later change did reveal that to not be the case).
> FWIW, I'm not going to touch stuff under contrib/, etc.
Sounds fine.
>
>> > =A0/*
>> > - * Delete a rotated logfiles, when using clasic filenames.
>> > + * Delete a rotated logfiles, when using classic filenames.
>>
>> "a logfiles" is inconsistent. =A0Without looking up the implementation,
>> I'm going to guess that "a rotated logfile" is correct.
>
> When proofreading the spell-checked output I focused too narrowly on
> just the typo that I fixed so that other typos in the same line/sentence
> mostly went unnoticed. Will do better in the next couple of commits.
It's easy to do -- I actually found myself also doing so a couple
times when going through these. (There were quite a number of them,
and thanks for going through them all by hand!)
-Ben Kaduk
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAK2BMK7W2SC0j7-TDSVaiCkzn5nk7tDC14Sy-txRxCmrUWps=g>