blob: 6c093c9cdd9b4b09e65ddcc367585e056146488f [file] [log] [blame]
[email protected]377ab1da2011-03-17 15:36:281# Copyright (c) 2011 The Chromium Authors. All rights reserved.
[email protected]ca8d1982009-02-19 16:33:122# 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]f1293792009-07-31 18:09:567See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
[email protected]50d7d721e2009-11-15 17:56:188for more details about the presubmit API built into gcl.
[email protected]ca8d1982009-02-19 16:33:129"""
10
[email protected]379e7dd2010-01-28 17:39:2111_EXCLUDED_PATHS = (
[email protected]3e4eb112011-01-18 03:29:5412 r"^breakpad[\\\/].*",
13 r"^net/tools/spdyshark/[\\\/].*",
14 r"^skia[\\\/].*",
15 r"^v8[\\\/].*",
16 r".*MakeFile$",
[email protected]4306417642009-06-11 00:33:4017)
[email protected]ca8d1982009-02-19 16:33:1218
[email protected]ca8d1982009-02-19 16:33:1219
[email protected]22c9bd72011-03-27 16:47:3920def _CheckNoInterfacesInBase(input_api, output_api):
[email protected]6a4c8e682010-12-19 03:31:3421 """Checks to make sure no files in libbase.a have |@interface|."""
[email protected]839c1392011-04-29 20:15:1922 pattern = input_api.re.compile(r'^\s*@interface', input_api.re.MULTILINE)
[email protected]6a4c8e682010-12-19 03:31:3423 files = []
[email protected]22c9bd72011-03-27 16:47:3924 for f in input_api.AffectedSourceFiles(input_api.FilterSourceFile):
[email protected]6a4c8e682010-12-19 03:31:3425 if (f.LocalPath().find('base/') != -1 and
[email protected]0b2f07b02011-05-02 17:29:0026 f.LocalPath().find('base/test/') == -1 and
27 not f.LocalPath().endswith('_unittest.mm')):
[email protected]6a4c8e682010-12-19 03:31:3428 contents = input_api.ReadFile(f)
29 if pattern.search(contents):
30 files.append(f)
31
32 if len(files):
33 return [ output_api.PresubmitError(
34 'Objective-C interfaces or categories are forbidden in libbase. ' +
35 'See http://groups.google.com/a/chromium.org/group/chromium-dev/' +
36 'browse_thread/thread/efb28c10435987fd',
37 files) ]
38 return []
39
[email protected]650c9082010-12-14 14:33:4440
[email protected]22c9bd72011-03-27 16:47:3941def _CommonChecks(input_api, output_api):
42 """Checks common to both upload and commit."""
43 results = []
44 results.extend(input_api.canned_checks.PanProjectChecks(
45 input_api, output_api, excluded_paths=_EXCLUDED_PATHS))
46 results.extend(_CheckNoInterfacesInBase(input_api, output_api))
47 return results
[email protected]1f7b4172010-01-28 01:17:3448
[email protected]b337cb5b2011-01-23 21:24:0549
50def _CheckSubversionConfig(input_api, output_api):
51 """Verifies the subversion config file is correctly setup.
52
53 Checks that autoprops are enabled, returns an error otherwise.
54 """
55 join = input_api.os_path.join
56 if input_api.platform == 'win32':
57 appdata = input_api.environ.get('APPDATA', '')
58 if not appdata:
59 return [output_api.PresubmitError('%APPDATA% is not configured.')]
60 path = join(appdata, 'Subversion', 'config')
61 else:
62 home = input_api.environ.get('HOME', '')
63 if not home:
64 return [output_api.PresubmitError('$HOME is not configured.')]
65 path = join(home, '.subversion', 'config')
66
67 error_msg = (
68 'Please look at http://dev.chromium.org/developers/coding-style to\n'
69 'configure your subversion configuration file. This enables automatic\n'
[email protected]c6a3c10b2011-01-24 16:14:2070 'properties to simplify the project maintenance.\n'
71 'Pro-tip: just download and install\n'
72 'http://src.chromium.org/viewvc/chrome/trunk/tools/build/slave/config\n')
[email protected]b337cb5b2011-01-23 21:24:0573
74 try:
75 lines = open(path, 'r').read().splitlines()
76 # Make sure auto-props is enabled and check for 2 Chromium standard
77 # auto-prop.
78 if (not '*.cc = svn:eol-style=LF' in lines or
79 not '*.pdf = svn:mime-type=application/pdf' in lines or
80 not 'enable-auto-props = yes' in lines):
81 return [
[email protected]79ed7e62011-02-21 21:08:5382 output_api.PresubmitNotifyResult(
[email protected]b337cb5b2011-01-23 21:24:0583 'It looks like you have not configured your subversion config '
[email protected]b5359c02011-02-01 20:29:5684 'file or it is not up-to-date.\n' + error_msg)
[email protected]b337cb5b2011-01-23 21:24:0585 ]
86 except (OSError, IOError):
87 return [
[email protected]79ed7e62011-02-21 21:08:5388 output_api.PresubmitNotifyResult(
[email protected]b337cb5b2011-01-23 21:24:0589 'Can\'t find your subversion config file.\n' + error_msg)
90 ]
91 return []
92
93
[email protected]1f7b4172010-01-28 01:17:3494def CheckChangeOnUpload(input_api, output_api):
95 results = []
96 results.extend(_CommonChecks(input_api, output_api))
[email protected]fe5f57c52009-06-05 14:25:5497 return results
[email protected]ca8d1982009-02-19 16:33:1298
99
100def CheckChangeOnCommit(input_api, output_api):
[email protected]fe5f57c52009-06-05 14:25:54101 results = []
[email protected]1f7b4172010-01-28 01:17:34102 results.extend(_CommonChecks(input_api, output_api))
[email protected]dd805fe2009-10-01 08:11:51103 # TODO(thestig) temporarily disabled, doesn't work in third_party/
104 #results.extend(input_api.canned_checks.CheckSvnModifiedDirectories(
105 # input_api, output_api, sources))
[email protected]fe5f57c52009-06-05 14:25:54106 # Make sure the tree is 'open'.
[email protected]806e98e2010-03-19 17:49:27107 results.extend(input_api.canned_checks.CheckTreeIsOpen(
[email protected]7f238152009-08-12 19:00:34108 input_api,
109 output_api,
[email protected]4efa42142010-08-26 01:29:26110 json_url='http://chromium-status.appspot.com/current?format=json'))
[email protected]806e98e2010-03-19 17:49:27111 results.extend(input_api.canned_checks.CheckRietveldTryJobExecution(input_api,
112 output_api, 'http://codereview.chromium.org', ('win', 'linux', 'mac'),
113 '[email protected]'))
114
[email protected]3e4eb112011-01-18 03:29:54115 results.extend(input_api.canned_checks.CheckChangeHasBugField(
116 input_api, output_api))
117 results.extend(input_api.canned_checks.CheckChangeHasTestField(
118 input_api, output_api))
[email protected]b337cb5b2011-01-23 21:24:05119 results.extend(_CheckSubversionConfig(input_api, output_api))
[email protected]fe5f57c52009-06-05 14:25:54120 return results
[email protected]ca8d1982009-02-19 16:33:12121
122
[email protected]5fa06292009-09-29 01:55:00123def GetPreferredTrySlaves():
124 return ['win', 'linux', 'mac']