-
Notifications
You must be signed in to change notification settings - Fork 358
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
Enable AOT build for windows-arm64 #2286
Conversation
1c19226
to
d146921
Compare
Turns out there are two versions of powershell: PowerShell Core (7.x, crossbow platform, named as
|
The TLS certificate workaround is confirmed to be working. Now it turns out even |
Looks like everything worked! |
This PR is another attempt to enable AOT snapshot for windows-arm64. It turns out the TLS issue we're seeing is likely not a windows runner bug but a windows feature and a dart bug: dart-lang/sdk#52266
In summary, the problem is that windows would lazily initialize some trusted certificates, but dart does not support this lazy initialization. The x86_64 windows runner image likely already have the missing root certificates initialized at some point by other calls made to other google services during its image provisioning, but this arm64 image isn't the same.
@nex3 I would like to give this another try, but I don't have access to arm64 runners. If you can push this commit to a branch directly in sass/dart-sass repo, you can use "run workflow" feature to test it without needing to merge this PR first.