-
Notifications
You must be signed in to change notification settings - Fork 156
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
feat: produce ABI-agnostic wheels for python #1324
Conversation
Signed-off-by: Frost Ming <mianghong@gmail.com>
Signed-off-by: Frost Ming <mianghong@gmail.com>
Signed-off-by: Frost Ming <mianghong@gmail.com>
Thanks a lot! We have reached the storage limit of pypi several times, and this could greatly help reduce our storage usage. |
@@ -110,5 +119,9 @@ def get_version(): | |||
ext_modules=[ | |||
EnvdExtension(name="envd", sources=["cmd/*"]), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The example provides two arguments define_macros=[("Py_LIMITED_API", "0x03060000")]
and py_limited_api=True
, whether we need to overwrite the init of EnvdExtension
to include them?
https://setuptools.pypa.io/en/latest/userguide/ext_modules.html#setuptools.Extension
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i don't think so, there is no python api involved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think so. The original example is for the abi3-compatible package. For envd, we don't rely on any python abi or version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it, seems fine
Co-authored-by: Jinjing Zhou <VoVAllen@users.noreply.github.com> Signed-off-by: Frost Ming <mianghong@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Signed-off-by: Frost Ming mianghong@gmail.com
The approach is taken from cibuildwheel Tips and Tricks, also quote:
This will reduce the number of wheels to upload to PyPI