Add a better error message when using --revision-range on Perforce repository

Review Request #2614 — Created Sept. 24, 2011 and submitted

Information

RBTools

Reviewers

When using post-review in a Perforce repository, if users included the --revision-range parameter, post-review would spin out with a confusing error message.

This patch displays something a little more useful, including a link to Review Board's documentation on Perforce's range path syntax.
Added a quick test to ensure that using --revision-range on a Perforce repository causes us to exit out.
Description From Last Updated

Can you add a docstring for this?

daviddavid
david
  1. 
      
  2. rbtools/clients/tests.py (Diff revision 1)
     
     
    Show all issues
    Can you add a docstring for this?
  3. 
      
mike_conley
david
  1. Ship It!
    1. Thanks - committed as 2f76966736
  2. 
      
mike_conley
Review request changed

Status: Closed (submitted)

Loading...