Skip to content
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

Tell Wheel to generate platform and interpreter specific wheels #194

Merged
merged 1 commit into from
Aug 13, 2015

Conversation

dstufft
Copy link
Contributor

@dstufft dstufft commented Aug 13, 2015

By telling wheel that we have extension modules, even though we have none, wheel will create a Wheel which is platform and interpreter specific. This will ensure that the pure Python wheels on PyPy do not trigger installs on CPython without the C speedups.

By telling wheel that we have extension modules, even though we
have none, wheel will create a Wheel which is platform and
interpreter specific. This will ensure that the pure Python wheels
on PyPy do not trigger installs on CPython without the C speedups.
@dstufft
Copy link
Contributor Author

dstufft commented Aug 13, 2015

This generates wheels that have a filename like SQLAlchemy-1.1.0b1.dev0-pp27-none-macosx_10_11_x86_64.whl, which broken down says SQLAlchemy version 1.1.0b1.dev0 for PyPy which supports Python 2.7 (pp is the short tag for PyPy) with no ABI restrictions on the OSX 10.11 x86_64 platform.

@@ -111,6 +123,8 @@ def run_setup(with_cext):
kwargs = {}
if with_cext:
kwargs['ext_modules'] = ext_modules
else:
kwargs['ext_modules'] = []
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I probably should have commented this, but just to note, this is required because when distutils sees that this project has ext modules (the trick from above) it expects there to be a list here instead of None, however it accepts a zero length list.

@zzzeek zzzeek merged commit 39517d1 into zzzeek:master Aug 13, 2015
@dstufft dstufft deleted the wheel-on-pypy branch August 13, 2015 19:06
@zzzeek
Copy link
Owner

zzzeek commented Aug 13, 2015

cool thanks!

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