From: "PierreGondois" <pierre.gondois@arm.com>
To: devel@edk2.groups.io
Cc: Rebecca Cran <rebecca@bsdio.com>,
Liming Gao <gaoliming@byosoft.com.cn>,
Bob Feng <bob.c.feng@intel.com>,
Yuwei Chen <yuwei.chen@intel.com>,
Sami Mujawar <sami.mujawar@arm.com>,
Pedro Falcato <pedro.falcato@gmail.com>,
Ray Ni <ray.ni@intel.com>
Subject: Re: [edk2-devel] [PATCH v2 1/1] BaseTools/Scripts/PatchCheck.py: Check for Change-id
Date: Wed, 31 Jan 2024 16:07:20 +0100 [thread overview]
Message-ID: <d50dadd0-85a3-4dbe-b032-ac641f0d5f35@arm.com> (raw)
In-Reply-To: <17A2CBB8731D31AE.28867@groups.io>
Hello Rebecca, Liming,
Just a ping in case this was forgotten,
Regards,
Pierre
On 12/21/23 09:32, PierreGondois via groups.io wrote:
> Code review tools like gerrit might use a 'Change-id' tag to track
> the evolution of patches. This tag should be removed before
> submitting a patch to the mailing-list.
> It has been observed that contributors sometimes forget to remove
> this tag. Add a check in PatchCheck.py to automate this.
>
> Also add a '--ignore-change-id' command line parameter to ignore
> the above check.
>
> Signed-off-by: Pierre Gondois <Pierre.Gondois@arm.com>
> Acked-by: Pedro Falcato <pedro.falcato@gmail.com>
> Reviewed-by: Yuwei Chen <yuwei.chen@intel.com>
> Reviewed-by: Rebecca Cran <rebecca@bsdio.com>
> ---
>
> Notes:
> v2:
> - Fix capitalization in helper: Change-id -> Change-Id
>
> BaseTools/Scripts/PatchCheck.py | 18 +++++++++++++++++-
> 1 file changed, 17 insertions(+), 1 deletion(-)
>
> diff --git a/BaseTools/Scripts/PatchCheck.py b/BaseTools/Scripts/PatchCheck.py
> index 7f372d40b570..1675dcbd7321 100755
> --- a/BaseTools/Scripts/PatchCheck.py
> +++ b/BaseTools/Scripts/PatchCheck.py
> @@ -3,7 +3,7 @@
> #
> # Copyright (c) 2015 - 2021, Intel Corporation. All rights reserved.<BR>
> # Copyright (C) 2020, Red Hat, Inc.<BR>
> -# Copyright (c) 2020, ARM Ltd. All rights reserved.<BR>
> +# Copyright (c) 2020 - 2023, Arm Limited. All rights reserved.<BR>
> #
> # SPDX-License-Identifier: BSD-2-Clause-Patent
> #
> @@ -26,6 +26,9 @@ class Verbose:
> SILENT, ONELINE, NORMAL = range(3)
> level = NORMAL
>
> +class PatchCheckConf:
> + ignore_change_id = False
> +
> class EmailAddressCheck:
> """Checks an email address."""
>
> @@ -111,6 +114,8 @@ class CommitMessageCheck:
> self.check_signed_off_by()
> self.check_misc_signatures()
> self.check_overall_format()
> + if not PatchCheckConf.ignore_change_id:
> + self.check_change_id_format()
> self.report_message_result()
>
> url = 'https://github.com/tianocore/tianocore.github.io/wiki/Commit-Message-Format'
> @@ -307,6 +312,12 @@ class CommitMessageCheck:
> break
> last_sig_line = line.strip()
>
> + def check_change_id_format(self):
> + cid='Change-Id:'
> + if self.msg.find(cid) != -1:
> + self.error('\"%s\" found in commit message:' % cid)
> + return
> +
> (START, PRE_PATCH, PATCH) = range(3)
>
> class GitDiffCheck:
> @@ -780,11 +791,16 @@ class PatchCheckApp:
> group.add_argument("--silent",
> action="store_true",
> help="Print nothing")
> + group.add_argument("--ignore-change-id",
> + action="store_true",
> + help="Ignore the presence of 'Change-Id:' tags in commit message")
> self.args = parser.parse_args()
> if self.args.oneline:
> Verbose.level = Verbose.ONELINE
> if self.args.silent:
> Verbose.level = Verbose.SILENT
> + if self.args.ignore_change_id:
> + PatchCheckConf.ignore_change_id = True
>
> if __name__ == "__main__":
> sys.exit(PatchCheckApp().retval)
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#114904): https://edk2.groups.io/g/devel/message/114904
Mute This Topic: https://groups.io/mt/104076488/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
next parent reply other threads:[~2024-01-31 15:07 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <17A2CBB8731D31AE.28867@groups.io>
2024-01-31 15:07 ` PierreGondois [this message]
2024-02-04 14:50 ` [edk2-devel] [PATCH v2 1/1] BaseTools/Scripts/PatchCheck.py: Check for Change-id Rebecca Cran
2023-12-21 8:32 PierreGondois
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d50dadd0-85a3-4dbe-b032-ac641f0d5f35@arm.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox