Send the base commit ID for a change to the server when using Git.

Review Request #4398 — Created Aug. 5, 2013 and submitted

Information

RBTools
master

Reviewers

Send the base commit ID for a change to the server when using Git.

When uploading a change to a new enough Review Board server, the base
commit ID is sent along in the request. This ID/revision represents the
upstream commit that the diff (or its parent diff) applies to. It can be
used by the server to communicate with hosting services that otherwise
do not work solely with the content found in the diffs.
Tested with my Review Board change, and saw the base commit ID in the
database.

Posted this change on reviews.reviewboard.org, which doesn't have the
base commit ID support, and it didn't fail.
Description From Last Updated

Col: 33 E126 continuation line over-indented for hanging indent

reviewbotreviewbot
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        rbtools/commands/post.py
        rbtools/utils/diffs.py
        rbtools/api/resource.py
        rbtools/clients/git.py
      Ignored Files:
    
    
  2. rbtools/commands/post.py (Diff revision 1)
     
     
    Show all issues
    Col: 33
     E126 continuation line over-indented for hanging indent
    
  3. 
      
reviewbot
  1. This is a review from Review Bot.
      Tool: Pyflakes
      Processed Files:
        rbtools/commands/post.py
        rbtools/utils/diffs.py
        rbtools/api/resource.py
        rbtools/clients/git.py
      Ignored Files:
    
    
  2. 
      
david
  1. So you updated the git client, but the diff_changelist/diff_between_revisions/diff implementations for all the other clients also need to be changed to return dictionaries.
    1. The function that decides which of these to call is smart enough to handle the old tuple form.
      
      That said, I'll update, and I'll also fix post-review, which wasn't updated.
      
      (I guess I pushed at one point. Didn't mean to do that. I'll fix in a separate review request.)
  2. 
      
chipx86
Review request changed
Status:
Completed