[mb] Add presubmit checks
Also comment out some currently unused configs the presubmit check now complains about. They'll be used soon, once respective bots are added. BUG=chromium:616035 TBR=kjellander@chromium.org NOTRY=true Review-Url: https://codereview.chromium.org/2076053002 Cr-Commit-Position: refs/heads/master@{#37056}
This commit is contained in:
parent
b8238f8977
commit
3249e23e5a
35
infra/mb/PRESUBMIT.py
Normal file
35
infra/mb/PRESUBMIT.py
Normal file
@ -0,0 +1,35 @@
|
||||
# Copyright 2016 the V8 project authors. All rights reserved.
|
||||
# Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
|
||||
#
|
||||
# Use of this source code is governed by a BSD-style license
|
||||
# that can be found in the LICENSE file in the root of the source
|
||||
# tree. An additional intellectual property rights grant can be found
|
||||
# in the file PATENTS. All contributing project authors may
|
||||
# be found in the AUTHORS file in the root of the source tree.
|
||||
|
||||
|
||||
def _CommonChecks(input_api, output_api):
|
||||
results = []
|
||||
|
||||
# Validate the format of the mb_config.pyl file.
|
||||
mb_script = input_api.os_path.join(input_api.PresubmitLocalPath(), '..',
|
||||
'..', 'tools', 'mb', 'mb.py')
|
||||
mb_config_path = input_api.os_path.join(input_api.PresubmitLocalPath(),
|
||||
'mb_config.pyl')
|
||||
cmd = [input_api.python_executable, mb_script, 'validate', '--config-file',
|
||||
mb_config_path]
|
||||
kwargs = {'cwd': input_api.PresubmitLocalPath()}
|
||||
results.extend(input_api.RunTests([
|
||||
input_api.Command(name='mb_validate',
|
||||
cmd=cmd, kwargs=kwargs,
|
||||
message=output_api.PresubmitError)]))
|
||||
|
||||
return results
|
||||
|
||||
|
||||
def CheckChangeOnUpload(input_api, output_api):
|
||||
return _CommonChecks(input_api, output_api)
|
||||
|
||||
|
||||
def CheckChangeOnCommit(input_api, output_api):
|
||||
return _CommonChecks(input_api, output_api)
|
@ -90,10 +90,10 @@
|
||||
'mixins': ['release_bot', 'minimal_symbols', 'dcheck_always_on'],
|
||||
},
|
||||
|
||||
'shared': {
|
||||
'gn_args': 'is_component_build=true',
|
||||
'gyp_defines': 'component=shared_library',
|
||||
},
|
||||
#'shared': {
|
||||
# 'gn_args': 'is_component_build=true',
|
||||
# 'gyp_defines': 'component=shared_library',
|
||||
#},
|
||||
|
||||
'static': {
|
||||
'gn_args': 'is_component_build=false',
|
||||
@ -137,9 +137,9 @@
|
||||
'gyp_defines': 'target_arch=x64',
|
||||
},
|
||||
|
||||
'x86': {
|
||||
'gn_args': 'target_cpu="x86"',
|
||||
'gyp_defines': 'target_arch=ia32',
|
||||
},
|
||||
#'x86': {
|
||||
# 'gn_args': 'target_cpu="x86"',
|
||||
# 'gyp_defines': 'target_arch=ia32',
|
||||
#},
|
||||
},
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user