Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(605)

Unified Diff: scripts/slave/recipe_modules/chromium/example.py

Issue 2466793002: Add option to not use compile.py in chromium recipe_module (Closed)
Patch Set: small modify Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: scripts/slave/recipe_modules/chromium/example.py
diff --git a/scripts/slave/recipe_modules/chromium/example.py b/scripts/slave/recipe_modules/chromium/example.py
index d3fbfbd84d8877acced36f0b20a1978aa7272aad..2e19695052e3818f90825ac05c2d4980158e17a3 100644
--- a/scripts/slave/recipe_modules/chromium/example.py
+++ b/scripts/slave/recipe_modules/chromium/example.py
@@ -34,9 +34,10 @@ def RunSteps(api):
env = {}
if api.properties.get('goma_disable', False):
env.update({'GOMA_DISABLED': 'true'})
-
+ use_compile_py = api.properties.get('use_compile_py', True)
api.chromium.compile(targets=['All'], out_dir=out_dir,
use_goma_module=use_goma_module,
+ use_compile_py=use_compile_py,
env=env)
def GenTests(api):
@@ -48,6 +49,15 @@ def GenTests(api):
out_dir='/tmp',
)
+ yield api.test('basic_out_dir_without_compile_py') + api.properties(
+ mastername='chromium.linux',
+ buildername='Android Builder (dbg)',
+ slavename='build1-a1',
+ buildnumber='77457',
+ out_dir='/tmp',
+ use_compile_py=False,
+ )
+
yield api.test('basic_out_dir_with_goma_module') + api.properties(
mastername='chromium.linux',
buildername='Android Builder (dbg)',

Powered by Google App Engine
This is Rietveld 408576698