commit cc2277061f701c43b1c072f093c03440459a2064
author Gabe Black <gabeblack@chromium.org> Tue May 20 13:13:33 2014 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Wed May 21 04:18:07 2014 +0000
tree 7e4fef9c2ba308d77421977212664d25b29ef862
parent 3f1792c19cf462eae97557b221fcd40e5e918ace

Fix the return information for GetDepChangesForChange.

That function returns a tuple of lists of objects, not a tuple of the objects
themselves.

BUG=None
TEST=Ran run_tests.

Change-Id: Ic32058501aba2566a76dca80e4173d333d02556a
Reviewed-on: https://chromium-review.googlesource.com/200643
Reviewed-by: Yu-Ju Hong <yjhong@chromium.org>
Commit-Queue: Gabe Black <gabeblack@chromium.org>
Tested-by: Gabe Black <gabeblack@chromium.org>

buildbot/validation_pool.py

diff --git a/buildbot/validation_pool.py b/buildbot/validation_pool.py
index b0fdbac..abc44ea 100644
--- a/buildbot/validation_pool.py
+++ b/buildbot/validation_pool.py
@@ -734,8 +734,8 @@
     """Look up the Gerrit/CQ dependency changes for |change|.
 
     Returns:
-      A tuple of GerritPatch objects which are change's Gerrit
-      dependencies, and CQ dependencies.
+      (gerrit_deps, cq_deps): The change's Gerrit dependencies and CQ
+      dependencies, as lists of GerritPatch objects.
 
     Raises:
       DependencyError: If we could not resolve a dependency.