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

Fix macOS CI #117

Merged
merged 1 commit into from
May 24, 2024
Merged

Fix macOS CI #117

merged 1 commit into from
May 24, 2024

Conversation

marktsuchida
Copy link
Member

@marktsuchida marktsuchida commented May 24, 2024

Work in progress -- as I haven't managed to reproduce the issue locally.

Upgrade to macos-12 for wheel building.

macos-11 is deprecated and will be removed on 2024-06-28.
@tlambert03
Copy link
Contributor

that seems to have done it 😂
we might also consider adding -v to swig_opts in setup.py?

@tlambert03
Copy link
Contributor

that seems to have done it 😂

oh never mind... we'll need to temporarily let the Build & deploy / Build wheels on ${{ matrix.os }} ${{ matrix.macos_arch }} (pull_reques job run on pull requests... and just make sure it's not pushing to pypi

@marktsuchida
Copy link
Member Author

Presumably the cause was that the SWIG binary we're using crashes on macOS 11 (but not 12). I suspected that it was built for 12+ but

$ otool -l venv/lib/python3.11/site-packages/swig/data/bin/swig  |grep minos
    minos 11.0

(PyPI swig 4.2.1 on arm64) so maybe it crashes at run time even though not rejected by the OS.

(Or maybe it was some other weird transient issue, but I'm happy it's fixed!)

@marktsuchida marktsuchida marked this pull request as ready for review May 24, 2024 17:50
@marktsuchida marktsuchida force-pushed the fix-ci-macos-wheels branch from 2aa9a67 to 15ed019 Compare May 24, 2024 17:50
@marktsuchida marktsuchida changed the title Try to fix macOS CI Fix macOS CI May 24, 2024
@marktsuchida
Copy link
Member Author

(Force-pushed to re-disable wheels for PRs.)

@marktsuchida marktsuchida merged commit 60f03c4 into main May 24, 2024
28 checks passed
@marktsuchida marktsuchida deleted the fix-ci-macos-wheels branch May 24, 2024 17:55
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