commit 4844f8964c596dbdb93507b20511d5cb427c7fad
author Daisuke Nojiri <dnojiri@chromium.org> Thu Oct 08 09:54:33 2015 -0700
committer chrome-bot <chrome-bot@chromium.org> Thu Oct 08 13:51:21 2015 -0700
tree 2bee56bd5045de4f328a4ff90d1dc65f96316215
parent 28d48d69aa6bd439b647597bac15226e237370bd

pre-upload: allow change-id

This change makes checkpatch ignore change-id check, which prevents CLs from
being uploaded to Gerrit.

BUG=none
BRANCH=master
TEST=repo upload this CL without the error

Change-Id: I02cc045b73e3aad98e17c2b95c8e6d267a456dbe
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/304628
Reviewed-by: Filipe Brandenburger <filbranden@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

pre-upload.py

diff --git a/pre-upload.py b/pre-upload.py
index 849dadc..11b51fe 100755
--- a/pre-upload.py
+++ b/pre-upload.py
@@ -1098,6 +1098,7 @@
   # information on that file in our source trees, so let's suppress the
   # warning.
   options.append('--ignore=FILE_PATH_CHANGES')
+  options.append('--ignore=GERRIT_CHANGE_ID')
   cmd = ['%s/checkpatch.pl' % hooks_dir] + options + ['-']
   cmd_result = cros_build_lib.RunCommand(cmd=cmd,
                                          print_cmd=False,