-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
- added dependancy for wheel to setup_requires #21855
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
Conversation
xref #21819 I am still a bit concerned about this,
works for me without wheel installed, even if it includes the ominous message
. If I install
from it works and if I remove @franzhaas Why are you trying to build the wheel like this? Looking through our CI, it looks like we explicitly install This leads to conclude that while we do not strictly need to depend on wheel as a build-time dependency, in 3/4 of our CI systems we have installed it anyway, there is no harm in adding it as a build-time dependency (and we fix @franzhaas 's use case to boot). Can you also add a line to https://github.com/matplotlib/matplotlib/blob/cced93b1ee29e0653f70e96ffb90ec2cc1341c8d/doc/devel/dependencies.rst (which notable we missed doing when we added setuptool-scm, that is our mistake 🐑 ) |
I agree that wheel is not necessary as part of |
@franzhaas Why are you trying to build the wheel like this? This is because for my environment i build a bunch of wheels from source. (although normaly i take the wheel of matplotlib from pypi) Having a standard python installed and run this one line for every wheel i intend to use is convinient. Once I start preparing the build environment it gets involved. I have to document how to do it, where to place them, and what is in it, and that propably ends up varying over time, and over wheels. So buttom line.: Thanks a lot, |
Well I'd say |
You should probably stop using |
I believe this PR is not going anywhere. It only affects a specific way to build wheels, which is very niche, there are alternatives available and documented. The current situation is plenty good enough, the proposed improvement is minor or maybe even disputed, i close this PR. |
Thank you for being understanding @franzhaas . I do see where you are coming from here, but agree with @dopplershift that if you know you want wheels, making sure your standard Python environment has and up-to-date pip + wheel installed is reasonable expectation. Hopefully we will hear from you again! |
PR Summary
Dear all
py -m pip wheel git+https://github.com/matplotlib/matplotlib.git#egg=matplotlib
failed for me with the error.:
error: invalid command 'bdist_wheel'
this is due to the lack of the wheel dependency for setup_requires
This request was previously bundled with a request to remove the setuptools requirements.
Thanks!
Franz