• 
      

    Add --non-interactive to SVN command lines.

    Review Request #7308 — Created May 16, 2015 and submitted

    Information

    RBTools
    release-0.7.x
    af4282a...

    Reviewers

    When the username and password are passed in to SVN, it can sometimes prompt
    the user for whether or not they want to save the password permanently (and
    likely make other prompts). This change adds --non-interactive to all SVN
    command lines to make sure that nothing gets blocked waiting for user input.

    Ran unit tests.

    Description From Last Updated

    This will end up running svn --non-interactive log --non-interactive.

    brenniebrennie
    reviewbot
    1. Tool: PEP8 Style Checker
      Processed Files:
          rbtools/clients/svn.py
      
      
      
      Tool: Pyflakes
      Processed Files:
          rbtools/clients/svn.py
      
      
    2. 
        
    chipx86
    1. Looks fine in general. Trying to think if there are any cases where this is going to cause problems, though. Such as verification of an HTTPS certificate or SSH host fingerprint.

      1. Because we pipe stdin and stdout, there's really no chance of interactivity in svn commands actually being useful anyway.

    2. 
        
    brennie
    1. One nit and then it looks good to me.

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

      This will end up running svn --non-interactive log --non-interactive.

    3. 
        
    david
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-0.7.x (984f5e6)