-
Notifications
You must be signed in to change notification settings - Fork 812
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
_fixup_sdk_dirs in setup.py will become redundant #1897
Comments
Thanks for the heads-up Steve! |
pywin32's build will now fail unless setuptools is fairly recent, so once that lands, I'll just force a requirement on that new version. I'm sure you can imagine how much I want to kill some of the crazy stuff setup.py does :) |
#1936 includes removal of the _fixup_sdk_dirs() mechanism. |
kxrob
added a commit
to kxrob/pywin32
that referenced
this issue
Aug 22, 2022
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.)
This was referenced Aug 22, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Just as an FYI, I submitted the patch for distutils/setuptools that will make this hack in your setup.py redundant:
pywin32/setup.py
Line 530 in e1c0237
The PR is at pypa/distutils#153 (and should go into
setuptools
after that) for reference.I haven't tried, but I believe the code in your setup.py will still work, and none of the asserts will trigger. So you may not notice anything at all (unless you happen to see that there are twice as many include/lib directories in the compiler commands). There's no good way to detect that it's running with a fixed setuptools, so I guess just be aware that if the asserts do start triggering, it may be intentional and they can be relaxed a bit.
The text was updated successfully, but these errors were encountered: