Re: [PATCH] checkpatch: exclude gerrit's Change-Id line from commit description

From: Joe Perches
Date: Tue Dec 12 2023 - 14:02:46 EST


On Tue, 2023-12-12 at 10:53 +0100, Antonio Borneo wrote:
> On Mon, Apr 10, 2023, 12:03 Antonio Borneo <borneo.antonio@xxxxxxxxx> 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.