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

GitHub action to set up Cygwin Failed to run #3982

Closed
suzp1984 opened this issue Mar 9, 2024 · 3 comments · Fixed by #3995
Closed

GitHub action to set up Cygwin Failed to run #3982

suzp1984 opened this issue Mar 9, 2024 · 3 comments · Fixed by #3995
Assignees

Comments

@suzp1984
Copy link
Contributor

suzp1984 commented Mar 9, 2024

!!! Before submitting a new bug report, please ensure you have searched for any existing bugs and utilized
the Ask AI feature at https://ossrs.io or https://ossrs.net (for users in China). Duplicate issues or
questions that are overly simple or already addressed in the documentation will be removed without any
response.

Setup Cygwin Action Failed
HTTP status 404 fetching http://mirrors.kernel.org/sourceware/cygwin/x86_64/setup.zst.sig
HTTP status 404 fetching http://mirrors.kernel.org/sourceware/cygwin/x86_64/setup.xz.sig
HTTP status 404 fetching http://mirrors.kernel.org/sourceware/cygwin/x86_64/setup.bz2.sig
HTTP status 404 fetching http://mirrors.kernel.org/sourceware/cygwin/x86_64/setup.ini.sig

Because the signatures are all merged to one file:
http://mirrors.kernel.org/sourceware/cygwin/x86_64/sha512.sum

Impact
Github Workflow Setup Cygwin action.

To Reproduce
check
https://github.com/ossrs/srs/actions/runs/8213483121/job/22464987462?pr=3981

Expected behavior
Github workflow can works well.

suzp1984 added a commit to suzp1984/srs that referenced this issue Mar 9, 2024
suzp1984 added a commit to suzp1984/srs that referenced this issue Mar 9, 2024
@winlinvip
Copy link
Member

Actually, I'm not an expert in Windows programming, so I'm not very familiar with Cygwin. However, I've approved your pull request. If it proves to work effectively, I believe making such a modification will be very beneficial.

@winlinvip winlinvip linked a pull request Mar 18, 2024 that will close this issue
@winlinvip winlinvip self-assigned this Mar 18, 2024
winlinvip pushed a commit to suzp1984/srs that referenced this issue Mar 18, 2024
@suzp1984
Copy link
Contributor Author

I forget to patch the release action. It should has same cygwin-install-action problem. I will patch it in another PR.

suzp1984 added a commit to suzp1984/srs that referenced this issue Mar 18, 2024
@suzp1984
Copy link
Contributor Author

suzp1984 commented Mar 18, 2024

http://mirrors.kernel.org/sourceware/cygwin/x86_64/

The user is referring to a screenshot with specific dimensions, taken at a certain time, and is pointing to a location where the image can be found, which is in the assets section of the 'ossrs/srs' repository on GitHub. The image seems to be related to a commit or issue in the repository.

Look, the mirrors updated on 17th-Mar, to fixed the signature missing problem.
So, I think the check-sig option can be opened by default.
The new PR #3995 just upgrade the release action to latest commit too.

TRANS_BY_GPT4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants