commit a805216863256bd92406b250bf384c5416981ff7
author Alex Klein <saklein@chromium.org> Wed Feb 03 14:28:22 2021 -0700
committer Commit Bot <commit-bot@chromium.org> Fri Feb 19 23:55:26 2021 +0000
tree 9c572564daa0a0054abff11e98b3059d367b2f3d
parent bac05a93e9604be3d6f8c8057b23b977cd513e1a

NeedsChromeSource: Minor fixes.

The USE flag handling in the controller was not correct.
The build target depgraph was an unnecessarily wide net, only
the sysroot depgraph is needed to check for chrome requirements.

BUG=chromium:1086714
TEST=manually ran endpoint

Change-Id: Ifd6ab36af2e6aa95beba609285826a4ac08506a2
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/2676824
Tested-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Alex Klein <saklein@chromium.org>
Reviewed-by: Michael Mortensen <mmortensen@google.com>

api/controller/packages.py

diff --git a/api/controller/packages.py b/api/controller/packages.py
index 5c4b8ce..fe69826 100644
--- a/api/controller/packages.py
+++ b/api/controller/packages.py
@@ -351,7 +351,7 @@
   compile_source = input_proto.install_request.flags.compile_source
   pkgs = [controller_util.deserialize_package_info(pi) for pi in
           input_proto.install_request.packages]
-  use_flags = input_proto.install_request.use_flags
+  use_flags = [f.flag for f in input_proto.install_request.use_flags]
 
   result = packages.needs_chrome_source(
       build_target,