Do unicode migration for rbtools/clients/

Review Request #6693 — Created Dec. 15, 2014 and submitted

Information

RBTools
master
749b67c...

Reviewers

This change updates the code in the rbtools/clients/ directory to have a
correct separation of unicode vs bytes, and use unicode literals everywhere.

  • Ran unit tests.
  • Posted changes with different SCMs.
Description From Last Updated

We should format this properly so the summary is one line.

chipx86chipx86

Might be nicer with one param per line, starting on the dl.insert line. Looks like that would fit?

chipx86chipx86
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/clients/cvs.py
        rbtools/clients/tests.py
        rbtools/clients/__init__.py
        rbtools/clients/bazaar.py
        rbtools/clients/git.py
        rbtools/clients/perforce.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/clients/cvs.py
        rbtools/clients/tests.py
        rbtools/clients/__init__.py
        rbtools/clients/bazaar.py
        rbtools/clients/git.py
        rbtools/clients/perforce.py
    
    
  2. 
      
chipx86
  1. 
      
  2. rbtools/clients/__init__.py (Diff revision 1)
     
     
     
     
     
    Show all issues

    We should format this properly so the summary is one line.

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

    Might be nicer with one param per line, starting on the dl.insert line. Looks like that would fit?

  4. 
      
david
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/clients/cvs.py
        rbtools/clients/tests.py
        rbtools/clients/__init__.py
        rbtools/clients/bazaar.py
        rbtools/clients/git.py
        rbtools/clients/perforce.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/clients/cvs.py
        rbtools/clients/tests.py
        rbtools/clients/__init__.py
        rbtools/clients/bazaar.py
        rbtools/clients/git.py
        rbtools/clients/perforce.py
    
    
  2. 
      
chipx86
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to master (83f66ea)
Loading...