Bug 174834 - svn-apply should be able to apply patches in reverse
Summary: svn-apply should be able to apply patches in reverse
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Enhancement
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-07-25 14:13 PDT by Matthew Hanson
Modified: 2017-07-25 17:29 PDT (History)
5 users (show)

See Also:


Attachments
Patch v1 (3.01 KB, patch)
2017-07-25 14:19 PDT, Matthew Hanson
dbates: review-
dbates: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew Hanson 2017-07-25 14:13:41 PDT
I would like to take advantage of svn-apply when rolling out changes.

I propose adding a -R|--reverse option (API borrowed from the patch utility and git-apply) to the svn-apply script. The work here is minimal, since the underlying call to VCSUtils::generatePatchCommand already understands how to reverse patches.
Comment 1 Radar WebKit Bug Importer 2017-07-25 14:14:22 PDT
<rdar://problem/33521443>
Comment 2 Matthew Hanson 2017-07-25 14:19:48 PDT
Created attachment 316393 [details]
Patch v1

Endow svn-apply with the ability to reverse apply patches.
Comment 3 Sam Weinig 2017-07-25 17:03:43 PDT
Will this be different that the already existing svn-unapply?
Comment 4 Daniel Bates 2017-07-25 17:20:33 PDT
Comment on attachment 316393 [details]
Patch v1

R- as svn-unapply does this.
Comment 5 Matthew Hanson 2017-07-25 17:29:05 PDT
I was unaware that svn-unapply existed; after taking a look I think it will serve my needs just fine.