Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 12 Feb 2019 08:31:52 -0800 (PST)
From:      "Rodney W. Grimes" <freebsd@pdx.rh.CN85.dnsmgr.net>
To:        Warner Losh <imp@bsdimp.com>
Cc:        "Rodney W. Grimes" <rgrimes@freebsd.org>, Dmitry Morozovsky <marck@freebsd.org>, Cy Schubert <Cy.Schubert@cschubert.com>, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-12@freebsd.org
Subject:   Re: svn commit: r344051 - in stable/12/sbin: newfs tunefs
Message-ID:  <201902121631.x1CGVqWE076720@pdx.rh.CN85.dnsmgr.net>
In-Reply-To: <CANCZdfoghrVR3uMQM3oqzv31g732iOHJP=iEeO2qD8kbgUwVrw@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
> On Tue, Feb 12, 2019 at 9:21 AM Rodney W. Grimes <
> freebsd@pdx.rh.cn85.dnsmgr.net> wrote:
> 
> > > On Tue, 12 Feb 2019, Cy Schubert wrote:
> > >
> > > > In message <201902121358.x1CDwGf9047873@repo.freebsd.org>, Dmitry
> > > > Morozovsky wr
> > > > ites:
> > > > > Author: marck (doc committer)
> > > > > Date: Tue Feb 12 13:58:16 2019
> > > > > New Revision: 344051
> > > > > URL: https://svnweb.freebsd.org/changeset/base/344051
> > > > >
> > > > > Log:
> > > > >   MFC 343548:
> > > > >
> > > > >     Allow dashes as a valid character in UFS labels.
> > > >
> > > > Approved by: Whom?
> > >
> > > Sorry, I missed commit log metadata from the original change.
> > >
> > > Lesson learnt.
> >
> > I think your fine on how you did this, I certainly already
> > knew from reading the ^head commits that this change had
> > infact been approved by Kirk, and had been flagged for MFC.
> >
> > I do not see value in duplicationg all that info in the
> > stable branches, but perhaps we should formalize that in
> > the committers guide on commit messages?
> >
> 
> Let's not overreact. Someone made a commit, someone else didn't go check
> the original commit to see it was legit and asked a question. The original
> bonafides were provided. Case closed. No need to make a federal case out of
> it. The doc committer learned to take a little extra care on their commit
> messages, it seems, as doc committers committing to src get a little extra
> scrutiny. The original complainer likely noticed the info was in the
> original commit and may check there in the future before complaining. Or
> maybe he won't. It isn't a big deal either way.
> 
> None of this requires a change to the rules.

I advocating better documenting better how we want to operate,
if you see no value in that, so be it, but imho we are
way to large and way to loose to continue to funciton in
the manner we have been.

If all of our process was better documented and layed down
we would have less of this type of discussion on a much less
frequent basis.  

Again, imho, this is not an over reaction, but a simple reaction to
what occurred, with a proposed, and what imho, is a reasonable step
forward, rather than just staying at status quo.

New committers need to know many more things than what is currently
in our committers guide, lets improve that.

-- 
Rod Grimes                                                 rgrimes@freebsd.org



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