Add windows arm64 support #183
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -15,7 +15,7 @@ jobs: | |||||||||||||||||||||
|
|
||||||||||||||||||||||
| build_and_publish: | ||||||||||||||||||||||
|
|
||||||||||||||||||||||
| uses: OpenAstronomy/github-actions-workflows/.github/workflows/publish.yml@80387e9fbf2f4aeeacc7c2db79d4d1c17154ae95 # v2.2.2 | ||||||||||||||||||||||
| if: (github.repository == 'liberfa/pyerfa') | ||||||||||||||||||||||
| with: | ||||||||||||||||||||||
|
|
||||||||||||||||||||||
|
|
@@ -52,6 +52,11 @@ jobs: | |||||||||||||||||||||
| - cp3*win32 | ||||||||||||||||||||||
| - cp3*win_amd64 | ||||||||||||||||||||||
| - pp3*-win_amd64 | ||||||||||||||||||||||
| # Windows arm64 wheels | ||||||||||||||||||||||
| - cp311*win_arm64 | ||||||||||||||||||||||
| - cp312*win_arm64 | ||||||||||||||||||||||
| - cp313*win_arm64 | ||||||||||||||||||||||
| - cp314*win_arm64 | ||||||||||||||||||||||
|
||||||||||||||||||||||
| - cp311*win_arm64 | |
| - cp312*win_arm64 | |
| - cp313*win_arm64 | |
| - cp314*win_arm64 | |
| - cp3*-win_arm64 |
There was a problem hiding this comment.
That makes sense. Though the problem I ran into is that when I run cp3*-win_arm64 it tried to build for python 3.9 and fails when building numpy which is only available on arm64 for 3.11+ so I specified 3.11-3.14 individually. Should I replace it instead with just one version like cp311*win_arm64 or maybe cp314*win_arm64? Or is it possible to do something kind of like cp31+([1-9])*win_arm64 that specifies any python after 3.11?
There was a problem hiding this comment.
Oh you're right, it won't work on 3.9 or 3.10. However, it's still important to use a single line here, as the underlying workflow will create as many jobs as there are lines, and we ultimately don't need more than the one wheel. This should do:
| - cp311*win_arm64 | |
| - cp312*win_arm64 | |
| - cp313*win_arm64 | |
| - cp314*win_arm64 | |
| - cp3{11,12,13,14}-win_arm64 |
There was a problem hiding this comment.
That makes sense, thanks!
There was a problem hiding this comment.
It probably doesn't really matter here, but it looks like another option that works is cp31[1-9]*win_arm64 in case you wanted it to build/test for versions after 3.14 later without having to go back and update the line.
There was a problem hiding this comment.
The report will still need to be updated for cibw to target newer versions. Also, this won't work for 3.20 and later.
In any case, 3.15 might introduce some new ABI we'll want to use too (PEP 803 and 809), which will warrant a revisit
There was a problem hiding this comment.
Unresolved just in case someone else is like me and sees the cp3* and wonders why it isn't used here. (I don't want to re-run just to add a comment...)
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.
note that this hash doesn't match the version in comment, but points ahead of it, because version 2.2.3 isn't released yet.
https://github.com/OpenAstronomy/github-actions-workflows
(pyerfa maintainers: this class of issue can be treated as security problems and as such, can be linted against using zizmor. Let me know if you're interested and I can set it up for this repo, as I previously did for astropy)
Uh oh!
There was an error while loading. Please reload this page.
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.
Sorry about that! @neutrinoceros should we wait for a new release of OpenAstronomy/github-actions-workflows and then bump the hash/version to that v2.2.3 release?
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.
yes ! It's not difficult to release, we just need someone to pick it up :)
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.
2.3.0 is out
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.
Awesome! I'll go ahead and update the hash/version number.
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.
Can this be merged now?