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

Revert "Windows CI Fix, main branch (2024.06.11.)" #281

Merged
merged 1 commit into from
Aug 5, 2024

Conversation

krasznaa
Copy link
Member

Reverts #279.

The claim in actions/runner-images#10004 is that the GitHub Windows runners are now back to not needing this workaround anymore. So, let's see...

@krasznaa krasznaa added the cicd Changes the CI or CD settings label Jun 12, 2024
@krasznaa krasznaa requested a review from stephenswat June 12, 2024 07:52
@stephenswat
Copy link
Member

Apparently not. 😟

@krasznaa
Copy link
Member Author

I guess we can keep it open to have an easy way of testing this going forward. 🤔

@krasznaa krasznaa force-pushed the revert-279-WindowsCIFix-main-20240611 branch from 7db522d to 611312e Compare June 14, 2024 08:20
Copy link
Member

@stephenswat stephenswat left a comment

Choose a reason for hiding this comment

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

Great!

core/CMakeLists.txt Outdated Show resolved Hide resolved
@stephenswat stephenswat force-pushed the revert-279-WindowsCIFix-main-20240611 branch from 611312e to 65b7db2 Compare June 14, 2024 13:34
@krasznaa krasznaa force-pushed the revert-279-WindowsCIFix-main-20240611 branch from 65b7db2 to 87e80b8 Compare August 3, 2024 08:47
@krasznaa
Copy link
Member Author

krasznaa commented Aug 3, 2024

Are these latest failures fixed by #287? I thought that was for a slightly different issue. 🤔 I don't remember such missing glibc symbols as errors so far.

@stephenswat
Copy link
Member

Are these latest failures fixed by #287? I thought that was for a slightly different issue. 🤔 I don't remember such missing glibc symbols as errors so far.

Yeah, the glibc issues are fixed by #287, until Microsoft removes support for node 10 fully.

@krasznaa krasznaa force-pushed the revert-279-WindowsCIFix-main-20240611 branch from 87e80b8 to 7baab7b Compare August 5, 2024 12:34
@stephenswat
Copy link
Member

Looks good. 👍

@krasznaa krasznaa merged commit eaf551f into main Aug 5, 2024
55 checks passed
@krasznaa krasznaa deleted the revert-279-WindowsCIFix-main-20240611 branch August 5, 2024 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cicd Changes the CI or CD settings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants