Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Mime-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset=UTF-8
Date:   Fri, 09 Jun 2023 18:37:37 +0300
Message-Id: <CT88CCOYEZ1Z.12FUJMD15YPR@suppilovahvero>
From:   "Jarkko Sakkinen" <jarkko@kernel.org>
To:     "Dan Carpenter" <dan.carpenter@linaro.org>
Cc:     "Bagas Sanjaya" <bagasdotme@gmail.com>,
        "Franziska Naepelt" <franziska.naepelt@googlemail.com>,
        <keyrings@vger.kernel.org>, <dhowells@redhat.com>,
        <dwmw2@infradead.org>, <linux-kernel@vger.kernel.org>,
        "Franziska Naepelt" <franziska.naepelt@gmail.com>,
        "kernel test robot" <lkp@intel.com>,
        "Linux SPDX Licenses" <linux-spdx@vger.kernel.org>,
        "Linux Kernel Janitors" <kernel-janitors@vger.kernel.org>
Subject: Re: [PATCH v2] certs/extract-cert: Fix checkpatch issues
References: <20230601190508.56610-1-franziska.naepelt@gmail.com>
 <20230602085902.59006-1-franziska.naepelt@gmail.com>
 <ZH8mhIrjyBvTF4oZ@debian.me>
 <e39efb7f-5d8f-4433-83b3-8eea8a6c0486@kadam.mountain>
 <CT5NH4XXIYQF.5XXJE6JA5FZP@suppilovahvero>
 <c464c4ee-038c-47bf-857a-b11a89680e82@kadam.mountain>
In-Reply-To: <c464c4ee-038c-47bf-857a-b11a89680e82@kadam.mountain>
Precedence: bulk

On Fri Jun 9, 2023 at 5:01 PM EEST, Dan Carpenter wrote:
> On Tue, Jun 06, 2023 at 05:51:09PM +0300, Jarkko Sakkinen wrote:
> > It's not a "punishment". It's more like that I really have to take the
> > time to read the prose...
>
> The thing about imperative tense is that it was used as a punishment on
> me once five years ago.  I wrote a quite bad commit message and a senior
> maintainer told me to re-write it properly and I realized that it was
> true.  My commit message was bad.  So I wrote a proper commit message.
> And then he yelled at me, "Can't you follow simple directions and write
> it in imperative tense like the documentation says?  Are you a
> shithead?"

Wow :-O I'm totally against name calling or any sort of shittiness like
that, and all for co-operation. Just told my personal thoughts on the
matter. I'm sorry that this happened to you.

>
> So then I swore I would never talk to him again or to anyone who
> enforced the imperative tense rule.  That has only happened once in the
> intervening years.  I told the maintainer, "Fine.  Re-write the commit
> message however you like and give me Reported-by credit."  This was a
> cheeky response and it made the maintainer enraged.  I guess he thought
> that my boss would force me to fix the bug or something?  I felt bad for
> the Intel developer who had to fix my bug instead because I knew that
> the maintainer was going to be super angry if he gave me reported-by
> credit so I had put him in a bind.  I almost re-wrote the commit message
> so that he wouldn't have to deal with that.  Maybe this is how mothers
> feel when they try to take abuse from an angry husband instead of
> letting their kids suffer.  But I am a bad mother and I left.
>
> My boss would never have forced me to deal with that.  When he left for
> a different company he said, "Dan, I'm transitioning and XXX is taking
> over me and I have told him all your weirdness so he is prepared."  And
> it was a huge comfort to me because I know what my weakness are.
>
> You people on this thread all seem super nice.  And you're right that we
> should always try to be improve every aspect of our craft.
>
> When Jarkko talked about people who write too long commit messages, I
> thought about one developer in particular who writes too long commit
> messages.  He writes in imperative tense.  He takes everything so
> seriously and he's never seen a rule without following it.  His patches
> are always right.  People have told him that his commit messages are bad
> and too long and those people are right.  But they need to shut up.  The
> good things that he does and the bad things that he does are all part of
> the same package.  He can't change and I don't want him to feel anything
> but welcome.
>
> It's hard to be a good kernel developer without being at least slightly
> obsessive.  Both developers and maintainers are that way.  And I deal
> with a lot of people and accomodating maintainers you disagree with is
> part of the job.
>
> So long as everyone is kind to each other.  That's the main thing.

I 110% agree with this. I even bookmarked this response :-)

> regards,
> dan carpenter

BR, Jarkko