-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Enable Manylinux wheel builds #1057
Conversation
…nux/non-manylinux
Enable wheel task for 10.14 mac Add delocate, fix wheel script Add pip install wheel to mac run add dependencies and fix py2 wheel fix py2
updates:
|
Do you all plan to get yourselves synced up with pyarrow mainline? |
26a60cd
to
4e285cb
Compare
is_libpsp defaults to true print try catch islibpsp write test script go from wheelhouse only on py3 dist and build from wheelhouse
@texodus this should be good to go |
@wesm the custom arrow build we have for Emscripten might take a little fixing, but it shouldn't be too hard to get Perspective on 0.17 in the near future |
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.
Looks good! Reviewed copiously offline.
This PR makes Perspective's linux wheels fully Manylinux2010 and Manylinux2014 compatible, adds wheel build tasks for MacOS 10.14 (Mojave) and MacOS 10.15 (Catalina), as well as ensures passage of the
conda-forge
Mac build.Changelog
auditwheel
CMakeLists
to allow full Manylinux builds to pass due to a difference betweenPYTHON_INCLUDE_DIRS
andPython_INCLUDE_DIRS
.auditwheel show
andauditwheel repair
whenever Linux wheels are built, which ensures that all wheels are fully portable and compliant to the Manylinux PEPs.CLOCK_MONOTONIC
fromcompat_impl_osx
, which is breaking builds onconda-forge
. This is due toclock_gettime
andCLOCK_MONOTONIC
not being implemented in MacOS until 10.12. Because the method that callsclock_gettime
is not used anywhere in the compiled code, I commented out the original implementation and forced the offending method to return a 0.