Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) client-ip=2620:137:e000::3:4;
Message-ID: <9f361dcd86507486fb2a1a1540957f4ad4919af0.camel@perches.com>
Subject: Re: [PATCH] checkpatch: exclude gerrit's Change-Id line from commit
 description
From:   Joe Perches <joe@perches.com>
To:     Antonio Borneo <borneo.antonio@gmail.com>,
        Andy Whitcroft <apw@canonical.com>,
        Dwaipayan Ray <dwaipayanray1@gmail.com>,
        Lukas Bulwahn <lukas.bulwahn@gmail.com>
Cc:     linux-kernel@vger.kernel.org
Date:   Tue, 12 Dec 2023 11:02:39 -0800
In-Reply-To: <CAAj6DX0SpA0jFtKT0Pu02BuXsF5_=UG0N4H8iyB9Rf-gqK50dA@mail.gmail.com>
References: <20230410100255.16755-1-borneo.antonio@gmail.com>
         <CAAj6DX0SpA0jFtKT0Pu02BuXsF5_=UG0N4H8iyB9Rf-gqK50dA@mail.gmail.com>
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 8BIT
User-Agent: Evolution 3.48.4 (3.48.4-1.fc38) 
MIME-Version: 1.0
Precedence: bulk

On Tue, 2023-12-12 at 10:53 +0100, Antonio Borneo wrote:
> On Mon, Apr 10, 2023, 12:03 Antonio Borneo <borneo.antonio@gmail.com> wrote:
> > Checkpatch rejects patches that have empty commit description and
> > logs them with:
> >         WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one
> > But if the patch has a gerrit's Change-Id line placed before the
> > line Signed-off-by, then checkpatch considers the Change-Id line
> > as a valid commit description text.
> > 
> > Use the Change-Id tag as a marker of the end of the commit message,
> > thus not counting its line as part of the commit description.
> > 
> > This patch is not relevant for the Linux kernel development process

So don't expect me to pick it up for the linux kernel.

You are of course welcome to branch and include it in
your process.