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

totalcommander: Fix download and autoupdate links #14690

Merged
merged 2 commits into from
Jan 8, 2025

Conversation

kborowinski
Copy link
Contributor

TotalCommander x32 and x64 executables are stored in versioned subfolders matching the $cleanVersion checkver variable, instead of win folder:
https://totalcommander.ch/1150/tcmd1150x64.exe
https://totalcommander.ch/1150/tcmd1150x32.exe
vs
http://totalcommander.ch/win/tcmd1150x64.exe
http://totalcommander.ch/win/tcmd1150x32.exe

  • Use conventional PR title: <manifest-name[@version]|chore>: <general summary of the pull request>
  • I have read the Contributing Guide

Copy link
Contributor

github-actions bot commented Jan 7, 2025

All changes look good.

Wait for review from human collaborators.

totalcommander

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

@kborowinski
Copy link
Contributor Author

kborowinski commented Jan 7, 2025

@aliesbelik @rasa: I've contacted the TotalCommander author and he was kind enough to copy the latest release to http://totalcommander.ch/win folder, but from now on we should be using the new links as proposed in this PR. I've resolved the merge conflict to use the new links.

@kborowinski
Copy link
Contributor Author

/verify

Copy link
Contributor

github-actions bot commented Jan 8, 2025

All changes look good.

Wait for review from human collaborators.

totalcommander

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

@aliesbelik aliesbelik changed the title totalcommander: add version 11.50 and fix auto update links totalcommander: Fix download and autoupdate links Jan 8, 2025
@aliesbelik aliesbelik merged commit 41671bd into ScoopInstaller:master Jan 8, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants