Fix changes to lines starting with '-- ' with git-svn and git-p4.

Review Request #6829 — Created Jan. 23, 2015 and submitted

Information

RBTools
release-0.6.x
56cdd89...

Reviewers

When we are using git-svn or git-p4, we post-process the diff to change it from
something that looks like a git diff to something that looks like a subversion
diff or perforce diff.

This works most of the time, but our handling of the index lines had some
unintended consequences. If the file in question had a change to a line
starting with two hyphens and a space ('-- '), then the unified diff will
represent that as '--- old text / '+-- new text'. Since we only looked for
'--- filename' when scanning for file headers, we'd modify these lines that
were actually content, and the patch would fail to apply. I've changed this to
make sure that we only munge a line starting with '--- ' if the line after it
starts with '+++ '.

It's still possible to break this by changing a single line from '-- ...' to
'++ ...' but that should hopefully be a vanishingly small case, and I suspect
it will also break every other tool that handles diff files.

Manually tested git-svn and git-p4 cases. We don't have any unit tests for
these setups. Ran existing unit tests.

reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/clients/git.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/clients/git.py
    
    
  2. 
      
brennie
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

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