Re: [PATCH] pull: abort by default if fast-forwarding is impossible

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Jun 26, 2021 at 9:16 PM Felipe Contreras
<felipe.contreras@xxxxxxxxx> wrote:
>
> Elijah Newren wrote:
>
> > The code changes look good, but you'll need to add several test
> > changes as well, or this code change will cause test failures.
>
> Wouldn't you consider sending a patch without running 'make test'
> "cavalier"?
>
> > Thanks for working on this.
>
> Such a completely different tone for a "cavalier" patch depending 100%
> on the person who sent it. Weird.

First of all, attacking a random bystander like Alex is rather
uncalled for.  As far as we know, Alex is trying to help out and made
a mistake or oversight, but will likely correct the problem if kindly
pointed out.  This email is egregiously far from constructive
criticism for Alex.

Second, why do you grossly mischaracterize folks (not just me) and try
so hard to insinuate unfairness?  I don't get it.  Do you really not
understand?

Let me attempt to explain...



Not running the tests is a problem, sure.  I pointed it out.  It could
have been a mistake or oversight on Alex's part.  Now, if Alex were to
state it was not accidental and then repeatedly defend submitting
patches without running tests, I would absolutely start calling out
both his submission and his professed level of reasonable due
diligence.  He hasn't done that though.  People deserve a chance to
respond and correct things first.

We gave you that same opportunity.  It was pointed out to you rather
politely that you had neglected to act on a report that a patch caused
a segfault[1]; in fact, you re-submitted the old patch as-is and
suggested it might be a good default[2].  You had a chance to point
out that what you had done was just a mistake or oversight.  Instead,
you confirmed that you saw the report and just didn't bother acting on
it[3].  And then you defended your misbehavior[4].  Over[5] and
over[6] and over[7] again.  And you also defended your related
patches[8] instead of double checking or trying to correct them --
even when prompted to double check[9] -- despite those additional
patches having severe problems of their own[10].

So, yes, I used terms like negligent, reckless, careless, and cavalier
in my critiques with you, but I very carefully always used those terms
to describe either the patches you submitted or the amount of due
diligence that you repeatedly defended.  I never applied those terms
to you.  And those terms didn't apply to you, as you could have chosen
to change and use a different level of due diligence.  So, your
accusations of personal attacks, harassment, and ad hominem arguments
that you made against me earlier[11], and your insinuations of
unfairness both then and now, are all false.

This, of course, isn't your first time grossly mischaracterizing
events and trying to claim unfairness.  Let me give you a second
example of why there was no unfairness in case the first is hard for
you to understand.  Let's use [12] as another example, and explain the
lead-up to that time: I politely gave you a chance to correct a
misleading claim[13], which you then instead admitted to being no
mistake[14], and didn't bother to correct.  The original misleading
claim was a minor issue.  If it had been corrected (even if it had
been intentional), it could have been no big deal.  The reason I drew
significant attention to it[15] was due to your response.  I pointed
out quite clearly when I drew attention to it, that it was your
*response* to the error that was problematic, and even reiterated it
in a follow-up.  I do not see why you ignored that, mischaracterized
my objections as being entirely about the initial error, hand picked
an example that didn't even do what you said it did, all to lead up to
your "Could it be that this has absolutely nothing to do with the
action, and everything to do with the person doing the action?"  The
answer is quite simply "Of course not."  I would also call out other
people if they stated that misleading claims of theirs were
intentionally submitted and did not try to correct them.  I just
haven't ever seen anyone else do that.  (Let alone follow it up again
a short time later quite similarly[16], except that at least you
corrected it.)

Unfortunately, these aren't isolated incidents of claims of
unfairness, and it's not just me you hurl these accusations against.
In fact, in other cases with other people you go much further
overboard.  For example, you recently used the phrase "petty personal
animus"[17] with Junio.  Perhaps you don't see the problem, so let me
state it in a way you might recognize:
     Could it be that you always use these perjorative phrases in
questions just to provide plausible deniability that you are directly
accusing folks of severe personal failings?
Just as I shouldn't ask the above question (I only did so to try to
highlight how your questions come across), you shouldn't be asking
yours.  It's inappropriate.  Putting such negative words in a question
format might make the phrases less objectionable, but the questions
are still way out of line.  If you react negatively to my example
sentence above, hopefully that helps you see why that is
inappropriate.  Please correct this pattern.

And all of this hardly even touches on your interactions with
reviewers, which sadly is following the same arc as in 2013 and
2014[18].

In short, please stop:
  * Please stop attacking random bystanders like Alex (or other folks[19])
  * Please stop defending shockingly inadequate levels of due
diligence, and adopt a higher one.
  * Please stop accusing others of bias, unfairness, and other
shortcomings.  Learn to recognize why your behavior often results in
others changing theirs.
  * And please find ways to stop burning out reviewers, especially
since they are the rate-limiting resource in determining git's overall
velocity


Please know that I really do think you are talented.  I tried really
hard to help you change and improve because I thought you could have
been a great addition to the community.  I'm nowhere near as talented
as Michael in expressing it (again, [18]), but I really wish you had
chosen to change.  I agree with his sentiments back then.


[1] https://lore.kernel.org/git/YMYnVWSEgxvKRU9j@xxxxxxxxxxxxxxxxxxxxxxx/
[2] https://lore.kernel.org/git/20210613143155.836591-1-felipe.contreras@xxxxxxxxx/
[3] https://lore.kernel.org/git/60c647c1d9b5c_41f452089@natae.notmuch/
[4] https://lore.kernel.org/git/60c82a622ae66_e5292087f@natae.notmuch/
[5] https://lore.kernel.org/git/60c86ff87d598_e6332085b@natae.notmuch/
[6] https://lore.kernel.org/git/60c85bfd112a9_e633208d5@natae.notmuch/
[7] https://lore.kernel.org/git/60cb7f3e66cfd_1305720822@natae.notmuch/

[8] https://lore.kernel.org/git/60c85bfd112a9_e633208d5@natae.notmuch/
[9] https://lore.kernel.org/git/CABPp-BGZ2H1MVgw9RvSdogLMdqsX3n89NkkDYDa2VM3TRHn7tg@xxxxxxxxxxxxxx/
[10] https://lore.kernel.org/git/CABPp-BEXtJtkkh9Diuo4e1K1ci=vggGxkLRDfkpOH12LM8TCfA@xxxxxxxxxxxxxx/

[11] https://lore.kernel.org/git/60cb5a02f1b31_1259c2086f@natae.notmuch/
[12] https://lore.kernel.org/git/60cb7f3e66cfd_1305720822@natae.notmuch/
[13] https://lore.kernel.org/git/CABPp-BGstXDbzxpySw7q_jn22HD05MsrZeHNv+kXFHOFS2_WCQ@xxxxxxxxxxxxxx/
[14] https://lore.kernel.org/git/60c887f678c88_e63320846@natae.notmuch/
[15] https://lore.kernel.org/git/CABPp-BG53Kd7MhzE3hdq5fjBQVV2Ew3skcUCAuTfM5daP2wmZA@xxxxxxxxxxxxxx/
[16] https://lore.kernel.org/git/CABPp-BF1noWhiJadHzjJmnGo8hdZj6Fk7XnZ=u6BVVSGfHE7og@xxxxxxxxxxxxxx/

[17] https://lore.kernel.org/git/60d289c84fadf_312208dc@natae.notmuch/

[18] https://lore.kernel.org/git/53709788.2050201@xxxxxxxxxxxx/

[19] "if you eyes have trouble seeing", from
https://lore.kernel.org/git/60839422353fc_10cb9208c7@natae.notmuch/
       https://lore.kernel.org/git/87fszfafkh.fsf@xxxxxxxxxxxxxxx/



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux