Browse Source

[recipe] Remove depot_tools.{ninja_path, autoninja_path} recipe APIs

I have replaced all the references, and there should be no references to these APIs anymore.

A PSA has been sent already.
http://g/chromium-dev-internal/n2IL0Mbh0E8

Bug: 1340825
Change-Id: Ife882f4427f8841cacb9a70a99dbd6a90c741da2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/4194896
Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
Commit-Queue: Junji Watanabe <jwata@google.com>
Reviewed-by: Gavin Mak <gavinmak@google.com>
Junji Watanabe 2 năm trước cách đây
mục cha
commit
9d77ca716f

+ 2 - 6
recipes/README.recipes.md

@@ -128,8 +128,6 @@ the depot_tools repo.
 
 #### **class [DepotToolsApi](/recipes/recipe_modules/depot_tools/api.py#12)([RecipeApi][recipe_engine/wkt/RecipeApi]):**
 
-&emsp; **@property**<br>&mdash; **def [autoninja\_path](/recipes/recipe_modules/depot_tools/api.py#49)(self):**
-
 &emsp; **@property**<br>&mdash; **def [cros\_path](/recipes/recipe_modules/depot_tools/api.py#30)(self):**
 
 &emsp; **@property**<br>&mdash; **def [download\_from\_google\_storage\_path](/recipes/recipe_modules/depot_tools/api.py#17)(self):**
@@ -138,9 +136,7 @@ the depot_tools repo.
 
 &emsp; **@property**<br>&mdash; **def [gsutil\_py\_path](/recipes/recipe_modules/depot_tools/api.py#40)(self):**
 
-&emsp; **@property**<br>&mdash; **def [ninja\_path](/recipes/recipe_modules/depot_tools/api.py#44)(self):**
-
-&emsp; **@contextlib.contextmanager**<br>&mdash; **def [on\_path](/recipes/recipe_modules/depot_tools/api.py#58)(self):**
+&emsp; **@contextlib.contextmanager**<br>&mdash; **def [on\_path](/recipes/recipe_modules/depot_tools/api.py#48)(self):**
 
 Use this context manager to put depot_tools on $PATH.
 
@@ -151,7 +147,7 @@ with api.depot_tools.on_path():
   # run some steps
 ```
 
-&emsp; **@property**<br>&mdash; **def [presubmit\_support\_py\_path](/recipes/recipe_modules/depot_tools/api.py#54)(self):**
+&emsp; **@property**<br>&mdash; **def [presubmit\_support\_py\_path](/recipes/recipe_modules/depot_tools/api.py#44)(self):**
 
 &emsp; **@property**<br>&mdash; **def [root](/recipes/recipe_modules/depot_tools/api.py#25)(self):**
 

+ 0 - 10
recipes/recipe_modules/depot_tools/api.py

@@ -41,16 +41,6 @@ class DepotToolsApi(recipe_api.RecipeApi):
   def gsutil_py_path(self):
     return self.repo_resource('gsutil.py')
 
-  @property
-  def ninja_path(self):
-    ninja_exe = 'ninja.exe' if self.m.platform.is_win else 'ninja'
-    return self.repo_resource(ninja_exe)
-
-  @property
-  def autoninja_path(self):
-    autoninja = 'autoninja.bat' if self.m.platform.is_win else 'autoninja'
-    return self.repo_resource(autoninja)
-
   @property
   def presubmit_support_py_path(self):
     return self.repo_resource('presubmit_support.py')

+ 0 - 14
recipes/recipe_modules/depot_tools/examples/full.expected/basic.json

@@ -41,20 +41,6 @@
     ],
     "name": "gsutil_py_path"
   },
-  {
-    "cmd": [
-      "ls",
-      "RECIPE_REPO[depot_tools]/ninja"
-    ],
-    "name": "ninja_path"
-  },
-  {
-    "cmd": [
-      "ls",
-      "RECIPE_REPO[depot_tools]/autoninja"
-    ],
-    "name": "autoninja_path"
-  },
   {
     "cmd": [
       "ls",

+ 0 - 14
recipes/recipe_modules/depot_tools/examples/full.expected/win.json

@@ -41,20 +41,6 @@
     ],
     "name": "gsutil_py_path"
   },
-  {
-    "cmd": [
-      "ls",
-      "RECIPE_REPO[depot_tools]\\ninja.exe"
-    ],
-    "name": "ninja_path"
-  },
-  {
-    "cmd": [
-      "ls",
-      "RECIPE_REPO[depot_tools]\\autoninja.bat"
-    ],
-    "name": "autoninja_path"
-  },
   {
     "cmd": [
       "ls",

+ 0 - 6
recipes/recipe_modules/depot_tools/examples/full.py

@@ -32,12 +32,6 @@ def RunSteps(api):
   api.step(
       'gsutil_py_path', ['ls', api.depot_tools.gsutil_py_path])
 
-  api.step(
-      'ninja_path', ['ls', api.depot_tools.ninja_path])
-
-  api.step(
-      'autoninja_path', ['ls', api.depot_tools.autoninja_path])
-
   api.step(
       'presubmit_support_py_path',
       ['ls', api.depot_tools.presubmit_support_py_path])