Fix handling of basedir in 'rbt patch'

Review Request #5855 — Created May 21, 2014 and submitted

Information

RBTools
master
51e5f5f...

Reviewers

Depending on the version of the Review Board server and the particular
repository type, the returned diffset object from the API may have no basedir,
may have a basedir of None, or may be a string. In the first two cases, we want
to normalize this to an empty string.

This also handles a None repository base_path.

  • Ran 'rbt patch'
  • Ran unit tests.
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        rbtools/commands/patch.py
      Ignored Files:
    
    
  2. 
      
reviewbot
  1. This is a review from Review Bot.
      Tool: Pyflakes
      Processed Files:
        rbtools/commands/patch.py
      Ignored Files:
    
    
  2. 
      
chipx86
  1. Ship It!

  2. 
      
david
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        rbtools/commands/patch.py
        rbtools/clients/__init__.py
      Ignored Files:
    
    
  2. 
      
reviewbot
  1. This is a review from Review Bot.
      Tool: Pyflakes
      Processed Files:
        rbtools/commands/patch.py
        rbtools/clients/__init__.py
      Ignored Files:
    
    
  2. 
      
chipx86
  1. Ship It!

  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-0.6.x (09ee08e)
Loading...