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

issue #3982: fix setup cygwin action. #3983

Closed

Conversation

suzp1984
Copy link
Contributor

@suzp1984 suzp1984 commented Mar 9, 2024

fix setup Cygwin action by update to its lastest commit.

@suzp1984 suzp1984 force-pushed the issue/3982-fix-setup-cygwin-action branch from 3380e0f to 60084c8 Compare March 9, 2024 10:36
@suzp1984
Copy link
Contributor Author

suzp1984 commented Mar 9, 2024

This PR try to fix #3982 .

Root Cause
According to the https://github.com/cygwin/cygwin-install-action/blob/master/action.yml
cygwin-install-action will check the the signature by default.
But http://mirrors.kernel.org/sourceware/cygwin/x86_64/ moved the signatures to single file yesterday.
http://mirrors.kernel.org/sourceware/cygwin/x86_64/sha512.sum

And https://cygwin.com/setup-x86_64.exe didn't know how to check the signatures in new place.

Solutions
disable the signatures check temporary. I also update the cygwin-install-action to latest commit(this may be unnecessary) .
But I believe the cygwin will collaborate with kernel.org to fix this bug soon, then we can open the signature check again later.

@winlinvip winlinvip linked an issue Mar 18, 2024 that may be closed by this pull request
@winlinvip
Copy link
Member

Merged to #3973

@winlinvip winlinvip closed this Mar 18, 2024
winlinvip added a commit that referenced this pull request Mar 26, 2024
#3983 already fixed the `test` workflow, but I think the `release` will
have same issue.

---------

Co-authored-by: winlin <winlinvip@gmail.com>
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.

GitHub action to set up Cygwin Failed to run
2 participants