[ovs-dev] [PATCH] checkpatch: Add argument to skip gerrit change id check
Simon Horman
simon.horman at netronome.com
Tue Jun 16 14:05:01 UTC 2020
On Tue, Jun 16, 2020 at 04:14:23PM +0300, Roi Dayan wrote:
>
>
> On 2020-06-14 7:03 PM, Roi Dayan wrote:
> > This arg can be used internally by groups using gerrit for code reviews.
> >
> > Signed-off-by: Roi Dayan <roid at mellanox.com>
> > ---
> > utilities/checkpatch.py | 9 +++++++--
> > 1 file changed, 7 insertions(+), 2 deletions(-)
> >
> > diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py
> > index fc9e20bf1b5f..78c8c9ce49c7 100755
> > --- a/utilities/checkpatch.py
> > +++ b/utilities/checkpatch.py
> > @@ -182,6 +182,7 @@ __regex_if_macros = re.compile(r'^ +(%s) \([\S]([\s\S]+[\S])*\) { +\\' %
> >
> > skip_leading_whitespace_check = False
> > skip_trailing_whitespace_check = False
> > +skip_gerrit_change_id_check = False
> > skip_block_whitespace_check = False
> > skip_signoff_check = False
> >
> > @@ -814,7 +815,7 @@ def ovs_checkpatch_parse(text, filename, author=None, committer=None):
> > elif is_co_author.match(line):
> > m = is_co_author.match(line)
> > co_authors.append(m.group(2))
> > - elif is_gerrit_change_id.match(line):
> > + elif is_gerrit_change_id.match(line) and not skip_gerrit_change_id_check:
> > print_error(
> > "Remove Gerrit Change-Id's before submitting upstream.")
> > print("%d: %s\n" % (lineno, line))
> > @@ -885,7 +886,8 @@ Check options:
> > -s|--skip-signoff-lines Tolerate missing Signed-off-by line
> > -S|--spellcheck Check C comments and commit-message for possible
> > spelling mistakes
> > --t|--skip-trailing-whitespace Skips the trailing whitespace test"""
> > +-t|--skip-trailing-whitespace Skips the trailing whitespace test
> > + --skip-gerrit-change-id Skips the gerrit change id test"""
> > % sys.argv[0])
> >
> >
> > @@ -942,6 +944,7 @@ if __name__ == '__main__':
> > "skip-leading-whitespace",
> > "skip-signoff-lines",
> > "skip-trailing-whitespace",
> > + "skip-gerrit-change-id",
> > "spellcheck",
> > "quiet"])
> > except:
> > @@ -960,6 +963,8 @@ if __name__ == '__main__':
> > skip_signoff_check = True
> > elif o in ("-t", "--skip-trailing-whitespace"):
> > skip_trailing_whitespace_check = True
> > + elif o in ("--skip-gerrit-change-id"):
> > + skip_gerrit_change_id_check = True
> > elif o in ("-f", "--check-file"):
> > checking_file = True
> > elif o in ("-S", "--spellcheck"):
> >
>
> Hi,
>
> Any comments on this?
Reviewed-by: Simon Horman <simon.horman at netronome.com>
More information about the dev
mailing list