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

Unified Diff: scripts/slave/recipe_modules/goma/example.expected/linux.json

Issue 2480193002: Stop to use contextmanager for goma.build_with_goma (Closed)
Patch Set: rebase 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/goma/example.expected/linux.json
diff --git a/scripts/slave/recipe_modules/goma/example.expected/linux.json b/scripts/slave/recipe_modules/goma/example.expected/linux.json
index d54274ccdf26d01265fbc97b9eccbc37180079f4..56c8d6e116c6e43fc355a6af78a86c7cd4eced40 100644
--- a/scripts/slave/recipe_modules/goma/example.expected/linux.json
+++ b/scripts/slave/recipe_modules/goma/example.expected/linux.json
@@ -89,52 +89,11 @@
},
{
"cmd": [
- "python",
- "-u",
- "\nimport multiprocessing\nimport sys\n\njob_limit = 200\nif sys.platform.startswith('linux'):\n # Use 80 for linux not to load goma backend.\n job_limit = 80\n\ntry:\n jobs = min(job_limit, multiprocessing.cpu_count() * 10)\nexcept NotImplementedError:\n jobs = 50\n\nprint jobs\n"
- ],
- "name": "calculate the number of recommended jobs",
- "stdout": "/path/to/tmp/",
- "~followup_annotations": [
- "@@@STEP_LOG_LINE@python.inline@@@@",
- "@@@STEP_LOG_LINE@python.inline@import multiprocessing@@@",
- "@@@STEP_LOG_LINE@python.inline@import sys@@@",
- "@@@STEP_LOG_LINE@python.inline@@@@",
- "@@@STEP_LOG_LINE@python.inline@job_limit = 200@@@",
- "@@@STEP_LOG_LINE@python.inline@if sys.platform.startswith('linux'):@@@",
- "@@@STEP_LOG_LINE@python.inline@ # Use 80 for linux not to load goma backend.@@@",
- "@@@STEP_LOG_LINE@python.inline@ job_limit = 80@@@",
- "@@@STEP_LOG_LINE@python.inline@@@@",
- "@@@STEP_LOG_LINE@python.inline@try:@@@",
- "@@@STEP_LOG_LINE@python.inline@ jobs = min(job_limit, multiprocessing.cpu_count() * 10)@@@",
- "@@@STEP_LOG_LINE@python.inline@except NotImplementedError:@@@",
- "@@@STEP_LOG_LINE@python.inline@ jobs = 50@@@",
- "@@@STEP_LOG_LINE@python.inline@@@@",
- "@@@STEP_LOG_LINE@python.inline@print jobs@@@",
- "@@@STEP_LOG_END@python.inline@@@"
- ]
- },
- {
- "cmd": [
- "echo",
- "50"
- ],
- "name": "echo goma jobs"
- },
- {
- "cmd": [
- "echo",
- "50"
- ],
- "name": "echo goma jobs second"
- },
- {
- "cmd": [
"ninja",
- "-C",
- "out/Release",
"-j",
- "500"
+ "80",
+ "-C",
+ "out/Release"
],
"name": "ninja"
},
@@ -207,7 +166,7 @@
"--ninja-log-compiler",
"goma",
"--ninja-log-command",
- "['ninja', '-C', 'out/Release', '-j', '500']",
+ "['ninja', '-j', '80', '-C', 'out/Release']",
"--ninja-log-exit-status",
"0",
"--goma-stats-file",
« no previous file with comments | « scripts/slave/recipe_modules/goma/example.py ('k') | scripts/slave/recipe_modules/goma/example.expected/mac.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698