Summary

Do unicode migration for rbtools/clients/

Review Request #6693 — Created Dec. 15, 2014 and submitted Dec. 16, 2014, 1:30 p.m.

Information

Owner

david

Repository

RBTools

Branch

master

Bugs

Depends On

Commit

749b67c...

Reviewers

Groups

rbtools

People

Description

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

Testing Done


Ran unit tests.
Posted changes with different SCMs.

Issues

Description	From	Last Updated
We should format this properly so the summary is one line.	chipx86	Dec. 15, 2014, 3:20 p.m.
Might be nicer with one param per line, starting on the dl.insert line. Looks like that would fit?	chipx86	Dec. 15, 2014, 3:20 p.m.

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

rbtools/clients/__init__.py (Diff revision 1)
The issue has been resolved. Show all issues
```
We should format this properly so the summary is one line.
```

rbtools/clients/perforce.py (Diff revision 1)

The issue has been resolved. Show all issues

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

Commit:

ccf310cb007741d592b15640955f0ca1d941ce57

749b67c66fec8febe14492470950a94528ea0cbe

Diff:

Revision 2 (+115 -105)

Show changes

	rbtools/clients/__init__.py
	rbtools/clients/bazaar.py
	rbtools/clients/cvs.py
	rbtools/clients/git.py
	rbtools/clients/perforce.py
	rbtools/clients/tests.py

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

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to master (83f66ea)