Ignore MSYS's Python // Resolve #439 Resolve #488

platformio · Jun 3, 2017 · bf51881 · bf51881
1 parent fae0a10
commit bf51881
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/installer/helpers.js b/lib/installer/helpers.js
@@ -147,7 +147,7 @@ export async function getPythonExecutable(customDirs = null) {
   }
 
   for (const executable of candidates.values()) {
-    if (!executable.includes('msys') && (await isPython2(executable))) {
+    if (await isPython2(executable)) {
       return executable;
     }
   }
@@ -156,7 +156,7 @@ export async function getPythonExecutable(customDirs = null) {
 }
 
 function isPython2(executable) {
-  const args = ['-c', 'import sys; print \'.\'.join(str(v) for v in sys.version_info[:2])'];
+  const args = ['-c', 'import sys; assert "msys" not in sys.executable.lower(); print ".".join(str(v) for v in sys.version_info[:2])'];
   return new Promise(resolve => {
     utils.runCommand(
       executable,