linux-kernel - Re: [PATCH] checkpatch: relax check for revert commit

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <e5d59c005cdf0183b866fd24a012ccc35118f121.camel@perches.com>
Date:   Mon, 09 Apr 2018 03:50:36 -0700
From:   Joe Perches <joe@...ches.com>
To:     Jia He <hejianet@...il.com>, Andy Whitcroft <apw@...onical.com>
Cc:     linux-kernel@...r.kernel.org, Jia He <jia.he@...-semitech.com>
Subject: Re: [PATCH] checkpatch: relax check for revert commit

On Mon, 2018-04-09 at 14:37 +0800, Jia He wrote:
> 
> On 4/9/2018 1:07 AM, Joe Perches Wrote:
> > On Sun, 2018-04-08 at 10:12 +0800, Jia He wrote:
> > > Hi Andy & Joe
> > > 
> > > Although it is minor, it is a real bug, I thought.
> > > 
> > > Is there any comment? Thank you
> > 
> > If you really want to do something useful here,
> > quote the quote characters and compare for that.
> > 
> > 
> 
> Hi Joe, thanks for the comments
> I am not sure I understand you comments here.

Change the tests to allow quote characters instead
of ignoring them.

To do this matching, any internal  quote character in
the actual commit string and the commit string from the
patch must be perl quoted with \Q<string>\E so that the
expected title can be matched properly.