-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Release 20.8b1 Missing Binary #1669
Comments
20.8b0 is also missing a release binary: https://github.com/psf/black/releases/tag/20.8b0 |
You can find sources and wheels on PyPI, for at least both of those releases: PyPI is the usual place to distribute Python packages, rather than GitHub Releases. |
It was really nice to have the self enclosed binaries... not requiring a python environment really helped using this in CI. |
Ah, do you mean like the |
I'm not sure where that file comes from, @ambv did you do anything to generate it? It's an ELF file that works on Linux but I wasn't aware that we had anything in this repo that would build such a file. |
Perhaps built with Pyinstaller or similar? |
@hugovk Maybe they've been removed (for reasons that I don't know about) since you posted this, but neither cc @kratsg |
They were removed due to a build error causing them to be out of date. We're hoping to have that fixed for 20.9 release coming the end of this month. |
Great! Thanks very much for the helpful and fast reply. 🙇 |
@JelleZijlstra, #362 asked for binary releases so Black could be run conveniently and without needing CPython 3.6+ set up. Linux and Windows binaries started to be automatically created and uploaded after their respective PRs were merged, #410 and #422.
I personally want to see platform specific binaries continue to be available as it enables more people to use Black without much cost. They were already created and uploaded automatically. Maybe this time we should use GitHub Actions instead of Travis CI + AppVeyor. |
@ichard26 I might point you to the very nice work that @henryiii has done for Scikit-HEP to get this working very nicely with Please let us know if you have any questions! |
I think this could easily be a "universal" wheel (name may be slightly out of date now that it's Python 3 only), I don't think it needs any "true" binaries. There is a MyPyC extension, which could probably be built using cibuildwheel, though you could also provide a pure Python wheel too, as a fallback. I'm not sure how important / useful that is (or really what it is). (Unless there really are compiled components that I didn't notice on quick glance) |
I'd also recommend deploying to TestPyPI on (say) merges to master, that way the deploy mechanism is also regularly tested so there's no surprises on release day. |
@matthewfeickert, I think @ichard26 is referring to self contained, binary packages, while you and several of us are referring to wheels on PyPI. The developer instructions discuss downloading artifacts from any CI build, including PRs, so release day shouldn't be a surprise. :) |
@matthewfeickert thanks for the information! I'm a bit lost though as cibuildwheel seems to builds edit: oh thanks @henryiii, we posted at the same time :) |
Level 0: SDist We are currently missing level 1 (and also level 10, I gather) |
We already provide Pure Python wheels, but a mistake was made during build so the wheels were yanked. @ cooperlees already addressed this:
As said already, the next release should have PEP 427 binaries unless another mistake is made :) |
Ah yes, that's really handy for testing that the artifacts are generated properly. I still suggest uploading to TestPyPI to keep the upload mechanism itself well oiled :) |
https://pypi.org/project/black/#files -- there still isn't a binary for the latest release. |
@pradyunsg thanks for the reminder but this issue is for self-contained binaries that don't need a Python installation to run. |
Right, I understand that; except that isn't what the OP is about here.
|
Given how popular For details see: https://pythonwheels.com/ |
While having independent binaries is great, not having a wheel uploaded on PyPI is not. My understanding was that this issue is tracking the re-upload of a wheel, or a new release with a wheel. I'm not sure why this was closed because independent binaries are built in CI now. |
Honestly I don't even understand, I'm confused at this point :/
Yes, there are comments about wheels here, like #1669 (comment), but those ones aren't from the OP. I'm very sorry if I'm the one being dumb here and completely misreading the situation (totally possible!) but I don't believe wheels are what the OP was talking about. Also, regardless of that, I'd like to share my appreciation for the consistent reminders for us to publish a release with wheels. I share and understand your frustration. Finally, I'm just going to open a separate issue asking for a release with wheels. |
FWIW, the wheels were uploaded and deleted subsequently. I'd post a link, but I'm on my mobile and Github search isn't surfacing the discussion that led to the deletion of the wheel. That said, I think making a new black release would be a good idea overall, and adopting a regular release cadence (like pip, especially given that you also follow calver) would be great! |
done see GH-2125
The discussion was via private channels among the core team but I can post the snippets relevant (although I should probably ask for permission from them, but I'm ambivalent about doing so due to some mixed feelings :/) IIRC it boiled down to this:
Although I'm not entirely sure what the deletion was all about since apparently Python favours packages over modules in case of a conflict (which is good since only the module version of black in the wheels were outdated). |
Describe the bug A clear and concise description of what the bug is.
The most recent release only contains source code packages, no binaries. https://github.com/psf/black/releases/tag/20.8b1
To Reproduce Steps to reproduce the behavior:
Expected behavior A clear and concise description of what you expected to happen.
I expected a binary to be available, like previous releases.
Environment (please complete the following information):
N/A
Does this bug also happen on master? To answer this, you have two options:
N/A
Additional context Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: