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

Unified Diff: scripts/slave/recipes/infra/infra_repo_trybot.py

Issue 2045933002: build: only run infra recipe simulation tests on 64-bit Linux (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Created 4 years, 6 months 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
« no previous file with comments | « no previous file | scripts/slave/recipes/infra/infra_repo_trybot.expected/rietveld_tests_on_win.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipes/infra/infra_repo_trybot.py
diff --git a/scripts/slave/recipes/infra/infra_repo_trybot.py b/scripts/slave/recipes/infra/infra_repo_trybot.py
index a4a7ba5e6d7e18324eaadea9ca6310ceab4584db..bbb33dbfeb94a3460a075518ccb19b3b7f88a601 100644
--- a/scripts/slave/recipes/infra/infra_repo_trybot.py
+++ b/scripts/slave/recipes/infra/infra_repo_trybot.py
@@ -49,9 +49,12 @@ def RunSteps(api):
api.python('python tests', 'test.py', ['test'], cwd=api.path['checkout'])
- api.python(
- 'recipe tests', api.path['checkout'].join('recipes', 'recipes.py'),
- ['simulation_test', 'test'])
+ # TODO(phajdan.jr): should we make recipe tests run on other platforms?
+ if api.platform.is_linux and api.platform.bits == 64:
+ api.python(
+ 'recipe tests', api.path['checkout'].join('recipes', 'recipes.py'),
+ ['simulation_test', 'test'])
+
api.python(
'recipe lint', api.path['checkout'].join('recipes', 'recipes.py'),
['lint'])
« no previous file with comments | « no previous file | scripts/slave/recipes/infra/infra_repo_trybot.expected/rietveld_tests_on_win.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698