Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 12 Nov 2018 06:00:19 -0800 (PST)
From:      "Rodney W. Grimes" <freebsd@pdx.rh.CN85.dnsmgr.net>
To:        cem@freebsd.org
Cc:        "Rodney W. Grimes" <rgrimes@freebsd.org>, woodsb02@freebsd.org, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r340326 - head
Message-ID:  <201811121400.wACE0JK5025628@pdx.rh.CN85.dnsmgr.net>
In-Reply-To: <CAG6CVpV9tryq0yj9_r-rfinpYwmEwFS32Y25fxq%2BURMvY38BXw@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
> I absolutely approve of Ben's excellent addition to UPDATING, a text document.
At no point did I raise ANY objectection to the content
of his change, stop trying to make that the issue.

> Your demands for additional scrutiny and time-wasting around code
> review are ridiculous and disproportionate, and certainly a poor
> reward for Ben's straightforward and unimpeachable effort to improve
> our UPDATING documentation.

My demands are reasonable, as I stated them, your ignoring them
and casting focus on my non dmeands however is pointless.

> 
> Ben, thank you for submitting and committing the UPDATING change and
> please try to ignore Rod's hyper-criticism.

What you are totally ignoring is that Ben infact TAGGED me into
the review process, causing me to process yet another email only
to go look at the review to find it had been closed by a commit
in < 4 hours total elapsed time.

That is BULLSHIT and I am calling it BULLSHIT.  And though Ben
is the poor victom of example here, he is not the only guilty
party, and though this is a trivial change, it is not the change
that I am ranting about, but the fact that the process is
broken.

IF YOUR GOING TO TAG SOMEONE INTO A REVIEW YOU MUST GIVE THEM
TIME TO RESPOND BEFORE YOU COMMIT, otherwise your just wasting
there time.  

Do NOT tag me into a review if your going to commit it before
I can possibly be expected to respond.  I well rant on anyone
who does that.

CLEAR NOW?

> Conrad
> 
> 
> On Sat, Nov 10, 2018 at 5:07 PM Rodney W. Grimes
> <freebsd@pdx.rh.cn85.dnsmgr.net> wrote:
> >
> > [ Charset UTF-8 unsupported, converting... ]
> > > On Sun, 11 Nov 2018 at 2:43 am, Rodney W. Grimes <
> > > freebsd@pdx.rh.cn85.dnsmgr.net> wrote:
> > >
> > > > If your going to bother with a differential, and invite people to
> > > > it you should give them at least 24 hours, and preferably 72 hours
> > > > to respond to the new review.  Reviews that last < 4 hours are not
> > > > code reviews.
> > > >
> > > > --
> > > > Rod Grimes
> > > > rgrimes@freebsd.org
> > > >
> > > Hi Rod,
> > >
> > > Sorry. The main reason I submitted the code review is because I do not have
> > > my src commit bit, so needed to seek approval to commit.
> >
> > All the more reason for it to wait until the people you *invited* to
> > the review to have a chance to respond.
> >
> > I do not believe an "accept" in a review is an "approve to commit
> > beyond your normal scope".
> >
> > Was cem aware that he was "approving a non src committer to commit
> > this change to the src tree" as your review has no mention that you
> > are seeking src bit approval to commit there.
> >
> > >
> > > I felt the need to commit the UPDATING entry was time sensitive, given the
> > > change it was notifying users of had already been committed.
> > >
> > > Regards,
> > > Ben
> > > --
> >
> > --
> > Rod Grimes                                                 rgrimes@freebsd.org
> >
> 
> 

-- 
Rod Grimes                                                 rgrimes@freebsd.org



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