Created attachment 38867 [details] Patch v1 Reviewed by NOBODY (OOPS!). The mark-fixed subcommand is for those times when you don't use bugzilla-tool to commit a patch, but you want to use it to close the bug with a committed-revision message. * Scripts/bugzilla-tool: (bug_comment_from_svn_revision): Added. Extracted from bug_comment_from_commit_text(). (bug_comment_from_commit_text): Extracted bug_comment_from_svn_revision() from this method. (MarkBugFixed.__init__): Added. (MarkBugFixed._determine_bug_id_and_svn_revision): Added. Attempts to use bug id and svn revision passed in from the command-line, but falls back to checking the last svn commit log if either isn't defined. (MarkBugFixed.execute): Added. Adds a comment about the revision that fixed the bug and closes the bug. (BugzillaTool.__init__): Added mark-fixed subcommand. * Scripts/modules/bugzilla.py: (Bugzilla.fetch_title_from_bug): Added. * Scripts/modules/scm.py: (SCM.last_svn_commit_log): Added. Subclasses must override. (SVN.last_svn_commit_log): Added. Uses svnversion to find the last commit in an svn working directory and then runs svn log. (Git.last_svn_commit_log): Added. Runs git-svn-log with a limit of one log message. --- 4 files changed, 115 insertions(+), 2 deletions(-)
Comment on attachment 38867 [details] Patch v1 > + return self.run_command(['svn', 'log', '-r', revision]) It would be nice to add --non-interactive to avoid having this hang on a password prompt. Of course, one could argue that this should be done for all svn commands which it isn't....
Comment on attachment 38867 [details] Patch v1 Thanks for the review, David! I realized that when you pass a revision but no bug id that it should look for the bug id in that revision, not the latest one. I want to update the patch to fix that.
Created attachment 38886 [details] Patch v2
Comment on attachment 38886 [details] Patch v2 It would be nice to add --non-interactive to the svn commands to avoid having this hang on a password prompt. Of course, one could argue that this should be done for all svn commands which it isn't....
(In reply to comment #4) > (From update of attachment 38886 [details]) > It would be nice to add --non-interactive to the svn commands to avoid having > this hang on a > password prompt. Of course, one could argue that this should be done for all > svn commands which it isn't.... Oops! Sorry, my bad. I meant to fix this from Patch v1. Will make this change for the new commands before landing.
Committed r47949: <http://trac.webkit.org/changeset/47949>
I'm starting to think we shoudl split bugzilla-tool out into multiple tools. One of which deals with bugzilla stuff, like this. Donno. I keep meaning to split the option parsing code out into its own file and then make it easy to break the Command subclasses up into files. We could at least group the commands into related categories. Related to that, I really want --help to just print a list of commands and add a "bugzilla help command_name" behavior for printing full option lists.
(In reply to comment #7) > I'm starting to think we shoudl split bugzilla-tool out into multiple tools. > One of which deals with bugzilla stuff, like this. Donno. The mark-fixed command would do well ask mark-bug-fixed, methinks.