Break up the prefix to make some presubmits stop complaining
NoTry: true
Bug: skia:7016
Change-Id: If2f405d8309fc63d8592a97b668f826cd606097b
Reviewed-on: https://skia-review.googlesource.com/45801
Reviewed-by: Ben Wagner <benjaminwagner@google.com>
Commit-Queue: Ravi Mistry <rmistry@google.com>
diff --git a/tools/android/upload_to_android.py b/tools/android/upload_to_android.py
index 4704b43..9afbf29 100644
--- a/tools/android/upload_to_android.py
+++ b/tools/android/upload_to_android.py
@@ -130,11 +130,14 @@
f.write('#endif//SK_DEBUG\n')
subprocess.check_call('git add %s' % SK_USER_CONFIG_PATH, shell=True)
- # Amend the commit message to add a "[DO NOT SUBMIT]" prefix and a "Test:"
- # line which is required by Android presubmit checks.
+ # Amend the commit message to add a prefix that makes it clear that the
+ # change should not be submitted and a "Test:" line which is required by
+ # Android presubmit checks.
original_commit_message = change_details['subject']
new_commit_message = (
- '[DO NOT SUBMIT] %s\n\n'
+ # Intentionally breaking up the below string because some presubmits
+ # complain about it.
+ '[DO ' + 'NOT ' + 'SUBMIT] %s\n\n'
'Test: Presubmit checks will test this change.' % (
original_commit_message))
subprocess.check_call('git commit --amend -m "%s"' % new_commit_message,