Circumvented issue with installing pywinpty on py38; Added py38/win/l…

…atest Signed-off-by: Andreas Maier <[email protected]>
zhmcclient · Dec 29, 2024 · 768b6af · 768b6af
1 parent 94f2337
commit 768b6af
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -103,6 +103,11 @@ jobs:
                 \"python-version\": \"3.8\", \
                 \"package_level\": \"minimum\" \
               }, \
+              { \
+                \"os\": \"windows-latest\", \
+                \"python-version\": \"3.8\", \
+                \"package_level\": \"latest\" \
+              }, \
               { \
                 \"os\": \"windows-latest\", \
                 \"python-version\": \"3.13\", \

diff --git a/changes/noissue.6.fix.rst b/changes/noissue.6.fix.rst
@@ -0,0 +1,2 @@
+Circumvented an issue when installing pywinpty 2.0.14 with latest version of
+maturin on Python 3.8, by excluding pywinpty 2.0.14.
diff --git a/dev-requirements.txt b/dev-requirements.txt
@@ -175,7 +175,8 @@ pip-check-reqs>=2.5.1; python_version >= '3.9'
 
 # pywinpty is used by terminado <- notebook <- jupyter
 # pywinpty>=1.0 requires maturin to build. Meanwhile, it works.
-pywinpty>=2.0.12; os_name == "nt"
+# pywinpty 2.0.14 has an issue with latest maturin, see https://github.com/andfoy/pywinpty/issues/486
+pywinpty>=2.0.12,!=2.0.14; os_name == "nt"
 
 # pytz is actually covered in requirements.txt, but we need to repeat it here
 # because development packages pull it in, so the exclusion of 2024.2 is active
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		Circumvented an issue when installing pywinpty 2.0.14 with latest version of
		maturin on Python 3.8, by excluding pywinpty 2.0.14.