-
Notifications
You must be signed in to change notification settings - Fork 9
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
Build and test installers natively on Apple Silicon M1 (macOS 14) #249
Build and test installers natively on Apple Silicon M1 (macOS 14) #249
Conversation
.github/workflows/build.yml
Outdated
# include: | ||
# - os: macos-11 | ||
# arch: arm64 | ||
os: [macos-11, macos-12, macos-13, macos-14] |
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.
also added macOS-13 here (runs on Intel; macos-14 always runs on M1)
if [[ "$MNE_CROSSCOMPILE_ARCH" == "arm64" ]]; then | ||
echo "PLATFORM_ARG=--platform=osx-arm64" >> $GITHUB_ENV | ||
echo "EXE_ARG=--conda-exe=${CONDA_PREFIX}/standalone_conda/conda.exe" >> $GITHUB_ENV | ||
fi |
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.
we shouldn't need this anymore, but I'm not sure about the implications
… installer Python
@larsoner I think this is good to go |
Awesome @hoechenberger ! |
* upstream/main: [pre-commit.ci] pre-commit autoupdate (mne-tools#251) Build and test installers natively on Apple Silicon M1 (macOS 14) (mne-tools#249) [pre-commit.ci] pre-commit autoupdate (mne-tools#247)
@larsoner I know you love this stuff ;)
I just wanted to get us started
We need to change
extract_version.sh
, I think you have a much better understanding of what needs to be done than I do. Please do feel free to take over if you want!!FYI macOS-14 exclusively runs on M1, while earlier macOS runners exclusively run on Intel. This just to explain why I removed the architecture identifier from the job matrix in favor of just having the OS version listed.
Closes #248