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

Unified Diff: scripts/slave/recipe_modules/chromium/example.expected/codesearch.json

Issue 2480193002: Stop to use contextmanager for goma.build_with_goma (Closed)
Patch Set: fix pylint 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.expected/codesearch.json
diff --git a/scripts/slave/recipe_modules/chromium/example.expected/codesearch.json b/scripts/slave/recipe_modules/chromium/example.expected/codesearch.json
index d83d3f766669a34310e4e74aa02cd59d90f209a7..0e3c730aa12382f4fa52e4e3316bed4b47654e59 100644
--- a/scripts/slave/recipe_modules/chromium/example.expected/codesearch.json
+++ b/scripts/slave/recipe_modules/chromium/example.expected/codesearch.json
@@ -104,33 +104,6 @@
]
},
{
- "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": [],
"name": "preprocess_for_goma"
},
@@ -173,15 +146,42 @@
},
{
"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": [
"RECIPE_PACKAGE_REPO[depot_tools]/ninja",
+ "-j",
+ "50",
"-w",
"dupbuild=err",
"-C",
"None/out/Debug",
"-k",
"0",
- "-j",
- "50",
"All"
],
"env": {
@@ -252,7 +252,7 @@
"--ninja-log-compiler",
"goma-clang",
"--ninja-log-command",
- "['RECIPE_PACKAGE_REPO[depot_tools]/ninja', '-w', 'dupbuild=err', '-C', 'None/out/Debug', '-k', '0', '-j', 50, 'All']",
+ "['RECIPE_PACKAGE_REPO[depot_tools]/ninja', '-j', 50, '-w', 'dupbuild=err', '-C', 'None/out/Debug', '-k', '0', 'All']",
"--ninja-log-exit-status",
"0",
"--buildbot-buildername",

Powered by Google App Engine
This is Rietveld 408576698