bot_update: add depot_tools to PATH so that e.g. apply_issue can be found

BUG=437264

Review URL: https://codereview.chromium.org/1660943002

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@298556 0039d316-1c4b-4281-b951-d872f2087c98

recipe_modules/bot_update/api.py

@@ -57,6 +57,10 @@ class BotUpdateApi(recipe_api.RecipeApi):
     assert isinstance(cmd, (list, tuple))
     bot_update_path = self.resource('bot_update.py')
     kwargs.setdefault('infra_step', True)
+    kwargs.setdefault('env', {})
+    kwargs['env'].setdefault('PATH', '%(PATH)s')
+    kwargs['env']['PATH'] = self.m.path.pathsep.join([
+        str(self._module.PACKAGE_DIRECTORY), kwargs['env']['PATH']])
     return self.m.python(name, bot_update_path, cmd, **kwargs)
 
   @property

recipe_modules/bot_update/example.expected/basic.json

@@ -24,6 +24,9 @@
       "src@abc"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update (without patch)",
     "~followup_annotations": [
       "@@@STEP_TEXT@Some step text@@@",

recipe_modules/bot_update/example.expected/basic_output_manifest.json

@@ -25,6 +25,9 @@
       "--output_manifest"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update",
     "~followup_annotations": [
       "@@@STEP_TEXT@Some step text@@@",

recipe_modules/bot_update/example.expected/basic_with_branch_heads.json

@@ -25,6 +25,9 @@
       "--with_branch_heads"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update - with branch heads",
     "~followup_annotations": [
       "@@@STEP_TEXT@Some step text@@@",

recipe_modules/bot_update/example.expected/clobber.json

@@ -25,6 +25,9 @@
       "--clobber"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update",
     "~followup_annotations": [
       "@@@STEP_LOG_LINE@json.output@{@@@",

recipe_modules/bot_update/example.expected/forced.json

@@ -25,6 +25,9 @@
       "--force"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update",
     "~followup_annotations": [
       "@@@STEP_TEXT@Some step text@@@",

recipe_modules/bot_update/example.expected/no_shallow.json

@@ -25,6 +25,9 @@
       "--no_shallow"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update",
     "~followup_annotations": [
       "@@@STEP_LOG_LINE@json.output@{@@@",

recipe_modules/bot_update/example.expected/off.json

@@ -24,6 +24,9 @@
       "src@HEAD"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update",
     "~followup_annotations": [
       "@@@STEP_LOG_LINE@json.output@{@@@",

recipe_modules/bot_update/example.expected/reset_root_solution_revision.json

@@ -24,6 +24,9 @@
       "src@revision"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update",
     "~followup_annotations": [
       "@@@STEP_LOG_LINE@json.output@{@@@",

recipe_modules/bot_update/example.expected/svn_mode.json

@@ -25,6 +25,9 @@
       "--force"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update",
     "~followup_annotations": [
       "@@@STEP_TEXT@Some step text@@@",

recipe_modules/bot_update/example.expected/trychange.json

@@ -26,6 +26,9 @@
       "+refs/change/1/2/333"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update",
     "~followup_annotations": [
       "@@@STEP_TEXT@Some step text@@@",

recipe_modules/bot_update/example.expected/trychange_oauth2.json

@@ -28,6 +28,9 @@
       "src@HEAD"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update",
     "~followup_annotations": [
       "@@@STEP_TEXT@Some step text@@@",

recipe_modules/bot_update/example.expected/tryjob.json

@@ -26,6 +26,9 @@
       "src@HEAD"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update",
     "~followup_annotations": [
       "@@@STEP_TEXT@Some step text@@@",

recipe_modules/bot_update/example.expected/tryjob_fail.json

@@ -26,6 +26,9 @@
       "src@HEAD"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update",
     "~followup_annotations": [
       "step returned non-zero exit code: 1",

recipe_modules/bot_update/example.expected/tryjob_fail_patch.json

@@ -26,6 +26,9 @@
       "src@HEAD"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update",
     "~followup_annotations": [
       "@@@STEP_TEXT@Some step text@@@",

recipe_modules/bot_update/example.expected/tryjob_fail_patch_download.json

@@ -26,6 +26,9 @@
       "src@HEAD"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update",
     "~followup_annotations": [
       "@@@STEP_TEXT@Some step text@@@",

recipe_modules/bot_update/example.expected/tryjob_v8.json

@@ -28,6 +28,9 @@
       "src/v8@abc"
     ],
     "cwd": "[SLAVE_BUILD]",
+    "env": {
+      "PATH": "RECIPE_PACKAGE[depot_tools]:%(PATH)s"
+    },
     "name": "bot_update",
     "~followup_annotations": [
       "@@@STEP_TEXT@Some step text@@@",