Make webkit-patch find-users useful
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 7 Oct 2014 05:26:20 +0000 (05:26 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 7 Oct 2014 05:26:20 +0000 (05:26 +0000)
https://bugs.webkit.org/show_bug.cgi?id=137475

Reviewed by Alexey Proskuryakov.

Make find-users use the CommitterList instead of accessing Bugzilla.

This essentially exposes the same functionality as webkitbot's whois command.

* Scripts/webkitpy/tool/commands/findusers.py:
(FindUsers.execute):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@174385 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/webkitpy/tool/commands/findusers.py

index 576d18a0452ea228da405602069f1870f2112f73..1146702d1318cceac6e69174b844a5ed63c2dc15 100644 (file)
@@ -1,3 +1,17 @@
+2014-10-06  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Make webkit-patch find-users useful
+        https://bugs.webkit.org/show_bug.cgi?id=137475
+
+        Reviewed by Alexey Proskuryakov.
+
+        Make find-users use the CommitterList instead of accessing Bugzilla.
+
+        This essentially exposes the same functionality as webkitbot's whois command.
+
+        * Scripts/webkitpy/tool/commands/findusers.py:
+        (FindUsers.execute):
+
 2014-10-06  Chris Fleizach  <cfleizach@apple.com>
 
         AX: iOS8: Crash at -[WebAccessibilityObjectWrapper accessibilityElementAtIndex:]
index ae4702373149ddcd77f78c1d07e24e7799f95032..8302e377f0bb74e3e5b01dd56ec29deab024cef7 100644 (file)
@@ -1,4 +1,5 @@
 # Copyright (c) 2011 Google Inc. All rights reserved.
+# Copyright (c) 2014 Apple Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
@@ -27,6 +28,7 @@
 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 from webkitpy.tool.multicommandtool import Command
+from webkitpy.common.config.committers import CommitterList
 
 
 class FindUsers(Command):
@@ -35,10 +37,6 @@ class FindUsers(Command):
 
     def execute(self, options, args, tool):
         search_string = args[0]
-        login_userid_pairs = tool.bugs.queries.fetch_login_userid_pairs_matching_substring(search_string)
-        for (login, user_id) in login_userid_pairs:
-            user = tool.bugs.fetch_user(user_id)
-            groups_string = ", ".join(user['groups']) if user['groups'] else "none"
-            print "%s <%s> (%s) (%s)" % (user['name'], user['login'], user_id, groups_string)
-        else:
-            print "No users found matching '%s'" % search_string
+        users = CommitterList().contributors_by_search_string(search_string)
+        for user in users:
+            print user