[Webkit-unassigned] [Bug 26914] bugzilla-tool fails for SVN users
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Jul 1 22:01:37 PDT 2009
https://bugs.webkit.org/show_bug.cgi?id=26914
------- Comment #4 from levin at chromium.org 2009-07-01 22:01 PDT -------
I'll let Adam's r+ stand, but here are some comments to consider:
> diff --git a/WebKitTools/Scripts/modules/scm.py b/WebKitTools/Scripts/modules/scm.py
> # All git-specific logic should go here.
> @@ -224,51 +234,56 @@ class Git(SCM):
> + @staticmethod
> + def commit_success_regexp():
> + return "^Committed r(?P<svn_revision>\d+)\.$"
This shouldn't have a \. in it.
Here's a sample git svn dcommit message:
Committing to http://svn.webkit.org/repository/webkit/trunk ...
M WebCore/ChangeLog
...
Committed r45448
M WebCore/ChangeLog
...
r45448 = 97e3c9608ffcf1f67fe127a738e043e2db154d7e (trunk)
> def files_changed_summary_for_commit(self, commit_id):
> - return self.run_command("git diff-tree --shortstat --no-commit-id " + commit_id)
> + return self.run_command(['git', 'diff-tree', '--short-stat', '--no-commit-id', commit_id])
Why did the parameter change to from "--shortstat" to "--short-stat"? (I
didn't see --short-stat listed as a valid option for "git diff-tree".)
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
More information about the webkit-unassigned
mailing list