Skip to content
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

fix: use win64 version of chromium when on arm64 windows #8927

Merged
merged 4 commits into from Sep 19, 2022
Merged

fix: use win64 version of chromium when on arm64 windows #8927

merged 4 commits into from Sep 19, 2022

Conversation

robrobrobrob
Copy link
Contributor

What kind of change does this PR introduce?
bugfix

Did you add tests for your changes?
not relavant

If relevant, did you update the documentation?
not relavant

Summary
no Win_arm available on storage.googleapis.com yet, and it currently defaults to win32

Does this PR introduce a breaking change?
nope

Other information

@robrobrobrob
Copy link
Contributor Author

I'm getting some unrelated failed checks, and don't see a way to re-run them. Can I get some help?

@OrKoN
Copy link
Collaborator

OrKoN commented Sep 9, 2022

Are win64 binaries executable on arm64 windows?

@robrobrobrob
Copy link
Contributor Author

Are win64 binaries executable on arm64 windows?

good catch, only as of Windows 11. I'll add that check

@robrobrobrob
Copy link
Contributor Author

should be good now

@OrKoN OrKoN enabled auto-merge (squash) September 19, 2022 07:06
@OrKoN OrKoN merged commit 64843b8 into puppeteer:main Sep 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants