diff options
author | ilevy@chromium.org <ilevy@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-01-07 18:33:23 +0000 |
---|---|---|
committer | ilevy@chromium.org <ilevy@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-01-07 18:33:23 +0000 |
commit | 5861efbd984d3700089510487c852a4d45c603e2 (patch) | |
tree | a1dca0b005bf662e2ab6c50860767b5e505799fa /PRESUBMIT.py | |
parent | f1cabab0ac6416da95665383a17a5f525a9d996a (diff) | |
download | chromium_src-5861efbd984d3700089510487c852a4d45c603e2.zip chromium_src-5861efbd984d3700089510487c852a4d45c603e2.tar.gz chromium_src-5861efbd984d3700089510487c852a4d45c603e2.tar.bz2 |
Silence Valid authors debug msg in presubmit
Silence the valid authors debug message, unless the check fails.
The message takes a lot of space because the author file is big, and
is not especially helpful (just lists parse results from the authors
file).
NOTRY=True
Review URL: https://chromiumcodereview.appspot.com/11782010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@175360 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'PRESUBMIT.py')
-rw-r--r-- | PRESUBMIT.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/PRESUBMIT.py b/PRESUBMIT.py index 34ab6be..17e3091 100644 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -774,9 +774,8 @@ def _CheckAuthorizedAuthor(input_api, output_api): input_api.re.match(r'[^#]+\s+\<(.+?)\>\s*$', line) for line in open(authors_path)) valid_authors = [item.group(1).lower() for item in valid_authors if item] - if input_api.verbose: - print 'Valid authors are %s' % ', '.join(valid_authors) if not any(fnmatch.fnmatch(author.lower(), valid) for valid in valid_authors): + input_api.logging.info('Valid authors are %s', ', '.join(valid_authors)) return [output_api.PresubmitPromptWarning( ('%s is not in AUTHORS file. If you are a new contributor, please visit' '\n' |