Skip to content

Only set up Cygwin on Windows CI if truly required #15661

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Only set up Cygwin on Windows CI if truly required #15661

wants to merge 2 commits into from

Conversation

HertzDevil
Copy link
Contributor

@HertzDevil HertzDevil commented Apr 14, 2025 •
edited
Loading

Extracted from #15660, this doesn't fix the issue but should greatly reduce CI noise as the majority of the affected PRs never touch the workflow definitions

@HertzDevil HertzDevil added platform:windows Windows support based on the MSVC toolchain / Win32 API topic:infrastructure/ci kind:chore labels Apr 14, 2025
@HertzDevil
Copy link
Contributor Author

This chmod: changing permissions of '.': Permission denied is really baffling since it's pretty much the final step of the build...?

@ysbaddaden ysbaddaden added this to the 1.17.0 milestone Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ysbaddaden ysbaddaden ysbaddaden approved these changes

@straight-shoota straight-shoota straight-shoota approved these changes

Assignees
No one assigned
Labels
kind:chore platform:windows Windows support based on the MSVC toolchain / Win32 API topic:infrastructure/ci
Projects
None yet
Milestone
1.17.0
Development

Successfully merging this pull request may close these issues.
3 participants
@HertzDevil @ysbaddaden @straight-shoota