-
Notifications
You must be signed in to change notification settings - Fork 325
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(ci): windows build #1094
fix(ci): windows build #1094
Conversation
@whizzzkid it is running in https://github.com/ipfs/ipfs-companion/runs/8139100657?check_suite_focus=true but takes longer than expected, when it timeouts you may want to inspect logs. |
@lidel looks like the build is stuck https://github.com/ipfs/ipfs-companion/runs/8139100657?check_suite_focus=true Turns out this is an issue: felixrieseberg/windows-build-tools#208 I added the suggested fix in the thread, sorry I couldn't test it locally without a windows machine. But that didn't kill the existing build. |
@lidel @SgtPooki I just realized I can run GH Actions on my own fork as well. so now this passes: https://github.com/whizzzkid/ipfs-companion/runs/8143445527?check_suite_focus=true Apologies for all the noise! |
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.
Thank you @whizzzkid ❤️
* main: fix(ci): windows build (ipfs#1094)
Looks like builds are broken on
windows-latest
: https://github.com/ipfs/ipfs-companion/actions/workflows/ci.ymlThe reported error:
gyp ERR! stack Error: Could not find any Visual Studio installation to use
I could not test this as I don't have perms to run CI nor do I have access to a windows machine atm, but there seems like two ways to fix this:
the github
actions/setup-node@v1
is also 2 major versions behind, so should be upgraded as well, but I'm unsure of the breaking API changes between the two.🤞🏽 this should fix the broken builds in theory.