Skip to content

Conversation

Xmader
Copy link
Member

@Xmader Xmader commented Aug 23, 2024

Otherwise pip would fallback to compile from source.

A wheel package file will only be selected by pip to install if the platform tag satisfies, regardless of whether the binary compatibility actually is.


The wheel package we build for arm64 is currently named pythonmonkey-0.x.x-cp310-cp310-macosx_14_0_arm64.whl. According to https://packaging.python.org/en/latest/specifications/binary-distribution-format/#file-format and https://packaging.python.org/en/latest/specifications/platform-compatibility-tags/, its platform tag is macosx_14_0_arm64, meaning it will only be selected by pip to install on macOS version 14.0 and above, regardless of whether the binary compatibility actually is.

If there's no binary compatibility issue (like we saw on Ubuntu about the glibc versions) on macOS, renaming the file to something like pythonmonkey-0.x.x-cp310-cp310-macosx_11_0_arm64.whl will let pip to install a prebuilt version of pythonmonkey rather than build from source.

Xmader added 5 commits August 23, 2024 20:03
…f macOS, otherwise pip would fallback to compile from source

Change the platform tag part of the wheel filename to `macosx_11_xxx` (means to support macOS 11).

A wheel package file will only be selected by pip to install if the platform tag satisfies, regardless of whether the binary compatibility actually is.

See https://packaging.python.org/en/latest/specifications/binary-distribution-format/#file-format
@Xmader Xmader requested a review from zollqir August 27, 2024 09:47
@Xmader Xmader added this to the v1.0.0 milestone Sep 6, 2024
Copy link
Collaborator

@zollqir zollqir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zollqir zollqir merged commit 6d9a396 into main Sep 6, 2024
7 checks passed
@zollqir zollqir deleted the Xmader/chore/wheel-macos-version-support branch September 6, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants