Make rbt behave better if it can't find a remote branch for hg repositories.

Review Request #5926 — Created June 3, 2014 and submitted

Information

RBTools
master
49b14af...

Reviewers

Right now, if rbt can't find the remote branch, it errors out with the string
"CRITICAL: tuple index out of range". This is a terrible error.

There's some code in the mercurial client to allow using --tracking-branch (or
the associated config value) in the case that none of the expected branch names
are found, but --tracking-branch was listed under the "git options" only. I've
moved this option to the general repository options section and clarified how
it behaves for mercurial repositories.

I've then made it so that if the mercurial client can't find an upstream
branch, it will error out with a useful message telling people what went wrong
and how to fix it.

  • Ran unit tests
  • Created a bare hg repo with no remote branches, and ran 'rbt post'. Saw the
    expected error message about being unable to determine the upstream branch.
Description From Last Updated

'Option' imported but unused

reviewbotreviewbot
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        rbtools/commands/post.py
        rbtools/clients/mercurial.py
        rbtools/commands/__init__.py
        rbtools/commands/diff.py
      Ignored Files:
    
    
  2. 
      
reviewbot
  1. This is a review from Review Bot.
      Tool: Pyflakes
      Processed Files:
        rbtools/commands/post.py
        rbtools/clients/mercurial.py
        rbtools/commands/__init__.py
        rbtools/commands/diff.py
      Ignored Files:
    
    
  2. rbtools/commands/diff.py (Diff revision 1)
     
     
    Show all issues
     'Option' imported but unused
    
  3. 
      
david
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        rbtools/commands/post.py
        rbtools/clients/mercurial.py
        rbtools/commands/__init__.py
        rbtools/commands/diff.py
      Ignored Files:
    
    
  2. 
      
reviewbot
  1. This is a review from Review Bot.
      Tool: Pyflakes
      Processed Files:
        rbtools/commands/post.py
        rbtools/clients/mercurial.py
        rbtools/commands/__init__.py
        rbtools/commands/diff.py
      Ignored Files:
    
    
  2. 
      
chipx86
  1. Ship It!

  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-0.6.x (53ce43a)
Loading...