Update IPC security PRESUBMIT check message to be clearer.
I hope.
Bug: none
Change-Id: I72a9c4847700e3f2cac5644c16e6d19149c86aae
Reviewed-on: https://chromium-review.googlesource.com/535056
Reviewed-by: Robert Sesek <[email protected]>
Reviewed-by: Emily Stark <[email protected]>
Reviewed-by: Dirk Pranke <[email protected]>
Commit-Queue: Daniel Cheng <[email protected]>
Cr-Commit-Position: refs/heads/master@{#479437}diff --git a/PRESUBMIT.py b/PRESUBMIT.py
index b076e42b..55131cdd 100644
--- a/PRESUBMIT.py
+++ b/PRESUBMIT.py
@@ -1684,7 +1684,7 @@
files.extend([' %s' % f.LocalPath() for f in entry['files']])
if missing_lines:
errors.append(
- '%s is missing the following lines:\n\n%s\n\nfor changed files:\n%s' %
+ '%s needs the following lines added:\n\n%s\n\nfor files:\n%s' %
(owners_file, '\n'.join(missing_lines), '\n'.join(files)))
results = []
@@ -1694,7 +1694,8 @@
else:
output = output_api.PresubmitPromptWarning
results.append(output(
- 'Found changes to IPC files without a security OWNER!',
+ 'Found OWNERS files that need to be updated for IPC security ' +
+ 'review coverage.\nPlease update the OWNERS files below:',
long_text='\n\n'.join(errors)))
return results
