Local Gerrit Change-Ids are not welcome in common repository. Inspired by checkpatch.pl from Linux Kernel.
Signed-off-by: Ilya Maximets <[email protected]> --- utilities/checkpatch.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py index 65d188d..fe266ac 100755 --- a/utilities/checkpatch.py +++ b/utilities/checkpatch.py @@ -320,6 +320,8 @@ def ovs_checkpatch_parse(text, filename): re.I | re.M | re.S) is_co_author = re.compile(r'(\s*(Co-authored-by: )(.*))$', re.I | re.M | re.S) + is_gerrit_change_id = re.compile(r'(\s*(change-id: )(.*))$', + re.I | re.M | re.S) for line in text.split('\n'): if current_file != previous_file: @@ -357,6 +359,10 @@ def ovs_checkpatch_parse(text, filename): elif is_co_author.match(line): m = is_co_author.match(line) co_authors.append(m.group(3)) + elif is_gerrit_change_id.match(line): + print_error( + "Remove Gerrit Change-Id's before submitting upstream.") + print("%d: %s\n" % (lineno, line)) elif parse == 2: newfile = hunks.match(line) if newfile: -- 2.7.4 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
