FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Enable Windows on Arm GitHub runners by diegorusso · Pull Request #131994 · python/cpython · GitHub

  1. python
  2. cpython

Repository navigation

Enable Windows on Arm GitHub runners - #131994

Merged
ambv merged 3 commits into
python:mainfrom
diegorusso:windows-aarch64-runners
Apr 3, 2025
ConversationCommits (3)ChecksFiles changed
Merged

Enable Windows on Arm GitHub runners#131994
ambv merged 3 commits into
python:mainfrom
diegorusso:windows-aarch64-runners

Conversation

diegorusso commented Apr 1, 2025

Copy link
Copy Markdown
Contributor

I had to put some if conditions in different steps because these runners are not generally available hence skipping checks on forks.

diegorusso marked this pull request as draft April 1, 2025 22:13
brandtbucher reviewed Apr 2, 2025
View reviewed changes

brandtbucher left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Can you remove the word "Native" from each of the JIT job names, now that we don't have the "Emulated" versions anymore?

diegorusso commented Apr 2, 2025

Copy link
Copy Markdown
Contributor Author

The Windows on Arm runners are failing because of https://developercommunity.visualstudio.com/t/ARM64-stack-corruption-in-1443-in-OpenS/10883345

After the upgrade to OpenSSL 3.0.16 (#131839) there is a stack corruption in OpenSSL that makes crash Python on Windows on Arm.

diegorusso marked this pull request as ready for review April 2, 2025 01:12

diegorusso commented Apr 3, 2025

Copy link
Copy Markdown
Contributor Author

The Windows on Arm runners are failing because of https://developercommunity.visualstudio.com/t/ARM64-stack-corruption-in-1443-in-OpenS/10883345

After the upgrade to OpenSSL 3.0.16 (#131839) there is a stack corruption in OpenSSL that makes crash Python on Windows on Arm.

This has been fixed here: #132051

ambv merged commit 0dbaeb9 into python:main Apr 3, 2025
diegorusso deleted the windows-aarch64-runners branch April 4, 2025 13:39
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

brandtbucher brandtbucher left review comments
savannahostrowski Awaiting requested review from savannahostrowski
ezio-melotti Awaiting requested review from ezio-melotti ezio-melotti is a code owner
hugovk Awaiting requested review from hugovk hugovk is a code owner
AA-Turner Awaiting requested review from AA-Turner AA-Turner is a code owner

Assignees

No one assigned

Labels

skip issue skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL