Bug 85742 - svn-apply cannot apply patches to files that contain space characters in their path
Summary: svn-apply cannot apply patches to files that contain space characters in thei...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-06 11:57 PDT by Daniel Bates
Modified: 2012-05-20 16:32 PDT (History)
3 users (show)

See Also:


Attachments
Patch and unit tests (8.51 KB, patch)
2012-05-20 15:38 PDT, Daniel Bates
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Bates 2012-05-06 11:57:23 PDT
svn-apply failed to apply the attachment #140425 [details] (from bug #85739) using git:

<http://queues.webkit.org/results/12507819>

I was able to apply the patch using SVN, but did get some SVN warning messages:

[[
copper:WebKit dbates$ Tools/Scripts/svn-apply /Users/dbates/Desktop/85739_r1.diff.txt 
Parsed 9 diffs from patch file(s).
patching file ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file Tools/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file Tools/Scripts/build-webkit
Hunk #1 succeeded at 552 (offset 307 lines).
patching file Tools/Scripts/copy-webkitlibraries-to-product-directory
A         Tools/Scripts/copy-webkitlibraries-to-product-directory
property 'svn:executable' set on 'Tools/Scripts/copy-webkitlibraries-to-product-directory'
A         WebKit.xcworkspace
patching file WebKit.xcworkspace/contents.xcworkspacedata
A         WebKit.xcworkspace/contents.xcworkspacedata
A         WebKit.xcworkspace/xcshareddata
patching file WebKit.xcworkspace/xcshareddata/WorkspaceSettings.xcsettings
A         WebKit.xcworkspace/xcshareddata/WorkspaceSettings.xcsettings
A         WebKit.xcworkspace/xcshareddata/xcschemes
patching file 'WebKit.xcworkspace/xcshareddata/xcschemes/All Source (target WebProcess).xcscheme Source (target WebProcess).xcscheme'
svn: warning: 'WebKit.xcworkspace/xcshareddata/xcschemes/All Source (target WebProcess).xcscheme' not found
patching file 'WebKit.xcworkspace/xcshareddata/xcschemes/All Source.xcscheme Source.xcscheme'
svn: warning: 'WebKit.xcworkspace/xcshareddata/xcschemes/All Source.xcscheme' not found
patching file 'WebKit.xcworkspace/xcshareddata/xcschemes/All Tools.xcscheme Tools.xcscheme'
svn: warning: 'WebKit.xcworkspace/xcshareddata/xcschemes/All Tools.xcscheme' not found
]]
Comment 1 Daniel Bates 2012-05-17 12:31:42 PDT
Attachment #142343 [details] (bug #86138) also fails to apply using svn-apply with a Git checkout.

I was able to apply it using svn-apply and a SVN checkout.
Comment 2 Daniel Bates 2012-05-20 15:19:20 PDT
(In reply to comment #1)
> Attachment #142343 [details] (bug #86138) also fails to apply using svn-apply with a Git checkout.
> 
> I was able to apply it using svn-apply and a SVN checkout.

The svn-apply failure with respect to attachment 142343 [details] is a separate issue. See bug #86973.
Comment 3 Daniel Bates 2012-05-20 15:38:57 PDT
Created attachment 142921 [details]
Patch and unit tests
Comment 4 WebKit Review Bot 2012-05-20 15:42:15 PDT
Attachment 142921 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Tools/ChangeLog', u'Tools/Scripts/VCSUtils..." exit_code: 1
Tools/Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffHeader.pl:97:  Line contains tab character.  [whitespace/tab] [5]
Tools/Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffHeader.pl:98:  Line contains tab character.  [whitespace/tab] [5]
Tools/Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffHeader.pl:107:  Line contains tab character.  [whitespace/tab] [5]
Tools/Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffHeader.pl:108:  Line contains tab character.  [whitespace/tab] [5]
Total errors found: 4 in 5 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 Eric Seidel (no email) 2012-05-20 15:49:06 PDT
Comment on attachment 142921 [details]
Patch and unit tests

You are an amazing man.  Thank you.
Comment 6 WebKit Review Bot 2012-05-20 16:32:00 PDT
Comment on attachment 142921 [details]
Patch and unit tests

Clearing flags on attachment: 142921

Committed r117715: <http://trac.webkit.org/changeset/117715>
Comment 7 WebKit Review Bot 2012-05-20 16:32:04 PDT
All reviewed patches have been landed.  Closing bug.