1978: diff support for Perforce moved files ("p4 move")

mattg******@gmai***** (Google Code) (Is this you? Claim this profile.)
chipx86
chipx86
Jan. 25, 2013
http://reviews.reviewboard.org/


What version are you running?

1.0.9

What's the URL of the page this enhancement relates to, if any?

reviewboard.host.server/r/123456/diff

Describe the enhancement and the motivation for it.

In release 2009.1 or thereabouts, Perforce added a first-class "move" operation, improving on the old integ-and-delete-and-edit way of moving files. Among other improvements, it can handle cases where you need to edit a file while moving it in the same change, and diff works reasonably for such files.

However, reviewboard diffs still show the file as entirely deleted, and entirely added, with none of the nice color highlighting for the regions that did/did not actually change.

What operating system are you using? What browser?

OS X 10.6, Chrome 9.

Please provide any additional information below.
chipx86
#1 chipx86
This is something I'd like to do at some point.
  • +Confirmed
  • +Component-DiffParser
    +Component-DiffViewer
chipx86
#2 chipx86
  • -Confirmed
    +PendingReview
  • -Priority-Medium
    +Priority-High
    +Milestone-Release1.7.x
  • +chipx86
chipx86
#3 chipx86
This is fixed and will be usable by 1.7.3 and RBTools 0.4.3.
  • -PendingReview
    +Fixed