-
-
Notifications
You must be signed in to change notification settings - Fork 627
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
Fix published wheel RECORDs. #18219
Fix published wheel RECORDs. #18219
Conversation
The hash format is fixed and a verification step added to prevent publishing invalid wheels going forward. Fixes pantsbuild#18111
I tested this out with a hacked up $ PY=python3.9 USE_PY39=true ./build-support/bin/release.sh publish
[=== 00:00 Releasing to PyPI and GitHub ===]
[=== 00:00 Fetching pre-built wheels for 2.16.0.dev6+git2983f166 ===]
Saving to /home/jsirois/dev/pantsbuild/jsirois-pants/dist/deploy.
Fetching https://binaries.pantsbuild.org/wheels%2Fpantsbuild.pants%2F2983f166e18b1c1a480b85bf722d16d514b41b17%2F2.16.0.dev6%2Bgit2983f166%2Findex.html
...
Fetching https://binaries.pantsbuild.org/wheels%2Fpantsbuild.pants%2F2983f166e18b1c1a480b85bf722d16d514b41b17%2F2.16.0.dev6%2Bgit2983f166%2Fpantsbuild.pants.testutil-2.16.0.dev6%2Bgit2983f166-py3-none-any.whl
Unpacking to: /tmp/tmp7ohr1anb/check-wheel/pantsbuild.pants-2.16.0.dev6...OK
Wrote whl with version 2.16.0.dev6 to /home/jsirois/dev/pantsbuild/jsirois-pants/dist/deploy/wheels/pantsbuild.pants/2983f166e18b1c1a480b85bf722d16d514b41b17/2.16.0.dev6/pantsbuild.pants-2.16.0.dev6-cp37-cp37m-macosx_10_15_x86_64.whl.
...
Unpacking to: /tmp/tmpxucwf4nn/check-wheel/pantsbuild.pants-2.16.0.dev6...OK
Wrote whl with version 2.16.0.dev6 to /home/jsirois/dev/pantsbuild/jsirois-pants/dist/deploy/wheels/pantsbuild.pants/2983f166e18b1c1a480b85bf722d16d514b41b17/2.16.0.dev6/pantsbuild.pants-2.16.0.dev6-cp38-cp38-manylinux2014_x86_64.whl.
Unpacking to: /tmp/tmpf63z1yx0/check-wheel/pantsbuild.pants-2.16.0.dev6...OK
Wrote whl with version 2.16.0.dev6 to /home/jsirois/dev/pantsbuild/jsirois-pants/dist/deploy/wheels/pantsbuild.pants/2983f166e18b1c1a480b85bf722d16d514b41b17/2.16.0.dev6/pantsbuild.pants-2.16.0.dev6-cp38-cp38-manylinux2014_aarch64.whl.
... |
Should we cherry-pick to 2.15.x so the release wheels there can be valid? |
Yeah, I think that makes sense. Coming up. |
The hash format is fixed and a verification step added to prevent publishing invalid wheels going forward. Fixes pantsbuild#18111 (cherry picked from commit 817db43)
The hash format is fixed and a verification step added to prevent
publishing invalid wheels going forward.
Fixes #18111