Merge pull request #1474 from pre-commit/unicode_error_py_launcher

asottile · web-flow · commit d71699d7cdd0 · 2020-05-27T15:02:41.000-07:00
avoid a UnicodeError on windows with non-charmap characters
diff --git a/pre_commit/languages/python.py b/pre_commit/languages/python.py
@@ -67,9 +67,10 @@ def _find_by_py_launcher(
 ) -> Optional[str]:  # pragma: no cover (windows only)
     if version.startswith('python'):
         num = version[len('python'):]
+        cmd = ('py', f'-{num}', '-c', 'import sys; print(sys.executable)')
+        env = dict(os.environ, PYTHONIOENCODING='UTF-8')
         try:
-            cmd = ('py', f'-{num}', '-c', 'import sys; print(sys.executable)')
-            return cmd_output(*cmd)[1].strip()
+            return cmd_output(*cmd, env=env)[1].strip()
         except CalledProcessError:
             pass
     return None
