Fix PyLint presbumit check on win.

Fix PyLint presbumit check on win.

We were passing a unicode string in the env block to subprocess.Popen which makes it unhappy. This forces the string to ascii first.

BUG=NONE
TEST=NONE


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=183569

[robertshield, 2013-02-19 20:57:02]

In case you saw a couple of git cl crash dumps in PyLint come by recently, this
patch fixes them.

[M-A Ruel, 2013-02-19 21:07:40]

https://codereview.chromium.org/12255083/diff/2001/presubmit_canned_checks.py
File presubmit_canned_checks.py (right):

https://codereview.chromium.org/12255083/diff/2001/presubmit_canned_checks.py...
presubmit_canned_checks.py:680: extra_paths_list + sys.path).encode('ascii',
'ignore')
utf8
technically we'd want the code page but utf8 is close enough most of the time.
then remove 'ignore'

[robertshield, 2013-02-20 15:23:37]

ptal

https://codereview.chromium.org/12255083/diff/2001/presubmit_canned_checks.py
File presubmit_canned_checks.py (right):

https://codereview.chromium.org/12255083/diff/2001/presubmit_canned_checks.py...
presubmit_canned_checks.py:680: extra_paths_list + sys.path).encode('ascii',
'ignore')
On 2013/02/19 21:07:40, Marc-Antoine Ruel wrote:
> utf8
> technically we'd want the code page but utf8 is close enough most of the time.
> then remove 'ignore'

Done.

[Marc-Antoine Ruel (Google), 2013-02-20 18:05:08]

lgtm

[commit-bot: I haz the power, 2013-02-20 18:06:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/robertshield@chromium.org/12255083/5001

[commit-bot: I haz the power, 2013-02-20 18:08:48]

Change committed as 183569