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)
     
     

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

  3. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-0.7.x (984f5e6)
Loading...