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

MAINT: rename MacOS arm64 wheels#147

Merged
rgommers merged 1 commit intoMacPython:masterfrom
tylerjereddy:treddy_rename_arm64_macos_thin_wheels
Nov 23, 2021
Merged

MAINT: rename MacOS arm64 wheels#147
rgommers merged 1 commit intoMacPython:masterfrom
tylerjereddy:treddy_rename_arm64_macos_thin_wheels

Conversation

@tylerjereddy
Copy link
Copy Markdown
Collaborator

  • automatically rename MacOS arm64 thin wheel files to
    use version 12_0 instead of 11_0; the verbose file
    move should print out the change for us if successful

* automatically rename MacOS arm64 thin wheel files to
use version `12_0` instead of `11_0`; the verbose file
move should print out the change for us if successful
set -xe
for file in `find ./wheelhouse -type f -name '*11_0_arm64*.whl'`; do mv -v "$file" "${file/11_0_arm64/12_0_arm64}"; done
displayName: "Rename MacOS arm64 wheels for version 12.0 minimum"
condition: eq(variables['PLAT'], 'arm64')
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

this is slight overkill since there's only one wheel per matrix entry, but it does seem to work locally and the paradigm was conveniently available from a stackoverflow snippet.. let's see what CI says..

Copy link
Copy Markdown
Contributor

@rgommers rgommers left a comment

Choose a reason for hiding this comment

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

While we're at it: the universal2 wheels should also be renamed to 12_0 (now scipy-1.8.0.dev0+2085.d18b2a4-cp310-cp310-macosx_10_9_universal2.whl).

@tylerjereddy
Copy link
Copy Markdown
Collaborator Author

That's true, though CI is passing and we don't have universal2 wheels on the release branch so maybe we can merge for now and open an issue.

@rgommers
Copy link
Copy Markdown
Contributor

Sure, that's fine with me - in it goes. Thanks Tyler!

@rgommers rgommers merged commit f1d2caa into MacPython:master Nov 23, 2021
@tylerjereddy tylerjereddy deleted the treddy_rename_arm64_macos_thin_wheels branch November 23, 2021 19:57
tylerjereddy added a commit to tylerjereddy/scipy-wheels that referenced this pull request Nov 23, 2021
* automatically rename MacOS arm64 thin wheel files to
use version `12_0` instead of `11_0`; the verbose file
move should print out the change for us if successful
@rgommers
Copy link
Copy Markdown
Contributor

rgommers commented Oct 6, 2022

For future archeologists: this was because of the kernel panic on macOS 11: scipy/scipy#14688 (comment)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants