Skip to content

Commit e519fff

Browse files
Fix interpreter detection issues on windows
Non existing paths were added, empty interpreter should never happen, should be set to sys.executable
1 parent d901d33 commit e519fff

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

examples/pynotepad/pynotepad/settings.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,10 @@ def __init__(self):
1313

1414
@property
1515
def interpreter(self):
16-
return self.settings.value('interpreter', '')
16+
interpreter = self.settings.value('interpreter', '')
17+
if not os.path.exists(interpreter):
18+
interpreter = ''
19+
return interpreter
1720

1821
@interpreter.setter
1922
def interpreter(self, value):

examples/pynotepad/pynotepad/utils.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@ def get_interpreters():
1515
for path in paths:
1616
if 'python' in path.lower():
1717
if 'scripts' in path.lower():
18-
path = os.path.abspath(os.path.join(path, os.pardir))
19-
executables.add(os.path.join(path, 'python.exe'))
18+
path = os.path.abspath(os.path.join(path, os.pardir))
19+
executable = os.path.join(path, 'python.exe')
20+
if os.path.exists(executable):
21+
executables.add(executable)
2022
return executables

0 commit comments

Comments
 (0)