Skip to content

Commit c5242d5

Browse files
author
allegroai
committed
Use wexpect_venv instead of wexpect due to raczben/wexpect#26 (comment)
1 parent d0c040d commit c5242d5

File tree

2 files changed

+2
-13
lines changed

2 files changed

+2
-13
lines changed

clearml_session/__main__.py

+1-12
Original file line numberDiff line numberDiff line change
@@ -14,18 +14,7 @@
1414

1515
if sys.platform == 'win32':
1616
import msvcrt # noqa
17-
import wexpect as pexpect # noqa
18-
19-
if sys.prefix != sys.base_prefix:
20-
# running in venv
21-
from clearml.utilities.version import Version
22-
if Version(pexpect.__version__) < Version("4.0.1"):
23-
# Wexpect issue https://github.com/raczben/wexpect/issues/26#issuecomment-1574339194
24-
try:
25-
import wexpect_venv as pexpect
26-
except ImportError:
27-
print("WARNING: wexpect_venv is required to connect via SSH when running in a Windows virtualenv."
28-
" Please install using 'pip install wexpect_venv' in this virtual environment")
17+
import wexpect_venv as pexpect
2918
else:
3019
import select # noqa
3120
import pexpect # noqa

requirements.txt

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
clearml >= 1.1.5
22
pexpect ; sys_platform != 'win32'
3-
wexpect ; sys_platform == 'win32'
3+
wexpect_venv ; sys_platform == 'win32'
44
pillow>=10.0.1 # not directly required, pinned by Snyk to avoid a vulnerability

0 commit comments

Comments
 (0)