Index: build/scripts/slave/compile.py |
=================================================================== |
--- build/scripts/slave/compile.py (revision 227745) |
+++ build/scripts/slave/compile.py (working copy) |
@@ -17,7 +17,6 @@ |
import os |
import re |
import shlex |
-import shutil |
import sys |
import time |
@@ -506,9 +505,8 @@ |
def common_make_settings( |
command, options, env, crosstool=None, compiler=None): |
""" |
- Sets desirable environment variables and command-line options |
- that are common to the Make and SCons builds. Used on Linux |
- and for the mac make build. |
+ Sets desirable environment variables and command-line options that are used |
+ in the Make build. |
""" |
assert compiler in (None, 'clang', 'goma', 'goma-clang', 'jsonclang') |
maybe_set_official_build_envvars(options, env) |
@@ -577,24 +575,7 @@ |
command.append('CC.host=' + env['CC']) |
command.append('CXX.host=' + env['CXX']) |
- if chromium_utils.IsMac(): |
- # The default process limit on 10.6 is 266 (`sysctl kern.maxprocperuid`), |
- # and about 100 processes are used by the system. The webkit bindings |
- # generation scripts open a preprocessor child process, so building at |
- # -j100 runs into the process limit. For now, just build with -j50. |
- goma_jobs = 50 |
- if options.clobber: |
- # Disable compiles on local machine. When the goma server-side object |
- # file cache is warm, this can speed up clobber builds by up to 30%. |
- env['GOMA_USE_LOCAL'] = '0' |
- # Temproary hack to try failing fast when the server looks unhealthy. |
- # 30 seconds is chosen because a local compile is almost certainly |
- # faster. |
- # crbug.com/257467 |
- env['GOMA_RETRY'] = '0' |
- env['GOMA_COMPILER_PROXY_RPC_TIMEOUT_SECS'] = '30' |
- else: |
- goma_jobs = 50 |
+ goma_jobs = 50 |
if jobs < goma_jobs: |
jobs = goma_jobs |
command.append('-j%d' % jobs) |
@@ -642,23 +623,6 @@ |
else: |
working_dir = options.src_dir |
- # Lots of test-execution scripts hard-code 'sconsbuild' as the output |
- # directory. Accomodate them. |
- # TODO: remove when build_dir is properly parameterized in tests. |
- sconsbuild = os.path.join(working_dir, 'sconsbuild') |
- if os.path.islink(sconsbuild): |
- if os.readlink(sconsbuild) != 'out': |
- os.remove(sconsbuild) |
- elif os.path.exists(sconsbuild): |
- dead = sconsbuild + '.dead' |
- if os.path.isdir(dead): |
- shutil.rmtree(dead) |
- elif os.path.isfile(dead): |
- os.remove(dead) |
- os.rename(sconsbuild, sconsbuild+'.dead') |
- if not os.path.lexists(sconsbuild): |
- os.symlink('out', sconsbuild) |
- |
os.chdir(working_dir) |
common_make_settings(command, options, env, options.crosstool, |
options.compiler) |
@@ -885,50 +849,6 @@ |
return result |
-def main_scons(options, args): |
- """Interprets options, clobbers object files, and calls scons. |
- """ |
- options.build_dir = os.path.abspath(options.build_dir) |
- if options.clobber: |
- print('Removing %s' % options.target_output_dir) |
- chromium_utils.RemoveDirectory(options.target_output_dir) |
- |
- os.chdir(options.build_dir) |
- |
- if sys.platform == 'win32': |
- command = ['hammer.bat'] |
- else: |
- command = ['hammer'] |
- |
- env = EchoDict(os.environ) |
- if sys.platform == 'linux2': |
- common_make_settings(command, options, env) |
- else: |
- command.extend(['-k']) |
- |
- command.extend([ |
- # Force scons to always check for dependency changes. |
- '--implicit-deps-changed', |
- '--mode=' + options.target, |
- ]) |
- |
- # Here's what you can uncomment if you need to see more info |
- # about what the build is doing on a slave: |
- # |
- # VERBOSE=1 (a setting in our local SCons config) replaces |
- # the "Compiling ..." and "Linking ..." lines with the |
- # actual executed command line(s) |
- # |
- # --debug=explain (a SCons option) will tell you why SCons |
- # is deciding to rebuild thing (the target doesn't exist, |
- # which .h file(s) changed, etc.) |
- # |
- #command.extend(['--debug=explain', 'VERBOSE=1']) |
- command.extend(options.build_args + args) |
- env.print_overrides() |
- return chromium_utils.RunCommand(command, env=env) |
- |
- |
def main_win(options, args): |
"""Interprets options, clobbers object files, and calls the build tool. |
""" |
@@ -1133,8 +1053,6 @@ |
ret = os.path.join(src_dir, 'out') |
elif build_tool in ['msvs', 'vs', 'ib']: |
ret = os.path.join(src_dir, 'build', target) |
- elif build_tool == 'scons': |
- ret = os.path.join(src_dir, 'sconsbuild', target) |
else: |
raise NotImplementedError() |
return os.path.abspath(ret) |
@@ -1170,7 +1088,7 @@ |
help='build mode (dev or official) controlling ' |
'environment variables set during build') |
option_parser.add_option('', '--build-tool', default=None, |
- help='specify build tool (ib, vs, scons, xcode)') |
+ help='specify build tool (ib, vs, xcode)') |
option_parser.add_option('', '--build-args', action='append', default=[], |
help='arguments to pass to the build tool') |
option_parser.add_option('', '--compiler', default=None, |
@@ -1236,7 +1154,6 @@ |
'make' : main_make, |
'make-android' : main_make_android, |
'ninja' : main_ninja, |
- 'scons' : main_scons, |
'xcode' : main_xcode, |
} |
main = build_tool_map.get(options.build_tool) |