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

[Package Issue]: calibre.calibre broken download links #133004

Closed
2 tasks done
wickles opened this issue Jan 7, 2024 · 7 comments · Fixed by #155866
Closed
2 tasks done

[Package Issue]: calibre.calibre broken download links #133004

wickles opened this issue Jan 7, 2024 · 7 comments · Fixed by #155866
Labels
In-PR Issue-Bug It either shouldn't be doing this or needs an investigation.
Milestone

Comments

@wickles
Copy link
Contributor

wickles commented Jan 7, 2024

Please confirm these before moving forward

  • I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
  • I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue

Installation issue.

Brief description of your issue

Manifests for some recent versions of Calibre are using github links for InstallerUrl but these become stale after a new release, since they delete old release artifacts from github for some reason. (@kovidgoyal why?)

This makes it impossible to install or upgrade when a new version is released but not yet included in winget.

Instead of

https://github.com/kovidgoyal/calibre/releases/download/v${version}/calibre-64bit-${version}.msi

the stable link should be

https://download.calibre-ebook.com/${version}/calibre-64bit-${version}.msi

Please do not submit or accept updates for calibre with github download links in the future unless this issue is resolved upstream and the download links become stable again. (@vedantmgoyal2009)

Steps to reproduce

Try to install or upgrade calibre.calibre to a bad version.

winget install calibre.calibre --version 7.2.0

Actual behavior

> winget install calibre.calibre --version 7.2.0
Found an existing package already installed. Trying to upgrade the installed package...
Found calibre [calibre.calibre] Version 7.2.0
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://github.com/kovidgoyal/calibre/releases/download/v7.2.0/calibre-64bit-7.2.0.msi
An unexpected error occurred while executing the command:
Download request status is not success.
0x80190194 : Not found (404).

Expected behavior

Old versions should remain accessible.

Environment

Windows Package Manager (Preview) v1.7.3481-preview
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22621.2861
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.22.3481.0

Screenshots and Logs

No response

@wickles wickles added the Issue-Bug It either shouldn't be doing this or needs an investigation. label Jan 7, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Triage This work item needs to be triaged by a member of the core team. label Jan 7, 2024
@microsoft microsoft deleted a comment from github-actions bot Jan 8, 2024
@stephengillie stephengillie removed the Needs-Triage This work item needs to be triaged by a member of the core team. label Jan 8, 2024
@wickles wickles changed the title [Package Issue]: calibre.calibre [Package Issue]: calibre.calibre broken download links Jan 19, 2024
@wickles
Copy link
Contributor Author

wickles commented Jan 19, 2024

@kovidgoyal could you please comment on why the old artifacts get deleted from the github repo? As far as I know it should not cost anything to persist all build artifacts on a public repo.

@kovidgoyal
Copy link

Because calibre releases are huge and frequent and I am not comfortable
using up so much storage/bandwidth from a free service. All calibre
releases are available in perpetuity from
https://download.calibre-ebook.com/ regardless of changes to other
hosting services.

@wickles
Copy link
Contributor Author

wickles commented Jan 23, 2024

I see, thank you for the feedback.

Closing since all versions have been fixed now.

@wickles wickles closed this as completed Jan 23, 2024
@denelon denelon added this to the 1.7 Packages milestone Feb 13, 2024
@wickles
Copy link
Contributor Author

wickles commented Mar 31, 2024

Unstable link was added again in #145749. Requesting @vedantmgoyal9 to update the link and stop submitting unstable github download links for calibre in the future.

@wickles wickles reopened this Mar 31, 2024
@alorle
Copy link

alorle commented May 31, 2024

The same thing happens with the versions 7.8.0 (#147834), 7.9.0 (#150076), 7.10.0 (#152040), 7.11.0 (#154954) and 7.12.0 (#155837).

@wickles
Copy link
Contributor Author

wickles commented May 31, 2024

@SpecterShell #155866 does not resolve the issue. The other versions mentioned in the previous comment are still broken. As an example/demonstration:

winget install calibre --version 7.10.0

fails with error 404.

@SpecterShell
Copy link
Contributor

Done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
In-PR Issue-Bug It either shouldn't be doing this or needs an investigation.
Projects
None yet
6 participants