Bug 31710 - MultiCommandTool should find Command objects automatically instead of with a manual list
Summary: MultiCommandTool should find Command objects automatically instead of with a ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-11-20 00:40 PST by Eric Seidel (no email)
Modified: 2009-11-20 06:54 PST (History)
1 user (show)

See Also:


Attachments
Patch (14.68 KB, patch)
2009-11-20 00:44 PST, Eric Seidel (no email)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2009-11-20 00:40:18 PST
MultiCommandTool should find Command objects automatically instead of with a manual list

This makes it easier to add commands, and it also brings the "name" of the command closer to its implementation (which has been a historical point of confusion).
Comment 1 Eric Seidel (no email) 2009-11-20 00:41:34 PST
This is setting the groundwork for making it easy to add "hidden" commands which don't show up in --help by default.
Comment 2 Eric Seidel (no email) 2009-11-20 00:44:13 PST
Created attachment 43561 [details]
Patch
Comment 3 Adam Barth 2009-11-20 00:52:45 PST
Comment on attachment 43561 [details]
Patch

kk
Comment 4 WebKit Commit Bot 2009-11-20 01:02:26 PST
Comment on attachment 43561 [details]
Patch

Rejecting patch 43561 from commit-queue.

Unexpected failure when landing patch!  Please file a bug against bugzilla-tool.
Failed to run "['WebKitTools/Scripts/bugzilla-tool', 'land-attachment', '--force-clean', '--non-interactive', '--quiet', '43561']" exit_code: 1
Last 500 characters of output:
ingsequence.py", line 49, in run
    self.test()
  File "/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/modules/landingsequence.py", line 113, in test
    LandingSequence.test(self)
  File "/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/modules/landingsequence.py", line 81, in test
    WebKitScripts.run_webkit_tests(launch_safari=not options.non_interactive, fail_fast=options.non_interactive, quiet=options.quiet, port=port)
NameError: global name 'WebKitScripts' is not defined
Comment 5 Eric Seidel (no email) 2009-11-20 01:07:58 PST
I think that my patch and adam's patch secretly conflicted.  I'll post a new one tomorrow.
Comment 6 Adam Barth 2009-11-20 06:46:10 PST
Comment on attachment 43561 [details]
Patch

Oops.  I screwed up the bugzilla-tool.  Should be fixed now.
Comment 7 WebKit Commit Bot 2009-11-20 06:49:51 PST
Comment on attachment 43561 [details]
Patch

Rejecting patch 43561 from commit-queue.

Unexpected failure when landing patch!  Please file a bug against bugzilla-tool.
Failed to run "['WebKitTools/Scripts/bugzilla-tool', 'land-attachment', '--force-clean', '--non-interactive', '--quiet', '43561']" exit_code: 1
Last 500 characters of output:
n
    self.test()
  File "/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/modules/landingsequence.py", line 113, in test
    LandingSequence.test(self)
  File "/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/modules/landingsequence.py", line 81, in test
    WebKitLandingScripts.run_webkit_tests(launch_safari=not self._options.non_interactive, fail_fast=self._options.non_interactive, quiet=self._options.quiet, port=self._port)
NameError: global name 'WebKitScripts' is not defined
Comment 8 Adam Barth 2009-11-20 06:54:43 PST
Comment on attachment 43561 [details]
Patch

Clearing flags on attachment: 43561

Committed r51237: <http://trac.webkit.org/changeset/51237>
Comment 9 Adam Barth 2009-11-20 06:54:50 PST
All reviewed patches have been landed.  Closing bug.