[Python-checkins] cpython: Closes #29213: Merged fix from 3.6.

[vinay.sajip]

https://hg.python.org/cpython/rev/c23224ebc9e5
changeset:   106395:c23224ebc9e5
parent:      106391:7fd931727764
user:        Vinay Sajip <vinay_sajip at yahoo.co.uk>
date:        Thu Feb 02 19:33:43 2017 +0000
summary:
  Closes #29213: Merged fix from 3.6.

files:
  Lib/venv/__init__.py             |  10 ++++------
  Lib/venv/scripts/nt/Activate.ps1 |   2 +-
  2 files changed, 5 insertions(+), 7 deletions(-)


diff --git a/Lib/venv/__init__.py b/Lib/venv/__init__.py
--- a/Lib/venv/__init__.py
+++ b/Lib/venv/__init__.py
@@ -320,19 +320,17 @@
                 dstfile = os.path.join(dstdir, f)
                 with open(srcfile, 'rb') as f:
                     data = f.read()
-                if srcfile.endswith('.exe'):
-                    mode = 'wb'
-                else:
-                    mode = 'w'
+                if not srcfile.endswith('.exe'):
                     try:
                         data = data.decode('utf-8')
                         data = self.replace_variables(data, context)
-                    except UnicodeDecodeError as e:
+                        data = data.encode('utf-8')
+                    except UnicodeError as e:
                         data = None
                         logger.warning('unable to copy script %r, '
                                        'may be binary: %s', srcfile, e)
                 if data is not None:
-                    with open(dstfile, mode) as f:
+                    with open(dstfile, 'wb') as f:
                         f.write(data)
                     shutil.copymode(srcfile, dstfile)
 
diff --git a/Lib/venv/scripts/nt/Activate.ps1 b/Lib/venv/scripts/nt/Activate.ps1
--- a/Lib/venv/scripts/nt/Activate.ps1
+++ b/Lib/venv/scripts/nt/Activate.ps1
@@ -26,7 +26,7 @@
 }
 
 deactivate -nondestructive
-
+
 $env:VIRTUAL_ENV="__VENV_DIR__"
 
 if (! $env:VIRTUAL_ENV_DISABLE_PROMPT) {

-- 
Repository URL: https://hg.python.org/cpython