[email protected] | 1f7b417 | 2010-01-28 01:17:34 | [diff] [blame] | 1 | # Copyright (c) 2010 The Chromium Authors. All rights reserved. |
[email protected] | ca8d1984 | 2009-02-19 16:33:12 | [diff] [blame] | 2 | # Use of this source code is governed by a BSD-style license that can be |
| 3 | # found in the LICENSE file. |
| 4 | |
| 5 | """Top-level presubmit script for Chromium. |
| 6 | |
[email protected] | f129379 | 2009-07-31 18:09:56 | [diff] [blame] | 7 | See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts |
[email protected] | 50d7d721e | 2009-11-15 17:56:18 | [diff] [blame] | 8 | for more details about the presubmit API built into gcl. |
[email protected] | ca8d1984 | 2009-02-19 16:33:12 | [diff] [blame] | 9 | """ |
| 10 | |
[email protected] | 379e7dd | 2010-01-28 17:39:21 | [diff] [blame] | 11 | _EXCLUDED_PATHS = ( |
[email protected] | 3347870 | 2009-03-05 14:03:14 | [diff] [blame] | 12 | r"breakpad[\\\/].*", |
[email protected] | abc6a51 | 2010-08-04 01:25:42 | [diff] [blame] | 13 | r"net/tools/spdyshark/[\\\/].*", |
[email protected] | 3347870 | 2009-03-05 14:03:14 | [diff] [blame] | 14 | r"skia[\\\/].*", |
[email protected] | 3347870 | 2009-03-05 14:03:14 | [diff] [blame] | 15 | r"v8[\\\/].*", |
[email protected] | 430641764 | 2009-06-11 00:33:40 | [diff] [blame] | 16 | ) |
[email protected] | ca8d1984 | 2009-02-19 16:33:12 | [diff] [blame] | 17 | |
[email protected] | 379e7dd | 2010-01-28 17:39:21 | [diff] [blame] | 18 | _TEXT_FILES = ( |
| 19 | r".*\.txt", |
| 20 | r".*\.json", |
| 21 | ) |
[email protected] | ca8d1984 | 2009-02-19 16:33:12 | [diff] [blame] | 22 | |
[email protected] | 1f7b417 | 2010-01-28 01:17:34 | [diff] [blame] | 23 | _LICENSE_HEADER = ( |
[email protected] | 38fdd9d | 2010-01-28 18:33:22 | [diff] [blame] | 24 | r".*? Copyright \(c\) 20[0-9\-]{2,7} The Chromium Authors\. All rights " |
| 25 | r"reserved\." "\n" |
[email protected] | 1f7b417 | 2010-01-28 01:17:34 | [diff] [blame] | 26 | r".*? Use of this source code is governed by a BSD-style license that can " |
| 27 | "be\n" |
| 28 | r".*? found in the LICENSE file\." |
| 29 | "\n" |
| 30 | ) |
| 31 | |
[email protected] | 650c908 | 2010-12-14 14:33:44 | [diff] [blame] | 32 | def _CheckSingletonInHeaders(input_api, output_api, source_file_filter): |
| 33 | """Checks to make sure no header files have |Singleton<|.""" |
| 34 | pattern = input_api.re.compile(r'Singleton<') |
| 35 | files = [] |
| 36 | for f in input_api.AffectedSourceFiles(source_file_filter): |
| 37 | if (f.LocalPath().endswith('.h') or f.LocalPath().endswith('.hxx') or |
| 38 | f.LocalPath().endswith('.hpp') or f.LocalPath().endswith('.inl')): |
| 39 | contents = input_api.ReadFile(f) |
| 40 | if pattern.search(contents): |
| 41 | files.append(f) |
| 42 | |
| 43 | if len(files): |
| 44 | return [ output_api.PresubmitError( |
| 45 | 'Found Singleton<T> in the following header files.\n' + |
| 46 | 'Please move them to an appropriate source file so that the ' + |
| 47 | 'template gets instantiated in a single compilation unit.', |
| 48 | files) ] |
| 49 | return [] |
[email protected] | 1f7b417 | 2010-01-28 01:17:34 | [diff] [blame] | 50 | |
[email protected] | 542d1ad2 | 2010-08-04 17:50:55 | [diff] [blame] | 51 | def _CheckConstNSObject(input_api, output_api, source_file_filter): |
| 52 | """Checks to make sure no objective-c files have |const NSSomeClass*|.""" |
| 53 | pattern = input_api.re.compile(r'const\s+NS\w*\s*\*') |
| 54 | files = [] |
| 55 | for f in input_api.AffectedSourceFiles(source_file_filter): |
| 56 | if f.LocalPath().endswith('.h') or f.LocalPath().endswith('.mm'): |
| 57 | contents = input_api.ReadFile(f) |
| 58 | if pattern.search(contents): |
| 59 | files.append(f) |
| 60 | |
| 61 | if len(files): |
| 62 | if input_api.is_committing: |
| 63 | res_type = output_api.PresubmitPromptWarning |
| 64 | else: |
| 65 | res_type = output_api.PresubmitNotifyResult |
| 66 | return [ res_type('|const NSClass*| is wrong, see ' + |
| 67 | 'http://dev.chromium.org/developers/clang-mac', |
| 68 | files) ] |
| 69 | return [] |
| 70 | |
| 71 | |
[email protected] | 1f7b417 | 2010-01-28 01:17:34 | [diff] [blame] | 72 | def _CommonChecks(input_api, output_api): |
[email protected] | fe5f57c5 | 2009-06-05 14:25:54 | [diff] [blame] | 73 | results = [] |
[email protected] | 430641764 | 2009-06-11 00:33:40 | [diff] [blame] | 74 | # What does this code do? |
| 75 | # It loads the default black list (e.g. third_party, experimental, etc) and |
| 76 | # add our black list (breakpad, skia and v8 are still not following |
| 77 | # google style and are not really living this repository). |
| 78 | # See presubmit_support.py InputApi.FilterSourceFile for the (simple) usage. |
[email protected] | 379e7dd | 2010-01-28 17:39:21 | [diff] [blame] | 79 | black_list = input_api.DEFAULT_BLACK_LIST + _EXCLUDED_PATHS |
| 80 | white_list = input_api.DEFAULT_WHITE_LIST + _TEXT_FILES |
[email protected] | 430641764 | 2009-06-11 00:33:40 | [diff] [blame] | 81 | sources = lambda x: input_api.FilterSourceFile(x, black_list=black_list) |
[email protected] | 379e7dd | 2010-01-28 17:39:21 | [diff] [blame] | 82 | text_files = lambda x: input_api.FilterSourceFile(x, black_list=black_list, |
| 83 | white_list=white_list) |
[email protected] | 430641764 | 2009-06-11 00:33:40 | [diff] [blame] | 84 | results.extend(input_api.canned_checks.CheckLongLines( |
[email protected] | ea141386 | 2010-05-05 23:24:53 | [diff] [blame] | 85 | input_api, output_api, source_file_filter=sources)) |
[email protected] | 430641764 | 2009-06-11 00:33:40 | [diff] [blame] | 86 | results.extend(input_api.canned_checks.CheckChangeHasNoTabs( |
[email protected] | ea141386 | 2010-05-05 23:24:53 | [diff] [blame] | 87 | input_api, output_api, source_file_filter=sources)) |
[email protected] | 430641764 | 2009-06-11 00:33:40 | [diff] [blame] | 88 | results.extend(input_api.canned_checks.CheckChangeHasNoStrayWhitespace( |
[email protected] | ea141386 | 2010-05-05 23:24:53 | [diff] [blame] | 89 | input_api, output_api, source_file_filter=sources)) |
[email protected] | 430641764 | 2009-06-11 00:33:40 | [diff] [blame] | 90 | results.extend(input_api.canned_checks.CheckChangeHasBugField( |
| 91 | input_api, output_api)) |
| 92 | results.extend(input_api.canned_checks.CheckChangeHasTestField( |
| 93 | input_api, output_api)) |
| 94 | results.extend(input_api.canned_checks.CheckChangeSvnEolStyle( |
[email protected] | ea141386 | 2010-05-05 23:24:53 | [diff] [blame] | 95 | input_api, output_api, source_file_filter=text_files)) |
[email protected] | 40cdf8b3 | 2009-06-26 23:00:37 | [diff] [blame] | 96 | results.extend(input_api.canned_checks.CheckSvnForCommonMimeTypes( |
| 97 | input_api, output_api)) |
[email protected] | 1f7b417 | 2010-01-28 01:17:34 | [diff] [blame] | 98 | results.extend(input_api.canned_checks.CheckLicense( |
[email protected] | ea141386 | 2010-05-05 23:24:53 | [diff] [blame] | 99 | input_api, output_api, _LICENSE_HEADER, source_file_filter=sources)) |
[email protected] | 542d1ad2 | 2010-08-04 17:50:55 | [diff] [blame] | 100 | results.extend(_CheckConstNSObject( |
| 101 | input_api, output_api, source_file_filter=sources)) |
[email protected] | 650c908 | 2010-12-14 14:33:44 | [diff] [blame] | 102 | results.extend(_CheckSingletonInHeaders( |
| 103 | input_api, output_api, source_file_filter=sources)) |
[email protected] | 1f7b417 | 2010-01-28 01:17:34 | [diff] [blame] | 104 | return results |
| 105 | |
| 106 | |
| 107 | def CheckChangeOnUpload(input_api, output_api): |
| 108 | results = [] |
| 109 | results.extend(_CommonChecks(input_api, output_api)) |
[email protected] | fe5f57c5 | 2009-06-05 14:25:54 | [diff] [blame] | 110 | return results |
[email protected] | ca8d1984 | 2009-02-19 16:33:12 | [diff] [blame] | 111 | |
| 112 | |
| 113 | def CheckChangeOnCommit(input_api, output_api): |
[email protected] | fe5f57c5 | 2009-06-05 14:25:54 | [diff] [blame] | 114 | results = [] |
[email protected] | 806e98e | 2010-03-19 17:49:27 | [diff] [blame] | 115 | if not input_api.json: |
| 116 | results.append(output_api.PresubmitNotifyResult( |
| 117 | 'You don\'t have json nor simplejson installed.\n' |
| 118 | ' This is a warning that you will need to upgrade your python ' |
| 119 | 'installation.\n' |
| 120 | ' This is no big deal but you\'ll eventually need to ' |
| 121 | 'upgrade.\n' |
| 122 | ' How? Easy! You can do it right now and shut me off! Just:\n' |
| 123 | ' del depot_tools\\python.bat\n' |
| 124 | ' gclient\n' |
| 125 | ' Thanks for your patience.')) |
[email protected] | 1f7b417 | 2010-01-28 01:17:34 | [diff] [blame] | 126 | results.extend(_CommonChecks(input_api, output_api)) |
[email protected] | dd805fe | 2009-10-01 08:11:51 | [diff] [blame] | 127 | # TODO(thestig) temporarily disabled, doesn't work in third_party/ |
| 128 | #results.extend(input_api.canned_checks.CheckSvnModifiedDirectories( |
| 129 | # input_api, output_api, sources)) |
[email protected] | fe5f57c5 | 2009-06-05 14:25:54 | [diff] [blame] | 130 | # Make sure the tree is 'open'. |
[email protected] | 806e98e | 2010-03-19 17:49:27 | [diff] [blame] | 131 | results.extend(input_api.canned_checks.CheckTreeIsOpen( |
[email protected] | 7f23815 | 2009-08-12 19:00:34 | [diff] [blame] | 132 | input_api, |
| 133 | output_api, |
[email protected] | 4efa4214 | 2010-08-26 01:29:26 | [diff] [blame] | 134 | json_url='http://chromium-status.appspot.com/current?format=json')) |
[email protected] | 806e98e | 2010-03-19 17:49:27 | [diff] [blame] | 135 | results.extend(input_api.canned_checks.CheckRietveldTryJobExecution(input_api, |
| 136 | output_api, 'http://codereview.chromium.org', ('win', 'linux', 'mac'), |
| 137 | '[email protected]')) |
| 138 | |
[email protected] | 7fa42f1 | 2009-11-09 20:54:16 | [diff] [blame] | 139 | # These builders are just too slow. |
| 140 | IGNORED_BUILDERS = [ |
| 141 | 'Chromium XP', |
[email protected] | 7fa42f1 | 2009-11-09 20:54:16 | [diff] [blame] | 142 | 'Chromium Mac', |
[email protected] | 0133392 | 2010-02-02 21:25:02 | [diff] [blame] | 143 | 'Chromium Arm (dbg)', |
[email protected] | 7fa42f1 | 2009-11-09 20:54:16 | [diff] [blame] | 144 | 'Chromium Linux', |
| 145 | 'Chromium Linux x64', |
[email protected] | 7fa42f1 | 2009-11-09 20:54:16 | [diff] [blame] | 146 | ] |
[email protected] | 806e98e | 2010-03-19 17:49:27 | [diff] [blame] | 147 | results.extend(input_api.canned_checks.CheckBuildbotPendingBuilds( |
[email protected] | 7fa42f1 | 2009-11-09 20:54:16 | [diff] [blame] | 148 | input_api, |
| 149 | output_api, |
[email protected] | 806e98e | 2010-03-19 17:49:27 | [diff] [blame] | 150 | 'http://build.chromium.org/buildbot/waterfall/json/builders?filter=1', |
[email protected] | 7fa42f1 | 2009-11-09 20:54:16 | [diff] [blame] | 151 | 6, |
| 152 | IGNORED_BUILDERS)) |
[email protected] | fe5f57c5 | 2009-06-05 14:25:54 | [diff] [blame] | 153 | return results |
[email protected] | ca8d1984 | 2009-02-19 16:33:12 | [diff] [blame] | 154 | |
| 155 | |
[email protected] | 5fa0629 | 2009-09-29 01:55:00 | [diff] [blame] | 156 | def GetPreferredTrySlaves(): |
| 157 | return ['win', 'linux', 'mac'] |