[email protected] | 1f7b417 | 2010-01-28 01:17:34 | [diff] [blame] | 1 | # Copyright (c) 2010 The Chromium Authors. All rights reserved. |
[email protected] | ca8d198 | 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] | ca8d198 | 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] | ca8d198 | 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] | ca8d198 | 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 | |
| 32 | |
[email protected] | 542d1ad2 | 2010-08-04 17:50:55 | [diff] [blame] | 33 | def _CheckConstNSObject(input_api, output_api, source_file_filter): |
| 34 | """Checks to make sure no objective-c files have |const NSSomeClass*|.""" |
| 35 | pattern = input_api.re.compile(r'const\s+NS\w*\s*\*') |
| 36 | files = [] |
| 37 | for f in input_api.AffectedSourceFiles(source_file_filter): |
| 38 | if f.LocalPath().endswith('.h') or f.LocalPath().endswith('.mm'): |
| 39 | contents = input_api.ReadFile(f) |
| 40 | if pattern.search(contents): |
| 41 | files.append(f) |
| 42 | |
| 43 | if len(files): |
| 44 | if input_api.is_committing: |
| 45 | res_type = output_api.PresubmitPromptWarning |
| 46 | else: |
| 47 | res_type = output_api.PresubmitNotifyResult |
| 48 | return [ res_type('|const NSClass*| is wrong, see ' + |
| 49 | 'http://dev.chromium.org/developers/clang-mac', |
| 50 | files) ] |
| 51 | return [] |
| 52 | |
| 53 | |
[email protected] | 1f7b417 | 2010-01-28 01:17:34 | [diff] [blame] | 54 | def _CommonChecks(input_api, output_api): |
[email protected] | fe5f57c5 | 2009-06-05 14:25:54 | [diff] [blame] | 55 | results = [] |
[email protected] | 430641764 | 2009-06-11 00:33:40 | [diff] [blame] | 56 | # What does this code do? |
| 57 | # It loads the default black list (e.g. third_party, experimental, etc) and |
| 58 | # add our black list (breakpad, skia and v8 are still not following |
| 59 | # google style and are not really living this repository). |
| 60 | # See presubmit_support.py InputApi.FilterSourceFile for the (simple) usage. |
[email protected] | 379e7dd | 2010-01-28 17:39:21 | [diff] [blame] | 61 | black_list = input_api.DEFAULT_BLACK_LIST + _EXCLUDED_PATHS |
| 62 | white_list = input_api.DEFAULT_WHITE_LIST + _TEXT_FILES |
[email protected] | 430641764 | 2009-06-11 00:33:40 | [diff] [blame] | 63 | sources = lambda x: input_api.FilterSourceFile(x, black_list=black_list) |
[email protected] | 379e7dd | 2010-01-28 17:39:21 | [diff] [blame] | 64 | text_files = lambda x: input_api.FilterSourceFile(x, black_list=black_list, |
| 65 | white_list=white_list) |
[email protected] | 430641764 | 2009-06-11 00:33:40 | [diff] [blame] | 66 | results.extend(input_api.canned_checks.CheckLongLines( |
[email protected] | ea141386 | 2010-05-05 23:24:53 | [diff] [blame] | 67 | input_api, output_api, source_file_filter=sources)) |
[email protected] | 430641764 | 2009-06-11 00:33:40 | [diff] [blame] | 68 | results.extend(input_api.canned_checks.CheckChangeHasNoTabs( |
[email protected] | ea141386 | 2010-05-05 23:24:53 | [diff] [blame] | 69 | input_api, output_api, source_file_filter=sources)) |
[email protected] | 430641764 | 2009-06-11 00:33:40 | [diff] [blame] | 70 | results.extend(input_api.canned_checks.CheckChangeHasNoStrayWhitespace( |
[email protected] | ea141386 | 2010-05-05 23:24:53 | [diff] [blame] | 71 | input_api, output_api, source_file_filter=sources)) |
[email protected] | 430641764 | 2009-06-11 00:33:40 | [diff] [blame] | 72 | results.extend(input_api.canned_checks.CheckChangeHasBugField( |
| 73 | input_api, output_api)) |
| 74 | results.extend(input_api.canned_checks.CheckChangeHasTestField( |
| 75 | input_api, output_api)) |
| 76 | results.extend(input_api.canned_checks.CheckChangeSvnEolStyle( |
[email protected] | ea141386 | 2010-05-05 23:24:53 | [diff] [blame] | 77 | input_api, output_api, source_file_filter=text_files)) |
[email protected] | 40cdf8b3 | 2009-06-26 23:00:37 | [diff] [blame] | 78 | results.extend(input_api.canned_checks.CheckSvnForCommonMimeTypes( |
| 79 | input_api, output_api)) |
[email protected] | 1f7b417 | 2010-01-28 01:17:34 | [diff] [blame] | 80 | results.extend(input_api.canned_checks.CheckLicense( |
[email protected] | ea141386 | 2010-05-05 23:24:53 | [diff] [blame] | 81 | input_api, output_api, _LICENSE_HEADER, source_file_filter=sources)) |
[email protected] | 542d1ad2 | 2010-08-04 17:50:55 | [diff] [blame] | 82 | results.extend(_CheckConstNSObject( |
| 83 | input_api, output_api, source_file_filter=sources)) |
[email protected] | 1f7b417 | 2010-01-28 01:17:34 | [diff] [blame] | 84 | return results |
| 85 | |
| 86 | |
| 87 | def CheckChangeOnUpload(input_api, output_api): |
| 88 | results = [] |
| 89 | results.extend(_CommonChecks(input_api, output_api)) |
[email protected] | fe5f57c5 | 2009-06-05 14:25:54 | [diff] [blame] | 90 | return results |
[email protected] | ca8d198 | 2009-02-19 16:33:12 | [diff] [blame] | 91 | |
| 92 | |
| 93 | def CheckChangeOnCommit(input_api, output_api): |
[email protected] | fe5f57c5 | 2009-06-05 14:25:54 | [diff] [blame] | 94 | results = [] |
[email protected] | 806e98e | 2010-03-19 17:49:27 | [diff] [blame] | 95 | if not input_api.json: |
| 96 | results.append(output_api.PresubmitNotifyResult( |
| 97 | 'You don\'t have json nor simplejson installed.\n' |
| 98 | ' This is a warning that you will need to upgrade your python ' |
| 99 | 'installation.\n' |
| 100 | ' This is no big deal but you\'ll eventually need to ' |
| 101 | 'upgrade.\n' |
| 102 | ' How? Easy! You can do it right now and shut me off! Just:\n' |
| 103 | ' del depot_tools\\python.bat\n' |
| 104 | ' gclient\n' |
| 105 | ' Thanks for your patience.')) |
[email protected] | 1f7b417 | 2010-01-28 01:17:34 | [diff] [blame] | 106 | results.extend(_CommonChecks(input_api, output_api)) |
[email protected] | dd805fe | 2009-10-01 08:11:51 | [diff] [blame] | 107 | # TODO(thestig) temporarily disabled, doesn't work in third_party/ |
| 108 | #results.extend(input_api.canned_checks.CheckSvnModifiedDirectories( |
| 109 | # input_api, output_api, sources)) |
[email protected] | fe5f57c5 | 2009-06-05 14:25:54 | [diff] [blame] | 110 | # Make sure the tree is 'open'. |
[email protected] | 806e98e | 2010-03-19 17:49:27 | [diff] [blame] | 111 | results.extend(input_api.canned_checks.CheckTreeIsOpen( |
[email protected] | 7f23815 | 2009-08-12 19:00:34 | [diff] [blame] | 112 | input_api, |
| 113 | output_api, |
[email protected] | 4efa4214 | 2010-08-26 01:29:26 | [diff] [blame^] | 114 | json_url='http://chromium-status.appspot.com/current?format=json')) |
[email protected] | 806e98e | 2010-03-19 17:49:27 | [diff] [blame] | 115 | results.extend(input_api.canned_checks.CheckRietveldTryJobExecution(input_api, |
| 116 | output_api, 'http://codereview.chromium.org', ('win', 'linux', 'mac'), |
| 117 | '[email protected]')) |
| 118 | |
[email protected] | 7fa42f1 | 2009-11-09 20:54:16 | [diff] [blame] | 119 | # These builders are just too slow. |
| 120 | IGNORED_BUILDERS = [ |
| 121 | 'Chromium XP', |
[email protected] | 7fa42f1 | 2009-11-09 20:54:16 | [diff] [blame] | 122 | 'Chromium Mac', |
[email protected] | 0133392 | 2010-02-02 21:25:02 | [diff] [blame] | 123 | 'Chromium Arm (dbg)', |
[email protected] | 7fa42f1 | 2009-11-09 20:54:16 | [diff] [blame] | 124 | 'Chromium Linux', |
| 125 | 'Chromium Linux x64', |
[email protected] | 7fa42f1 | 2009-11-09 20:54:16 | [diff] [blame] | 126 | ] |
[email protected] | 806e98e | 2010-03-19 17:49:27 | [diff] [blame] | 127 | results.extend(input_api.canned_checks.CheckBuildbotPendingBuilds( |
[email protected] | 7fa42f1 | 2009-11-09 20:54:16 | [diff] [blame] | 128 | input_api, |
| 129 | output_api, |
[email protected] | 806e98e | 2010-03-19 17:49:27 | [diff] [blame] | 130 | 'http://build.chromium.org/buildbot/waterfall/json/builders?filter=1', |
[email protected] | 7fa42f1 | 2009-11-09 20:54:16 | [diff] [blame] | 131 | 6, |
| 132 | IGNORED_BUILDERS)) |
[email protected] | fe5f57c5 | 2009-06-05 14:25:54 | [diff] [blame] | 133 | return results |
[email protected] | ca8d198 | 2009-02-19 16:33:12 | [diff] [blame] | 134 | |
| 135 | |
[email protected] | 5fa0629 | 2009-09-29 01:55:00 | [diff] [blame] | 136 | def GetPreferredTrySlaves(): |
| 137 | return ['win', 'linux', 'mac'] |