-
Notifications
You must be signed in to change notification settings - Fork 20
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
About wheel_repair.py for Windows #3
Comments
I thought about it but didn't do it because the script has some limitations right now (which doesn't affect my very simple extension):
I would love to hear about your use-case before thinking about packaging this. Do you have multiple PYDs in the wheel you're trying to build, are you using vcpkg, and what are the build systems behind C/C++ code in your extension, etc. We can also get on a call if you're available. |
Hello, @zooba, you seem literate in this topic, would you have thoughts to share, if time allows it? @vinayak-mehta So since I want to avoid For now, this not yet cibuildwheel-enabled windows Github Action job log shows 2 artifacts which are :
|
Not sure what thoughts you want me to have... I dislike this entire approach, regardless of operating system, and would rather people used stable/forward-compatible APIs. I don't want to have to debug the potential resource and name conflicts, and it bothers me that we take this approach to try and automatically fix issues that either need to be fixed by having conflicting packages communicate with each other, or by establishing package demands in a way that distributors and users can resolve it. Those probably weren't what you were hoping for :) Might have to be more specific |
I must agree, if I understand well, that the machomangling step seems perfectionist to me... What I wished in a repair tool for Windows was just: 1) inspect the needed third-party .dll files, 2) put them in the wheel archive |
Hello I have not tried your CLI wheel repair script yet, but it is fresher and slightly more advanced than machomangler which it wraps and leverages.
Have you thought about seperating it into a pypi.org package, maybe with a more windows-related name ?
Related cibuildwheel Windows wheel repair issue: pypa/cibuildwheel#459
The text was updated successfully, but these errors were encountered: