diff options
Diffstat (limited to 'infra/bots/recipe_modules')
7 files changed, 19 insertions, 446 deletions
diff --git a/infra/bots/recipe_modules/core/api.py b/infra/bots/recipe_modules/core/api.py index 714da7a3a5..901623601a 100644 --- a/infra/bots/recipe_modules/core/api.py +++ b/infra/bots/recipe_modules/core/api.py @@ -143,19 +143,6 @@ class SkiaApi(recipe_api.RecipeApi): # Run bot_update. - # TODO(borenet): Remove this hack after the transition to Kitchen: - # https://bugs.chromium.org/p/skia/issues/detail?id=7050 - if self.m.vars.need_chromium_checkout: - depot_tools = self.m.vars.checkout_root.join('depot_tools') - res = depot_tools.join( - 'recipes', 'recipe_modules', 'bot_update', 'resources') - self.m.git.checkout( - 'https://chromium.googlesource.com/chromium/tools/depot_tools.git', - dir_path=depot_tools, ref='06493714339009216197d59c5413da2a1efdf4a2') - def resource(r): - return res.join(r) - self.m.bot_update.resource = resource - # Hack the patch ref if necessary. if self.m.bot_update._issue and self.m.bot_update._patchset: self.m.bot_update._gerrit_ref = self.patch_ref( @@ -182,7 +169,4 @@ class SkiaApi(recipe_api.RecipeApi): if self.m.vars.need_chromium_checkout: with self.m.context(cwd=self.m.vars.checkout_root, env=self.m.vars.gclient_env): - # TODO(borenet): Replace this hack with 'self.m.gclient.runhooks' after - # the transition to Kitchen. - self.m.run(self.m.step, 'gclient runhooks', - cmd=[depot_tools.join('gclient'), 'runhooks']) + self.m.gclient.runhooks() diff --git a/infra/bots/recipe_modules/core/examples/full.expected/Housekeeper-Weekly-RecreateSKPs.json b/infra/bots/recipe_modules/core/examples/full.expected/Housekeeper-Weekly-RecreateSKPs.json index 0beadf0502..a2967213c3 100644 --- a/infra/bots/recipe_modules/core/examples/full.expected/Housekeeper-Weekly-RecreateSKPs.json +++ b/infra/bots/recipe_modules/core/examples/full.expected/Housekeeper-Weekly-RecreateSKPs.json @@ -31,93 +31,7 @@ "cmd": [ "python", "-u", - "RECIPE_MODULE[depot_tools::git]/resources/git_setup.py", - "--path", - "[CUSTOM_/_B_WORK]/depot_tools", - "--url", - "https://chromium.googlesource.com/chromium/tools/depot_tools.git" - ], - "name": "git setup" - }, - { - "cmd": [ - "git", - "retry", - "fetch", - "origin", - "--progress" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "env": { - "PATH": "RECIPE_PACKAGE_REPO[depot_tools]:<PATH>" - }, - "infra_step": true, - "name": "git fetch" - }, - { - "cmd": [ - "git", - "checkout", - "-f", - "06493714339009216197d59c5413da2a1efdf4a2" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "infra_step": true, - "name": "git checkout" - }, - { - "cmd": [ - "git", - "rev-parse", - "HEAD" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "infra_step": true, - "name": "read revision", - "stdout": "/path/to/tmp/", - "~followup_annotations": [ - "@@@STEP_TEXT@<br/>checked out 'deadbeef'<br/>@@@" - ] - }, - { - "cmd": [ - "git", - "clean", - "-f", - "-d", - "-x" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "infra_step": true, - "name": "git clean" - }, - { - "cmd": [ - "git", - "submodule", - "sync" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "infra_step": true, - "name": "submodule sync" - }, - { - "cmd": [ - "git", - "submodule", - "update", - "--init", - "--recursive" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "infra_step": true, - "name": "submodule update" - }, - { - "cmd": [ - "python", - "-u", - "[CUSTOM_/_B_WORK]/depot_tools/recipes/recipe_modules/bot_update/resources/bot_update.py", + "RECIPE_MODULE[depot_tools::bot_update]/resources/bot_update.py", "--spec-path", "cache_dir = '[CUSTOM_/_B_CACHE]'\nsolutions = [{'deps_file': '.DEPS.git', 'managed': False, 'name': 'skia', 'url': 'https://skia.googlesource.com/skia.git'}, {'deps_file': '.DEPS.git', 'managed': False, 'name': 'src', 'url': 'https://chromium.googlesource.com/chromium/src.git'}]", "--patch_root", @@ -184,17 +98,16 @@ }, { "cmd": [ - "[CUSTOM_/_B_WORK]/depot_tools/gclient", + "python", + "-u", + "RECIPE_PACKAGE_REPO[depot_tools]/gclient.py", "runhooks" ], "cwd": "[CUSTOM_/_B_WORK]", "env": { - "BUILDTYPE": "Release", - "CHROME_HEADLESS": "1", "CPPFLAGS": "-DSK_ALLOW_CROSSPROCESS_PICTUREIMAGEFILTERS=1", "DEPOT_TOOLS_UPDATE": "0", - "PATH": "<PATH>:RECIPE_PACKAGE_REPO[depot_tools]", - "SKIA_OUT": "[START_DIR]/build/out" + "PATH": "<PATH>:RECIPE_PACKAGE_REPO[depot_tools]" }, "name": "gclient runhooks" }, diff --git a/infra/bots/recipe_modules/flavor/examples/full.expected/Build-Debian9-GCC-x86_64-Release-Flutter_Android.json b/infra/bots/recipe_modules/flavor/examples/full.expected/Build-Debian9-GCC-x86_64-Release-Flutter_Android.json index e1e335f08a..df0b1af034 100644 --- a/infra/bots/recipe_modules/flavor/examples/full.expected/Build-Debian9-GCC-x86_64-Release-Flutter_Android.json +++ b/infra/bots/recipe_modules/flavor/examples/full.expected/Build-Debian9-GCC-x86_64-Release-Flutter_Android.json @@ -3,102 +3,14 @@ "cmd": [ "python", "-u", - "RECIPE_MODULE[depot_tools::git]/resources/git_setup.py", - "--path", - "[CUSTOM_/_B_WORK]/flutter/depot_tools", - "--url", - "https://chromium.googlesource.com/chromium/tools/depot_tools.git" - ], - "cwd": "[CUSTOM_/_B_WORK]/flutter/src", - "name": "git setup" - }, - { - "cmd": [ - "git", - "retry", - "fetch", - "origin", - "--progress" - ], - "cwd": "[CUSTOM_/_B_WORK]/flutter/depot_tools", - "env": { - "PATH": "RECIPE_PACKAGE_REPO[depot_tools]:<PATH>" - }, - "infra_step": true, - "name": "git fetch" - }, - { - "cmd": [ - "git", - "checkout", - "-f", - "06493714339009216197d59c5413da2a1efdf4a2" - ], - "cwd": "[CUSTOM_/_B_WORK]/flutter/depot_tools", - "infra_step": true, - "name": "git checkout" - }, - { - "cmd": [ - "git", - "rev-parse", - "HEAD" - ], - "cwd": "[CUSTOM_/_B_WORK]/flutter/depot_tools", - "infra_step": true, - "name": "read revision", - "stdout": "/path/to/tmp/", - "~followup_annotations": [ - "@@@STEP_TEXT@<br/>checked out 'deadbeef'<br/>@@@" - ] - }, - { - "cmd": [ - "git", - "clean", - "-f", - "-d", - "-x" - ], - "cwd": "[CUSTOM_/_B_WORK]/flutter/depot_tools", - "infra_step": true, - "name": "git clean" - }, - { - "cmd": [ - "git", - "submodule", - "sync" - ], - "cwd": "[CUSTOM_/_B_WORK]/flutter/depot_tools", - "infra_step": true, - "name": "submodule sync" - }, - { - "cmd": [ - "git", - "submodule", - "update", - "--init", - "--recursive" - ], - "cwd": "[CUSTOM_/_B_WORK]/flutter/depot_tools", - "infra_step": true, - "name": "submodule update" - }, - { - "cmd": [ - "[CUSTOM_/_B_WORK]/flutter/depot_tools/gclient", + "RECIPE_PACKAGE_REPO[depot_tools]/gclient.py", "runhooks" ], "cwd": "[CUSTOM_/_B_WORK]/flutter/src", "env": { - "BUILDTYPE": "Release", - "CHROME_HEADLESS": "1", - "PATH": "<PATH>:RECIPE_PACKAGE_REPO[depot_tools]", - "SKIA_OUT": "[CUSTOM_/_B_WORK]/flutter/src/third_party/skia/out/Build-Debian9-GCC-x86_64-Release-Flutter_Android" + "PATH": "<PATH>:RECIPE_PACKAGE_REPO[depot_tools]" }, - "name": "runhook" + "name": "gclient runhooks" }, { "cmd": [ diff --git a/infra/bots/recipe_modules/flavor/examples/full.expected/Build-Debian9-GCC-x86_64-Release-PDFium.json b/infra/bots/recipe_modules/flavor/examples/full.expected/Build-Debian9-GCC-x86_64-Release-PDFium.json index 4295bbb34d..9c31d772cc 100644 --- a/infra/bots/recipe_modules/flavor/examples/full.expected/Build-Debian9-GCC-x86_64-Release-PDFium.json +++ b/infra/bots/recipe_modules/flavor/examples/full.expected/Build-Debian9-GCC-x86_64-Release-PDFium.json @@ -3,123 +3,15 @@ "cmd": [ "python", "-u", - "RECIPE_MODULE[depot_tools::git]/resources/git_setup.py", - "--path", - "[CUSTOM_/_B_WORK]/depot_tools", - "--url", - "https://chromium.googlesource.com/chromium/tools/depot_tools.git" + "RECIPE_PACKAGE_REPO[depot_tools]/gclient.py", + "runhooks" ], "cwd": "[CUSTOM_/_B_WORK]/pdfium", "env": { - "DEPOT_TOOLS_UPDATE": "0" - }, - "name": "git setup" - }, - { - "cmd": [ - "git", - "retry", - "fetch", - "origin", - "--progress" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "env": { - "DEPOT_TOOLS_UPDATE": "0", - "PATH": "RECIPE_PACKAGE_REPO[depot_tools]:<PATH>" - }, - "infra_step": true, - "name": "git fetch" - }, - { - "cmd": [ - "git", - "checkout", - "-f", - "06493714339009216197d59c5413da2a1efdf4a2" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "env": { - "DEPOT_TOOLS_UPDATE": "0" - }, - "infra_step": true, - "name": "git checkout" - }, - { - "cmd": [ - "git", - "rev-parse", - "HEAD" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "env": { - "DEPOT_TOOLS_UPDATE": "0" - }, - "infra_step": true, - "name": "read revision", - "stdout": "/path/to/tmp/", - "~followup_annotations": [ - "@@@STEP_TEXT@<br/>checked out 'deadbeef'<br/>@@@" - ] - }, - { - "cmd": [ - "git", - "clean", - "-f", - "-d", - "-x" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "env": { - "DEPOT_TOOLS_UPDATE": "0" - }, - "infra_step": true, - "name": "git clean" - }, - { - "cmd": [ - "git", - "submodule", - "sync" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "env": { - "DEPOT_TOOLS_UPDATE": "0" - }, - "infra_step": true, - "name": "submodule sync" - }, - { - "cmd": [ - "git", - "submodule", - "update", - "--init", - "--recursive" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "env": { - "DEPOT_TOOLS_UPDATE": "0" - }, - "infra_step": true, - "name": "submodule update" - }, - { - "cmd": [ - "[CUSTOM_/_B_WORK]/depot_tools/gclient", - "runhook", - "gn_linux64" - ], - "cwd": "[CUSTOM_/_B_WORK]/pdfium", - "env": { - "BUILDTYPE": "Release", - "CHROME_HEADLESS": "1", "DEPOT_TOOLS_UPDATE": "0", - "PATH": "<PATH>:RECIPE_PACKAGE_REPO[depot_tools]", - "SKIA_OUT": "[CUSTOM_/_B_WORK]/pdfium/third_party/skia/out/Build-Debian9-GCC-x86_64-Release-PDFium" + "PATH": "<PATH>:RECIPE_PACKAGE_REPO[depot_tools]" }, - "name": "runhook" + "name": "gclient runhooks" }, { "cmd": [ diff --git a/infra/bots/recipe_modules/flavor/examples/full.expected/Build-Debian9-GCC-x86_64-Release-PDFium_SkiaPaths.json b/infra/bots/recipe_modules/flavor/examples/full.expected/Build-Debian9-GCC-x86_64-Release-PDFium_SkiaPaths.json index 06eff64fc8..2c0f40ff7e 100644 --- a/infra/bots/recipe_modules/flavor/examples/full.expected/Build-Debian9-GCC-x86_64-Release-PDFium_SkiaPaths.json +++ b/infra/bots/recipe_modules/flavor/examples/full.expected/Build-Debian9-GCC-x86_64-Release-PDFium_SkiaPaths.json @@ -3,123 +3,15 @@ "cmd": [ "python", "-u", - "RECIPE_MODULE[depot_tools::git]/resources/git_setup.py", - "--path", - "[CUSTOM_/_B_WORK]/depot_tools", - "--url", - "https://chromium.googlesource.com/chromium/tools/depot_tools.git" + "RECIPE_PACKAGE_REPO[depot_tools]/gclient.py", + "runhooks" ], "cwd": "[CUSTOM_/_B_WORK]/pdfium", "env": { - "DEPOT_TOOLS_UPDATE": "0" - }, - "name": "git setup" - }, - { - "cmd": [ - "git", - "retry", - "fetch", - "origin", - "--progress" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "env": { - "DEPOT_TOOLS_UPDATE": "0", - "PATH": "RECIPE_PACKAGE_REPO[depot_tools]:<PATH>" - }, - "infra_step": true, - "name": "git fetch" - }, - { - "cmd": [ - "git", - "checkout", - "-f", - "06493714339009216197d59c5413da2a1efdf4a2" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "env": { - "DEPOT_TOOLS_UPDATE": "0" - }, - "infra_step": true, - "name": "git checkout" - }, - { - "cmd": [ - "git", - "rev-parse", - "HEAD" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "env": { - "DEPOT_TOOLS_UPDATE": "0" - }, - "infra_step": true, - "name": "read revision", - "stdout": "/path/to/tmp/", - "~followup_annotations": [ - "@@@STEP_TEXT@<br/>checked out 'deadbeef'<br/>@@@" - ] - }, - { - "cmd": [ - "git", - "clean", - "-f", - "-d", - "-x" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "env": { - "DEPOT_TOOLS_UPDATE": "0" - }, - "infra_step": true, - "name": "git clean" - }, - { - "cmd": [ - "git", - "submodule", - "sync" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "env": { - "DEPOT_TOOLS_UPDATE": "0" - }, - "infra_step": true, - "name": "submodule sync" - }, - { - "cmd": [ - "git", - "submodule", - "update", - "--init", - "--recursive" - ], - "cwd": "[CUSTOM_/_B_WORK]/depot_tools", - "env": { - "DEPOT_TOOLS_UPDATE": "0" - }, - "infra_step": true, - "name": "submodule update" - }, - { - "cmd": [ - "[CUSTOM_/_B_WORK]/depot_tools/gclient", - "runhook", - "gn_linux64" - ], - "cwd": "[CUSTOM_/_B_WORK]/pdfium", - "env": { - "BUILDTYPE": "Release", - "CHROME_HEADLESS": "1", "DEPOT_TOOLS_UPDATE": "0", - "PATH": "<PATH>:RECIPE_PACKAGE_REPO[depot_tools]", - "SKIA_OUT": "[CUSTOM_/_B_WORK]/pdfium/third_party/skia/out/Build-Debian9-GCC-x86_64-Release-PDFium_SkiaPaths" + "PATH": "<PATH>:RECIPE_PACKAGE_REPO[depot_tools]" }, - "name": "runhook" + "name": "gclient runhooks" }, { "cmd": [ diff --git a/infra/bots/recipe_modules/flavor/flutter_flavor.py b/infra/bots/recipe_modules/flavor/flutter_flavor.py index ebc498edab..b9774d5204 100644 --- a/infra/bots/recipe_modules/flavor/flutter_flavor.py +++ b/infra/bots/recipe_modules/flavor/flutter_flavor.py @@ -22,17 +22,7 @@ class FlutterFlavorUtils(default_flavor.DefaultFlavorUtils): with self.m.context(cwd=flutter_dir): # Runhook to generate the gn binary in buildtools. - # TODO(borenet): Remove this hack and replace with - # 'self.m.gclient.runhooks()' after the transition to Kitchen: - # https://bugs.chromium.org/p/skia/issues/detail?id=7050 - depot_tools = self.m.vars.checkout_root.join('depot_tools') - self.m.git.checkout( - 'https://chromium.googlesource.com/chromium/tools/depot_tools.git', - dir_path=depot_tools, ref='06493714339009216197d59c5413da2a1efdf4a2') - self.m.run( - self.m.step, - 'runhook', - cmd=[depot_tools.join('gclient'), 'runhooks']) + self.m.gclient.runhooks() # Setup GN args. gn_args = [ diff --git a/infra/bots/recipe_modules/flavor/pdfium_flavor.py b/infra/bots/recipe_modules/flavor/pdfium_flavor.py index d0152ba1af..e67c3d93ea 100644 --- a/infra/bots/recipe_modules/flavor/pdfium_flavor.py +++ b/infra/bots/recipe_modules/flavor/pdfium_flavor.py @@ -18,17 +18,7 @@ class PDFiumFlavorUtils(default_flavor.DefaultFlavorUtils): # Runhook to generate the gn binary in buildtools. with self.m.context(cwd=pdfium_dir, env=self.m.vars.gclient_env): - # TODO(borenet): Remove this hack and replace with - # 'self.m.gclient.runhooks()' after the transition to Kitchen: - # https://bugs.chromium.org/p/skia/issues/detail?id=7050 - depot_tools = self.m.vars.checkout_root.join('depot_tools') - self.m.git.checkout( - 'https://chromium.googlesource.com/chromium/tools/depot_tools.git', - dir_path=depot_tools, ref='06493714339009216197d59c5413da2a1efdf4a2') - self.m.run( - self.m.step, - 'runhook', - cmd=[depot_tools.join('gclient'), 'runhook', 'gn_linux64']) + self.m.gclient.runhooks() # Install the sysroot. self.m.run( |