[Webkit-unassigned] [Bug 111066] svn-apply cannot apply patches which is generated by git to files that contain space characters in their path

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jun 5 01:45:24 PDT 2013


https://bugs.webkit.org/show_bug.cgi?id=111066





--- Comment #24 from Yuki Sekiguchi <yuki.sekiguchi at access-company.com>  2013-06-05 01:43:57 PST ---
Hi Daniel,

Thank you for kindly review.

I fixed codes which you comment.

(In reply to comment #22)
> (From update of attachment 203682 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=203682&action=review
> 
> This patch looks good. I have some minor nits.
> 
> > Tools/ChangeLog:8
> > +        If the path in a diff has space characters, parseGitDiffHeader() considers last nonspace characters as file path.
> 
> I'm unclear what you mean by this sentence.

I want to describe bug cause.
I changed this to the past tense.
Is this make sense?

> 
> > Tools/ChangeLog:9
> > +        When the diff have prefix, we consider next characters after "b/" as file path.
> 
> Is this sentence correct? I thought we currently considered the next characters after "b/" as part of a file path?

Sorry, I cannot understand your comment.
I added "part of a", but my fix might be wrong.

> > Tools/Scripts/VCSUtils.pm:114
> > +my $gitDiffPathPrefix = qr#^diff --git ([^\/]+\/)#;
> 
> It's unnecessary to escape the '/' character in this regular expression since we use '#' as the delimiter. Thus, we can simplify the regular expression in this line to read: ^diff --git ([^/]+/)
> 
> On another note, I wish we could come up with a more descriptive name for this variable. Maybe $gitDiffStartWithoutPrefixFirstDirectoryNameRegExp? or $gitDiffStartWithoutPrefixSourceDirectoryPrefixRegExp? or $gitDiffStartPathPrefixRegExp?

I like $gitDiffStartWithoutPrefixSourceDirectoryPrefixRegExp :)

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list