Skip to content
This repository has been archived by the owner on Feb 15, 2023. It is now read-only.

DEBUG only: MacOS x86_64 Python 3.10 wheel attempts #136

Closed

Conversation

tylerjereddy
Copy link
Collaborator

  • debug splitoff of WIP, MAINT: backport 3.10 support #135 where the first commit cuts out most of the CI except MacOS Azure
  • the second commit is for iterating on MacOS x86_64 Python 3.10 SciPy wheel build

* try to backport some of the pertinent
changes for Python 3.10 support to the wheels
repo `v1.7.x` branch, which needs to continue
supporting Python 3.7

* perhaps we can do away with some of the manylinux
bumps--not sure on those--things are going to change
a fair bit anyway with a huge bump in OpenBLAS version
as well; perhaps the bumps could be confined to just 3.10
using newer manylinux??

* point `BUILD_COMMIT` at the tip of the target maintenance
branch to get more useful feedback on where we stand
for the 3.10 wheel builds
* allow azure pipelines to run in PRs against
the v1.7.x branch (at least for debugging purposes..)
* bump the manylinux versions for some Azure
builds to see if we can get around the Python 2.4
errors

[skip travis]
* we're not ready to ship `universal2` wheels for Python 3.8
and 3.9, so remove those backport additions; for Python 3.10
`universal2` is apparently our only option (based on the comments
I've seen from Chuck at least?)

* still skipping travis CI for now to save credits

* point `BUILD_COMMIT` at latest version of the maintenance
branch to get updated feedback on our status here

[skip travis]
@tylerjereddy
Copy link
Collaborator Author

Looks like we are using -arch arm64 -arch x86_64 in the Python 3.10 build, perhaps via the multibuild machinery, resulting in errors like this:

/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk/usr/include/machine/_mcontext.h:31:2: error: architecture not supported
#error architecture not supported

Nearly identical errors have been reported in many other places like here: facebookresearch/detectron2#2288 (comment)

Suggested solutions include export ARCHFLAGS="-arch x86_64".

@tylerjereddy
Copy link
Collaborator Author

The multibuild README has a section on MacOS binary behavior with a provision on overriding wrap_wheel_builder() if you don't want to go with one of the available options. I've attempted to provide a separate control flow that proceeds with a native intel build without fusing or ARM-related stuff.

@tylerjereddy tylerjereddy force-pushed the treddy_172_backports_debug branch 5 times, most recently from 12a14a3 to 2d58b49 Compare October 30, 2021 20:32
tylerjereddy added a commit to tylerjereddy/scipy-wheels that referenced this pull request Oct 31, 2021
* based on the discussion in scipy/scipy#14567

* point to the `multibuild` branch from Isuru for thin x86_64
MacOS wheels along with specifying `x86_64` for `PLAT`

* delete pretty much all other CI entries for now (debug only)

* let's see if this behaves any better than the other debug
branch targeted at `v1.7.x`:
MacPython#136
tylerjereddy added a commit to tylerjereddy/scipy-wheels that referenced this pull request Oct 31, 2021
* based on the discussion in scipy/scipy#14567

* point to the `multibuild` branch from Isuru for thin x86_64
MacOS wheels along with specifying `x86_64` for `PLAT`

* delete pretty much all other CI entries for now (debug only)

* let's see if this behaves any better than the other debug
branch targeted at `v1.7.x`:
MacPython#136
@tylerjereddy tylerjereddy deleted the treddy_172_backports_debug branch November 5, 2021 02:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant