Chromium Code Reviews| Index: scripts/slave/recipes/chromium.chromedriver.recipe_autogen.py |
| diff --git a/scripts/slave/recipes/chromium.chromedriver.recipe_autogen.py b/scripts/slave/recipes/chromium.chromedriver.recipe_autogen.py |
| index a9c3324a77971675bf1266b50feec515ebf12970..813e58e7ee176ffe712fe6f83851f88c2ff7f19c 100644 |
| --- a/scripts/slave/recipes/chromium.chromedriver.recipe_autogen.py |
| +++ b/scripts/slave/recipes/chromium.chromedriver.recipe_autogen.py |
| @@ -53,27 +53,28 @@ def Linux32_steps(api): |
| build_properties.update(result.json.output.get('properties', {})) |
| # gclient update step; made unnecessary by bot_update |
| # gclient runhooks wrapper step |
| - env = {'CHROMIUM_GYP_SYNTAX_CHECK': '1', 'LANDMINES_VERBOSE': '1', |
| - 'DEPOT_TOOLS_UPDATE': '0', |
| - 'GYP_DEFINES': |
| - 'branding=Chrome buildtype=Official component=static_library'} |
| + env = {'LANDMINES_VERBOSE': '1', 'DEPOT_TOOLS_UPDATE': '0'} |
| api.python("gclient runhooks wrapper", api.path["build"].join("scripts", |
| "slave", "runhooks_wrapper.py"), env=env) |
| # cleanup_temp step |
| api.chromium.cleanup_temp() |
| + # meta build step |
| + api.python("meta build", api.path["checkout"].join("tools", "mb", "mb.py", |
|
jbudorick
2016/07/22 00:45:01
This should be:
api.python('meta build', api.pa
samuong
2016/07/22 02:49:12
Done.
|
| + "gen", "-m", "chromium.chromedriver", "-b", "Linux32", "//out/Default")) |
| # chromedriver compile.py step |
| api.python("compile", api.path["build"].join("scripts", "slave", |
| - "compile.py"), args=['--target', 'Release', |
| - 'chromium_builder_chromedriver']) |
| + "compile.py"), args=['--target', 'Default', |
| + 'chromedriver', 'chromedriver_tests', 'chromedriver_unittests']) |
| + # strip binary |
| + api.m.step('strip', cmd=['strip', str(api.path['checkout'].join( |
|
jbudorick
2016/07/22 00:45:01
What's with the strip step?
samuong
2016/07/22 02:49:12
I'm seeing very large (144M) binaries when I build
jbudorick
2016/07/22 02:58:31
That's what I get for not reading that bug closely
|
| + 'out', 'Default', 'chromedriver'))]) |
| # annotated_steps step |
| api.python("annotated_steps", api.path["build"].join("scripts", "slave", |
| "chromium", "chromedriver_buildbot_run.py"), |
| args=['--build-properties=%s' % api.json.dumps(build_properties, |
| separators=(',', ':')), '--factory-properties={"annotated_script":'+\ |
| '"chromedriver_buildbot_run.py","blink_config":"chromium",'+\ |
| - '"gclient_env":{"CHROMIUM_GYP_SYNTAX_CHECK":"1",'+\ |
| - '"DEPOT_TOOLS_UPDATE"'':"0","GYP_DEFINES":"branding=Chrome '+\ |
| - 'buildtype=Official component=static_library","LANDMINES_VERBOSE":'+\ |
| + '"gclient_env":{"DEPOT_TOOLS_UPDATE":"0","LANDMINES_VERBOSE":'+\ |
| '"1"},"needs_webdriver_java_tests":true,"use_xvfb_on_linux":true}'], |
| allow_subannotations=True) |