• 
      

    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:
    Completed
    Change Summary:
    Pushed to release-0.6.x (a2890ae)