-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
OS X 32-bit wheels fail import in 1.15.0 #11625
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
Comments
I don't think we provide 32 bit wheels for OS X. There was a recent attempt to build fat binaries, but it failed. @matthew-brett Comment? |
Does this work for earlier NumPy wheels? |
Does Python.org use Intel? |
Also temporarily removing 32-bit tests from OS X; see build failure [2] where importing NumPy failed. I filed [3] with the team. Updating the build journal for CircleCI based on. It seems the order of the modules has changed for some reason. [1]: https://ci.appveyor.com/project/dhermes/bezier/build/1.0.974.master/job/iih82aprmi76e8hl [2]: https://travis-ci.org/dhermes/bezier/builds/408724560 [3]: numpy/numpy#11625 [4]: https://circleci.com/gh/dhermes/bezier/1307
IIRC this has never worked properly, even when we were distributing fat binaries as DMG installers. |
and I can say empirically that I have been running unit tests for
This is just the nice name for "fat" binaries on OS X. For the latest official 3.6:
the binary supports both architectures:
|
Hmm - this should work. In fact, I thought we were testing 32-bit numpy, see: https://github.com/MacPython/numpy-wheels/blob/master/config.sh#L38 |
The
Though it may just be a red herring. It seems the
but
This (i.e. shipping portable dylibs) mirrors the current approach that SciPy has. |
I suspect what's happening is the |
Good call @matthew-brett. The
but the
The issue is that the
@matthew-brett may remember I actually raised this issue (matthew-brett/delocate#23) with |
@dhermes - do you also need scipy to work on 32-bit? I could try and address this in delocate, but 32-bit is dying fast, and I'd rather just patch the build process to put in the 32-bit library, after the initial delocate run. |
No I don't have any need for 32-bit, was just trying to support users that might. I don't have a dog in the fight and am happy to stop building 32-bit wheels if they aren't deemed useful. (The existence of a 32-bit |
@matthew-brett any progress on this front.? |
Sorry - no progress from me. The options are:
The second is a bit of work, maybe an hour or two. But - my strong suspicion is that that (maybe literally) no-one is using the 32-bit part of the build. Worth an email to the mailing list to check? |
@matthew-brett Worth an email, but I suspect that the 1.15.0 release has reached a much larger audience and so far the only complaint has come from @dhermes, and he doesn't actually use the 32 bit builds. Is it possible to build 32 bit specific wheels? That might be a fallback option if someone really needs them. |
I don't think there are any pure 32-bit Pythons out there, and the only practical way to get a 32-bit executable running is with:
where
That means that they'll look for dual arch ( |
+1 for dropping 32-bit, there's at most a few handfuls of users - not worth the effort or even the space in the binaries |
Going to close this, the consensus seems to be for dropping 32 bit support on the Mac. I'll add that as comment to the 1.15.1 release notes. |
SGTM, now I have some packaging code to delete 😀 |
With the
intel
/ fatpython
binaries provided by the python.org installers, trying to import in 32-bit mode fails with:Similar errors occur in 2.7 and 3.5.
The text was updated successfully, but these errors were encountered: