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(ci): windows build #1094

Merged
merged 8 commits into from
Sep 8, 2022
Merged

fix(ci): windows build #1094

merged 8 commits into from
Sep 8, 2022

Conversation

whizzzkid
Copy link
Contributor

@whizzzkid whizzzkid commented Sep 1, 2022

Looks like builds are broken on windows-latest: https://github.com/ipfs/ipfs-companion/actions/workflows/ci.yml

The 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.

@whizzzkid whizzzkid requested a review from lidel as a code owner September 1, 2022 05:40
@lidel lidel changed the title [Fix] Broken Builds fix(ci): window build Sep 1, 2022
@lidel lidel changed the title fix(ci): window build fix(ci): windows build Sep 1, 2022
@lidel
Copy link
Member

lidel commented Sep 1, 2022

@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.

.github/workflows/ci.yml Outdated Show resolved Hide resolved
@whizzzkid
Copy link
Contributor Author

@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.

@whizzzkid
Copy link
Contributor Author

whizzzkid commented Sep 1, 2022

@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!

Copy link
Member

@lidel lidel left a comment

Choose a reason for hiding this comment

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

Thank you @whizzzkid ❤️

@lidel lidel merged commit 044032f into ipfs:main Sep 8, 2022
whizzzkid added a commit to whizzzkid/ipfs-companion that referenced this pull request Sep 11, 2022
@lidel lidel mentioned this pull request Sep 26, 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.

2 participants