-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Pep518 binaries #4800
Pep518 binaries #4800
Conversation
This feature is unused and could result in undesired effects (eg. rebuilding numpy while installing build dependencies). We drop the feature here.
The documentation for
However, there is no wheel available for the @pfmoore Should I open a bug report about that? |
There's the test failure we're looking for. |
For reference the modification of the binary should be changing this:
To this:
|
"This feature is unused". You don't say what feature, other than the vague title "Pep518 binaries". But don't worry. You've seen my most recent comment on #4799, so let's wait till we get agreement on whether PEP 518 should change, or whether the recursive behaviour is required. |
Yes, will discuss there. |
Sorry, please discuss in #4799. |
This feature is unused and could result in undesired
effects (eg. rebuilding numpy while installing build
dependencies). We drop the feature here.
Edit: I should also mention that recursive behavior (the existing implementation) is extremely bug-prone and can timeout in some cases without warning.