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;
Date:   Mon, 5 Sep 2022 12:49:00 +0200
From:   Niklas =?iso-8859-1?Q?S=F6derlund?= 
        <niklas.soderlund@corigine.com>
To:     Joe Perches <joe@perches.com>
Cc:     Dwaipayan Ray <dwaipayanray1@gmail.com>,
        Lukas Bulwahn <lukas.bulwahn@gmail.com>,
        Jonathan Corbet <corbet@lwn.net>,
        Andy Whitcroft <apw@canonical.com>, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org, oss-drivers@corigine.com,
        Simon Horman <simon.horman@corigine.com>,
        Louis Peens <louis.peens@corigine.com>
Subject: Re: [PATCH] checkpatch: warn for non-standard fixes tag style
Message-ID: <YxXUHPWtyrt6kNI1@sleipner.dyn.berto.se>
References: <20220829155358.2546732-1-niklas.soderlund@corigine.com>
 <dc45a7021bb765ea34c5b9228454f255764c7bc9.camel@perches.com>
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <dc45a7021bb765ea34c5b9228454f255764c7bc9.camel@perches.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?iso-8859-1?Q?/yQmA7EWAsPD4XX9nLSDKo545KaS5XKCyWLKgP+vo+DLofaUdrUX9CIjjd?=
 =?iso-8859-1?Q?Z0RY+fkpnl1pu2PrRNgLNoQHnH8p+h0w/PSoOK3xfX2kntXMg25Dte1iGX?=
 =?iso-8859-1?Q?QpncVORG9BRg2xlIBeY+qiogOpbZ1yZT0MsFDJVctk3zLnJr6Sgs7Kj1X+?=
 =?iso-8859-1?Q?gbhOdxAJ+Wyexn5zb940ZUJZMJj/2nVcE7VGfaxbHzhVRdwTTytjgQaFHd?=
 =?iso-8859-1?Q?pNkePfNkgtT9nqXKbCt/SVrim6Seu7vjFGfp5Gfeye48xTXN1wtPcC9Ghl?=
 =?iso-8859-1?Q?hwvaV/qeSPsIdoN9XikyN0ONWm5okxc3O7uFwDi4Q2X6xRsspg0+NVyvxa?=
 =?iso-8859-1?Q?vxLaZrOW4C7WL9M7kqHAgEQC1PWv1yxuxeTW8HimYdiedUPVGSLYeEUWAe?=
 =?iso-8859-1?Q?yAyQk0U9hG3OSb+QELhvoAckhdYQexMh22fQ6b+Iu4seW98ERTRNJ+8FMe?=
 =?iso-8859-1?Q?4n/X8cGUqYQNn/M6kEtOC5rnrGd3baAWvE7cDqL0kTPXKlGYYoiZYLXFSX?=
 =?iso-8859-1?Q?HSwhk2JXBR+8wdwe7Ltrodw7wUHsJQJ3diT6RHOgnXMqy5hVy8/etLSWFu?=
 =?iso-8859-1?Q?GoVmMTZd9vijmzaodI6ptM5dQXFHt0xVlkpQIcri1O+24l5Vw+GE+cxcqg?=
 =?iso-8859-1?Q?9fAjB76AZVxAWL5LBHzyfr5RZRMGwvMZRbu4FpvSpkLU8bGSCp2wB1MaXR?=
 =?iso-8859-1?Q?9xLSMGy00FPm+p9YOcB74XWoA448BwJt7kyf7iRssk+ieGwAiPC4ChliQJ?=
 =?iso-8859-1?Q?llbkTUPvFyz1GsZKERmkn/sNplm1rFSR9FrawQ1Q60b0PCXocTIrrCRt/B?=
 =?iso-8859-1?Q?xA9Ci8U8PpdN7z8+TcAYQciNcJII92s6zX+Eei0WzXiOqA3UBIKQ8fODuQ?=
 =?iso-8859-1?Q?LmKwd2ay7fQpZ7ZGtoI+EAwNbb1AUh0KuZhEOwClyo1KbWouojqfOMmrFL?=
 =?iso-8859-1?Q?58gFCQs5fYQh9jBaqTw+Kvmh1jN+jorUnRsGMEaVkpfsz0IRNHN1e9FIfR?=
 =?iso-8859-1?Q?pmM2g64HO25vt10utiWtUTc7L4pb61/QgLBtG0+3XkheSypi+AECCGS2LN?=
 =?iso-8859-1?Q?seAc4sT3rQabFzYO8IjX0zM5Q0SliApErPur4wh4tWhxB9ZG3rplC61ylL?=
 =?iso-8859-1?Q?OCRYCqUpGdPGkSZUebedzD6wz6oP4LNOZ+yCV33p9Xbz0Vy4KB1HBWxiLe?=
 =?iso-8859-1?Q?L61DmQUUbjFYxrdwOaLocClP3VTeTZ+nKuxwHNTlhtpel5Zcfx2+XHEE5D?=
 =?iso-8859-1?Q?ki8oFM6es4Db9Q1KDtlYCjnjeZG30Exo/bpNsE0+ex34WK1DXAmQG1X/Gy?=
 =?iso-8859-1?Q?aCebM0nFxAQ8RoGxVzZ0rQwF1GdHx5KfHbluODaf47c1LJUOXFu5yUoYN7?=
 =?iso-8859-1?Q?KSV/P1XM7KbwSU8aIKgWar3Tq2GfDEu/BanJadVjTBDgPTzMHOQsDtmnbr?=
 =?iso-8859-1?Q?Fdxacn+b+NnSniiCBXcQUFhdT+AeOPvrrFyLG8w4kVIhgJ+DIYWgRJ+ZMu?=
 =?iso-8859-1?Q?9+CvZDIbCxJ2tqd0pgm+ZSbqwTH0lfAy5ldpyvmn2PKsrs70bk6q7Yd+Br?=
 =?iso-8859-1?Q?aCVJs3fII6p8mpKn9MOFx9/iDM1iWS0m+Bqlm4xkPQUMOwalL1QWZ39TuB?=
 =?iso-8859-1?Q?i8lRA6MZVsDFUDNwvy/Pjd7fLTDB0Ffs0vDIX8EsPi7PnRO18tnrkuMw?=
 =?iso-8859-1?Q?=3D=3D?=
X-OriginatorOrg: corigine.com
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW3PR13MB4108
X-Spam-Status: No, score=-1.7 required=5.0 tests=BAYES_00,DKIM_INVALID,
        DKIM_SIGNED,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,
        SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Joe,

Thanks for your feedback.

On 2022-08-29 23:06:43 -0400, Joe Perches wrote:
> > +			if ($line =~ 
> > /(fixes:)\s+([0-9a-f]{5,})\s+($balanced_parens)/i) {
> 
> Maybe use fixes:? so the colon is not required in poorly formed uses

I tried that but I think it brings more problems then it is worth. With 
that change the check would run for each line of the commit message that 
begins with the string 'fixes', not just in the tags section of the 
message. So it would warn for the commit message,

    The work on foo and bar introduced a bug that can be
    fixed by doing baz.

I think it's for this reason other checks for tags include the ':'.

> 
> > +				my $tag = $1;
> > +				$orig_commit = $2;
> > +				$title = $3;
> > +
> > +				$tag_case = 0 if $tag eq "Fixes:";
> > +				$tag_space = 0 if ($line =~ /^fixes: [0-9a-f]{5,} ($balanced_parens)/i);
> 
> fixes:? here too
> 
> Pity there's no simple way to consolidate this git commit test block
> with the existing git id commit block.

I agree.

-- 
Kind Regards,
Niklas S?derlund