Bug 58344

Summary: webkitpy.common.checkout.diff_parser errors when parsing SVN property changes
Product: WebKit Reporter: Daniel Bates <dbates>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: abarth, eric, jbedard, jcraig
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   

Description Daniel Bates 2011-04-12 10:23:59 PDT
It seems that webkitpy.common.checkout.diff_parser doesn't know how to parse SVN property changes.

1. Apply the patch for bug #53625
2. Run, svn propset allow-tabs 1 Tools/Scripts/webkitperl/VCSUtils_unittest/parseDiffWithMockFiles.pl
3. Run, Tools/Scripts/webkit-patch land 53625

Then I received the following output:

dbatesRIM:WebKit dbates$ Tools/Scripts/webkit-patch land 53625

...

Fetching: https://bugs.webkit.org/show_bug.cgi?id=53625&ctype=xml
Guessing "Eric Seidel" as reviewer from attachment 89082 [details] on bug 53625.
WARNING: Patch's size is 22 kbytes.
Patches 20k or smaller are more likely to be reviewed. Larger patches may sit unreviewed for a long time.
webkitpy.common.checkout.diff_parser: [ERROR] Unexpected diff format when parsing a chunk: ''
webkitpy.common.checkout.diff_parser: [ERROR] Unexpected diff format when parsing a chunk: 'Property changes on: Tools/Scripts/webkitperl/VCSUtils_unittest/parseDiffWithMockFiles.pl'
webkitpy.common.checkout.diff_parser: [ERROR] Unexpected diff format when parsing a chunk: '___________________________________________________________________'
webkitpy.common.checkout.diff_parser: [ERROR] Unexpected diff format when parsing a chunk: 'Added: allow-tabs'
webkitpy.common.checkout.diff_parser: [ERROR] Unexpected diff format when parsing a chunk: ''
Parsing ChangeLog: /Users/dbates/Desktop/WebKit/Tools/ChangeLog
Committed r83603: <http://trac.webkit.org/changeset/83603>
Updating bug 53625
Reading Keychain for bugs.webkit.org account and password.  Click "Allow" to continue...
...

In particular, notice the webkitpy.common.checkout.diff_parser errors.
Comment 1 Sam Sneddon [:gsnedders] 2023-11-16 06:59:04 PST
*** Bug 132210 has been marked as a duplicate of this bug. ***
Comment 2 Sam Sneddon [:gsnedders] 2023-11-16 07:00:46 PST
Do we actually still need support for SVN in the diff_parser?