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

qq-nt: fix checkver and update to 9.9.11 #125

Merged
merged 1 commit into from
Jun 9, 2024
Merged

qq-nt: fix checkver and update to 9.9.11 #125

merged 1 commit into from
Jun 9, 2024

Conversation

wenxuanjun
Copy link
Collaborator

qq-nt 的 manifest 没有正常更新,原因是 checkver 与 autoupdate 配置无法正确获取 url,报错如下:

.\bin\checkver.ps1 -Dir ..\..\..\buckets\scoopet -App qq-nt -u
qq-nt: 9.9.11 (scoop version is 9.9.9) autoupdate available
Autoupdating qq-nt
ERROR 使用“1”个参数调用“.ctor”时发生异常:“无效的 URI: 未能确定 URI 的格式。”

现修改 checkver 正则匹配与 autoupdate 中的 url 配置,能正确自动更新 manifest:

.\bin\checkver.ps1 -Dir ..\..\..\buckets\scoopet -App qq-nt -u
qq-nt: 9.9.11 (scoop version is 9.9.9) autoupdate available
Autoupdating qq-nt
Downloading QQ_9.9.11_240606_x86_01.exe to compute hashes!
QQ_9.9.11_240606_x86_01.exe (170.7 MB) [======================================================================] 100%
Computed hash: c14106bfff6ef44a006fc5a6bf8927933e67e8f9d30502f4d54d0736c790cfd4
Downloading QQ_9.9.11_240606_x64_01.exe to compute hashes!
QQ_9.9.11_240606_x64_01.exe (183.8 MB) [======================================================================] 100%
Computed hash: 281931793c9260c7f220b34e16517f8fb577026186e6c1c94f7eac6489861543
Writing updated qq-nt manifest

@wenxuanjun wenxuanjun requested a review from ivaquero as a code owner June 9, 2024 18:35
@wenxuanjun wenxuanjun changed the title Fix checkver and update to 9.9.11 qq-nt: Fix checkver and update to 9.9.11 Jun 9, 2024
@wenxuanjun wenxuanjun changed the title qq-nt: Fix checkver and update to 9.9.11 qq-nt: fix checkver and update to 9.9.11 Jun 9, 2024
@ivaquero ivaquero merged commit 4c4ac0b into ivaquero:master Jun 9, 2024
3 checks passed
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