Skip to content

Update APT DB to refer to available version of rclone - #240

Merged
Leonidas-from-XIV merged 1 commit into
ocaml-dune:mainfrom
Leonidas-from-XIV:fix-rclone-install
Aug 3, 2026
Merged

Update APT DB to refer to available version of rclone#240
Leonidas-from-XIV merged 1 commit into
ocaml-dune:mainfrom
Leonidas-from-XIV:fix-rclone-install

Conversation

@Leonidas-from-XIV

Copy link
Copy Markdown
Contributor

The deployment currently fails because the rclone Debian package is not on the mirror anymore. This is most likely due to a new version of rclone being put on and the old version removed, but the APT database still refers to the older version (1.60.1+dfsg-3ubuntu0.24.04.5).

This PR updates both workflows to update the APT database before attempting to install rclone. I tested it on my branch and while deployment fails (due to missing credentials; as expected) installing rclone (1.60.1+dfsg-3ubuntu0.24.04.6) succeeds. So I expect that this will fix the deployment error too.

Signed-off-by: Marek Kubica <marek@tarides.com>

@Sudha247 Sudha247 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me. I'm happy with going ahead and merging this; but I have a question - are there any downsides to using the script to install: https://rclone.org/downloads/#script-download-and-install?

@Leonidas-from-XIV

Copy link
Copy Markdown
Contributor Author

We had this before (#209) but I don't see the purpose of installing software from an untrusted third party (imagine the domain lapsing and a malicious user taking over rclone.org and serving a malware script from over there) when I can get the software from an already trusted party (the Ubuntu repositories hosted by GitHub), with checksums etc.

@Leonidas-from-XIV
Leonidas-from-XIV merged commit e287cf3 into ocaml-dune:main Aug 3, 2026
3 of 4 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