-
Notifications
You must be signed in to change notification settings - Fork 810
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unfreeze setuptools version again, workaround #1936
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for following up here, but I don't understand one of the changes.
@@ -46,13 +43,6 @@ jobs: | |||
run: | | |||
python setup.py bdist_wheel --skip-build | |||
|
|||
# 3.10 dropped support for bdist_wininst. If we had both 3.9 and 3.10 available in a single |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this change? It means we don't get .exe installers from CI.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The bdist_wininst has been completely removed recently from setuptools /distutils which is effective with current version of setuptools. (Commit message of 4. patch: Remove bdist_wininst from CI (gone in setuptools / distutils) -
Removed now in pypa/distutils@e28b6193 )
So its dead until some independent project would continue bdist_wininst or bdist_msi .
On the build machine you may use an older version of setuptools for some time if needed, or somehow fall back to deprecated legacy distutils in python (where the stub .exe were taken from anyway via setup.py/maybe_fixup_exes() ) ... ?
Or end .exe installers completely?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry the the delay and thanks for the explanation! Download stats of the .exe from github are quite minor compared to pypi, so I think I'll open an issue to phase them out, but might as well do that as late as practical. My local environment can still .exe installers, so 👍 and thanks!
after freeze in a22bd62. The workaround should become obsolete (silently) when distutils / setuptools is fixed midterm. Fix in Scintilla/makefile_pythonwin: nested nmake execution did not return errors (return code)
Newer distutils / setuptools versions already use and expose the MSVC & SDK dirs - taken from vcvarsall.bat . We need and should not use a different mechanism anymore. (Include & lib dirs of 2 different SDK kits at the same time were seen on compiler command lines on github CI - for the last months at least.)
686e0e1
to
b2110ac
Compare
No description provided.