Bug 167906

Summary: svn-create-patch duplicating deletes
Product: WebKit Reporter: Jonathan Bedard <jbedard>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: ap, dbates, ddkilzer, lforschler, sam
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=167863

Jonathan Bedard
Reported 2017-02-06 14:48:25 PST
svn-create-patch will duplicate deletes, https://bugs.webkit.org/attachment.cgi?id=300680 is a good example of this. Among other files, PerformanceTests/ES6SampleBench/Basic/util.js is deleted 3 times.
Attachments
Jonathan Bedard
Comment 1 2017-02-06 16:29:10 PST
I was able to reproduce this. This is a bug in svn-create-patch. If you delete a file and then move the directory the file that was deleted is in, the resulting patch will have duplicated the delete. The commands to reproduce this are: rm SomeDir/file.txt svn rm SomeDir/file.txt svn mv SomeDir SomeNewDir I would guess that this was caused by https://bugs.webkit.org/show_bug.cgi?id=167169.
Alexey Proskuryakov
Comment 2 2017-02-06 17:07:21 PST
Is this a duplicate of bug 167903?
Jonathan Bedard
Comment 3 2017-02-07 08:07:36 PST
*** Bug 167903 has been marked as a duplicate of this bug. ***
Jonathan Bedard
Comment 4 2017-02-07 08:08:36 PST
(In reply to comment #2) > Is this a duplicate of bug 167903? Yes, it is the same issue. Marked the other as a duplicate since this one has more than just a report.
Alexey Proskuryakov
Comment 5 2017-02-07 09:05:01 PST
*** This bug has been marked as a duplicate of bug 167903 ***
Alexey Proskuryakov
Comment 6 2017-02-07 09:36:59 PST
Jonathan and I look into this briefly, and this may end up being a separate bug - there are no deletes in bug 167903.
Jonathan Bedard
Comment 7 2017-02-07 10:14:26 PST
This is not a duplicate of bug 167903, they are different issues.
David Kilzer (:ddkilzer)
Comment 8 2018-09-03 11:10:19 PDT
This is a duplicate of: Bug 14590: svn-create-patch fails when svn mv is used on directory trees The patch I have to fix that (not v1) also fixes this issue. *** This bug has been marked as a duplicate of bug 14590 ***
Note You need to log in before you can comment on or make changes to this bug.