commit 4cbecbaa25a3285d3b3d385e07135feb1e3b61d0
author LaMont Jones <lamontjones@chromium.org> Tue May 12 11:54:27 2020 -0600
committer Commit Bot <commit-bot@chromium.org> Mon May 18 02:54:37 2020 +0000
tree 367db767fb0c0be167dadadbc412f01230d34da2
parent 98d8b656415debaa94c6ad674915d47bd40d589f

dependency: prefer sysroot in GetBuildDependencyGraphRequest

Also freshen API python bindings.

BUG=chromium:1081828, chromium:1053703
TEST=unit tests pass

Cq-Depend: chromium:2197281
Change-Id: I27b98a40b4dbc6197c8a07fc1098c5c45ebc4be6
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/2197226
Tested-by: LaMont Jones <lamontjones@chromium.org>
Commit-Queue: LaMont Jones <lamontjones@chromium.org>
Reviewed-by: Alex Klein <saklein@chromium.org>
Reviewed-by: Sean Abraham <seanabraham@chromium.org>

api/controller/dependency.py

diff --git a/api/controller/dependency.py b/api/controller/dependency.py
index ea243c1..fd3a434 100644
--- a/api/controller/dependency.py
+++ b/api/controller/dependency.py
@@ -16,6 +16,8 @@
 from chromite.api import faux
 from chromite.api import validate
 from chromite.api.controller import controller_util
+# TODO(crbug/1081828): stop using build_target and drop it from the proto.
+from chromite.lib import cros_build_lib
 from chromite.lib import portage_util
 from chromite.service import dependency
 
@@ -32,6 +34,9 @@
     graph: the proto object that represents the dependency graph (see DepGraph
       message in chromite/api/depgraph.proto)
   """
+  graph.sysroot.build_target.name = json_map['target_board']
+  graph.sysroot.path = json_map['sysroot_path']
+  # TODO(crbug/1081828): Drop this when no longer used.
   graph.build_target.name = json_map['target_board']
 
   for data in json_map['package_deps'].values():
@@ -62,7 +67,6 @@
 
 @faux.success(_GetBuildDependencyGraphResponse)
 @faux.empty_error
-@validate.require('build_target.name')
 @validate.validation_complete
 def GetBuildDependencyGraph(input_proto, output_proto, _config):
   """Create the build dependency graph.
@@ -72,10 +76,18 @@
     output_proto (GetBuildDependencyGraphResponse): The empty output message.
     _config (api_config.ApiConfig): The API call config.
   """
-  board = input_proto.build_target.name
+  if input_proto.HasField('sysroot'):
+    board = input_proto.sysroot.build_target.name
+    sysroot_path = input_proto.sysroot.path
+  else:
+    # TODO(crbug/1081828): stop using build_target and drop it from the proto.
+    board = input_proto.build_target.name
+    sysroot_path = cros_build_lib.GetSysroot(board or None)
+
   packages = [controller_util.PackageInfoToCPV(x) for x in input_proto.packages]
 
-  json_map, sdk_json_map = dependency.GetBuildDependency(board, packages)
+  json_map, sdk_json_map = dependency.GetBuildDependency(sysroot_path, board,
+                                                         packages)
   AugmentDepGraphProtoFromJsonMap(json_map, output_proto.dep_graph)
   AugmentDepGraphProtoFromJsonMap(sdk_json_map, output_proto.sdk_dep_graph)