Use {env}/bin instead of {env}/Scripts when Python is from MSYS2 …

…on Windows.

MSYS2 prefers a Linux filesystem layout, but is otherwise a native
Windows interpreter.

Fixes tox-dev#1982

CONTRIBUTORS

@@ -48,6 +48,7 @@ Ionel Maries Cristian
 Itxaka Serrano
 Jake Windle
 Jannis Leidel
+Jesse Schwartzentruber
 Joachim Brandon LeBlanc
 Johannes Christ
 John Mark Vandenberg

docs/changelog/1982.bugfix.rst

@@ -0,0 +1,3 @@
+Distinguish between normal Windows Python and MSYS2 Python when looking for
+virtualenv executable path.  Adds os.sep to :class:`InterpreterInfo`.
+- by :user:`jschwartzentruber`

src/tox/config/__init__.py

@@ -1053,6 +1053,11 @@ def get_envbindir(self):
             isinstance(self.python_info, NoInterpreterInfo)
             or tox.INFO.IS_WIN is False
             or self.python_info.implementation == "Jython"
+            or (
+                # this combination is MSYS2
+                tox.INFO.IS_WIN
+                and self.python_info.pathsep == "/"
+            )
             or (
                 tox.INFO.IS_WIN
                 and self.python_info.implementation == "PyPy"

src/tox/helper/get_version.py

@@ -1,6 +1,7 @@
 from __future__ import unicode_literals
 
 import json
+import os
 import platform
 import sys
 
@@ -11,6 +12,7 @@
     "version": sys.version,
     "is_64": sys.maxsize > 2 ** 32,
     "sysplatform": sys.platform,
+    "pathsep": os.sep,
     "extra_version_info": getattr(sys, "pypy_version_info", None),
 }
 info_as_dump = json.dumps(info)

src/tox/interpreters/__init__.py

@@ -103,6 +103,7 @@ def __init__(
         version_info,
         sysplatform,
         is_64,
+        pathsep,
         extra_version_info,
     ):
         self.implementation = implementation
@@ -111,6 +112,7 @@ def __init__(
         self.version_info = version_info
         self.sysplatform = sysplatform
         self.is_64 = is_64
+        self.pathsep = pathsep
         self.extra_version_info = extra_version_info
 
     def __str__(self):

tests/unit/interpreters/test_interpreters.py

@@ -192,7 +192,9 @@ def info(
         version_info="my-version-info",
         sysplatform="my-sys-platform",
     ):
-        return InterpreterInfo(implementation, executable, version_info, sysplatform, True, None)
+        return InterpreterInfo(
+            implementation, executable, version_info, sysplatform, True, "/", None
+        )
 
     def test_data(self):
         x = self.info("larry", "moe", "shemp", "curly")