Git client: properly handle deleted files when making SVN-compatible diff

Review Request #6396 — Created Oct. 2, 2014 and submitted

Information

RBTools
master
d0a7790...

Reviewers

When deleting a file, git-diff outputs "+++ /dev/null" and "deleted file mode", neither of which are understood by SVN. Rather, SVN expects the new filename to be the same as the deleted one. This is important when doing file renames or deletions when using git-svn.

Successfully used this code to post a review for a basic set of file renames (~10 files). Posted to RB 1.6 and it seemed to be happy.

Description From Last Updated

Col: 80 E501 line too long (90 > 79 characters)

reviewbotreviewbot
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/clients/git.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/clients/git.py
    
    
  2. rbtools/clients/git.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (90 > 79 characters)
    
  3. 
      
JR
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/clients/git.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/clients/git.py
    
    
  2. 
      
david
  1. Ship It!

  2. 
      
JR
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-0.6.x (ffba29b)
Loading...