Allow specifying svn credentials on the command-line/.reviewboardrc file.

Review Request #6323 — Created Sept. 13, 2014 and submitted

Information

RBTools
release-0.6.x
f45e229...

Reviewers

Most people allow their svn credentials to be cached as part of the checkout,
but not everyone does. In this case, users need some way to be able to specify
them. This change adds --svn-username and --svn-password arguments (and
associated config file keys).

  • Ran unit tests.
  • Posted a change using these parameters.
Description From Last Updated

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

reviewbotreviewbot

Since _run_svn prepends 'svn' to the command, won't this run svn svn add ...?

brenniebrennie

Since _run_svn prepends 'svn' to the command, wouldn't this execute svn svn delete ...?

brenniebrennie

Help text should have trailing periods for consistency. Also, the documentation for all the commands need to be updated to …

chipx86chipx86
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/clients/svn.py
        rbtools/commands/__init__.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/clients/svn.py
        rbtools/commands/__init__.py
    
    
  2. rbtools/clients/svn.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (84 > 79 characters)
    
  3. 
      
brennie
  1. 
      
  2. rbtools/clients/svn.py (Diff revision 1)
     
     
    Show all issues

    Since _run_svn prepends 'svn' to the command, won't this run svn svn add ...?

  3. rbtools/clients/svn.py (Diff revision 1)
     
     
    Show all issues

    Since _run_svn prepends 'svn' to the command, wouldn't this execute svn svn delete ...?

  4. 
      
david
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        rbtools/clients/svn.py
        rbtools/commands/__init__.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        rbtools/clients/svn.py
        rbtools/commands/__init__.py
    
    
  2. 
      
brennie
  1. Ship It!

  2. 
      
chipx86
  1. 
      
  2. rbtools/commands/__init__.py (Diff revision 2)
     
     
     
     
     
     
     
    Show all issues

    Help text should have trailing periods for consistency.

    Also, the documentation for all the commands need to be updated to include this.

  3. 
      
david
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/clients/svn.py
        rbtools/commands/__init__.py
    
    Ignored Files:
        docs/rbtools/rbt/commands/post.rst
        docs/rbtools/rbt/commands/diff.rst
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/clients/svn.py
        rbtools/commands/__init__.py
    
    Ignored Files:
        docs/rbtools/rbt/commands/post.rst
        docs/rbtools/rbt/commands/diff.rst
    
    
  2. 
      
chipx86
  1. Ship It!

  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

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