Skip to content

Conversation

@tannewt
Copy link
Member

@tannewt tannewt commented Nov 5, 2025

No description provided.

dhalbert
dhalbert previously approved these changes Nov 5, 2025
Copy link
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

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

@dhalbert
Copy link
Collaborator

dhalbert commented Nov 5, 2025

ehh:

fatal error: /Applications/Xcode_16.4.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/lipo: mpy-cross/build/mpy-cross and mpy-cross/build-arm64/mpy-cross-arm64 have the same architectures (arm64) and can't be in the same fat output file

@dhalbert
Copy link
Collaborator

dhalbert commented Nov 5, 2025

ehh

fatal error: /Applications/Xcode_16.4.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/lipo: mpy-cross/build/mpy-cross and mpy-cross/build-arm64/mpy-cross-arm64 have the same architectures (arm64) and can't be in the same fat output file

Intel macos is only supported for free runners on macos-13. So I think we need to punt on Intel macos mpy-cross now.
See actions/runner-images#10810

@dhalbert dhalbert dismissed their stale review November 5, 2025 02:13

build failure

Copy link
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

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

With this PR, we are no longer building Intel or universal macOS mpy-cross now. mpy-cross is being built with macos-latest, which is macOS 15. But without paying, we can no longer build on macos-15 on an Intel platform.

@dhalbert dhalbert merged commit 984e072 into adafruit:main Nov 5, 2025
633 checks passed
@dhalbert
Copy link
Collaborator

dhalbert commented Nov 5, 2025

It looks like GitHub changed their mind, and macos-15-intel is available: actions/runner-images#13045. We could add it back, or just wait and see if anyone has an issue with no more Intel mpy-cross binaries available.

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