-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Reported installed version of OpenMedia.4KVideoDownloader is incorrect #54507
Comments
No?
Note: you cannot install both versions (exe and msi) Manually installed the 4.19.4 exe version
Both 4.19.4 and 4.20.0 is installed at the same point at one time, however the installer cleans up the previous version, this seems to be something on your side. HOWEVER, my registry key is Something with your installer might have gone wrong at some point, i would advice you to reinstall |
I've been digging and as I said in my original post the issue indeed lies in the installer having let quite some garbage in the registry. Even the Settings app thinks there are two apps installed: Once I've uninstalled both "copies" of the app and reinstalled it via Note that such installer issues are not that uncommon, I also had a system where the registry contained entries for both VLC 3.0.12 and 3.0.16 (but in that case neither |
We do know of installers which do not clean up the previous version which is why we can add this field to the manifest: However as i said and tested, 4k Video Downloader does clean up the previous version, so i believe this is a issue between your PC and the Installer (since it also happened with VLC), my best guess is that it gets terminated too early or the cleanup part process errors out or gets permission denied. btw, the uninstall registry strings and the Settings app/Add or Remove Programs are tied together (which WinGet uses for matching) |
I've faced tons of issues with 4k Video Downloader's auto-update features over the years, ranging from failed upgrades ("you have to download the installer from the website and upgrade that way") to issues when multiple accounts have the app installed on the same machine. The VLC issue was probably created quite some time in the past, when I upgraded from 3.0.12 to whatever that release had told me to upgrade too. I'll keep monitoring the consistency of FWIW I've been managing what apps are installed on which PCs in the household with a set of scripts built on top of |
Wops forgot to reply, would it be bad if we made winget always uninstall 4k Video Downloader before installing the new version? Since you've experienced the issues with recent versions its probably not fixed... The way WinGet works is simply
So its exactly like you downloading the exe/msi from their website. |
The standard way of upgrading 4k Video Downloader is the in-app self-upgrade. This seems to download and run the EXE installer to perform upgrade: Sometimes this fails, and one has to download the EXE from their website 'cos that's what they offer by default. And recently what you get (from https://www.4kdownload.com/thanks-for-downloading?source=videodownloader) is even an "online installer" that downloads the actual installer under the hood. |
Just ran the self-upgrade process. Now the app's about box reports version This is consistent with Settings / Apps & features: The registry contains: |
Note that the issue with running an uninstall followed by an install might be that the configuration of the app would be different after the whole operation (depending on what the uninstaller does or doesn't remove). BTW upgrading apps with |
Yes, thats why i said you should try the MSI instead, these are our urls Lines 13 to 31 in 9df157a
The MSI's are the full installer and i very much doubt it will have any of the issues you are experiencing
Yeah that is why this should be a last resort and the people who use the software should warn us if it does
Yeah.. That is because it just runs the silent switches, we could add a custom switch (install only) or a upgrade switch (should be used if the Software got a switch for upgrade instead of using UninstallPrevious) I guess we should request a field for StartMenuShortcut, DesktopShortcut and any other switch you can think off at winget-cli |
Using the MSI will only solve the issue if the self-upgrade within the app installed via MSI doesn't exhibit the same registry problem / doesn't use the EXE to self-upgrade. I need to experiment by uninstalling, installing an old version via MSI, and then perform a self-upgrade. |
Glhf https://github.com/microsoft/winget-pkgs/tree/master/manifests/o/OpenMedia/4KVideoDownloader @KaranKad are you a 4k Video Download user? Are you using the burn or msi version and have you encountered this issue before? |
Cant say because I don't use it anymore, will try testing it tho. |
I've performed the following test:
Now the app reports it's version I could in theory run another test where 3 would be replaced by "install the app via its EXE installer." The key is probably 2. To be safe, now I have to run those steps on all my PCs 😭. |
Hi there, RevoUninstaller would like a word with you. |
Thanks for the hints. But I like to be in control, I don't know what these tools actually do. They must have some assumptions about what they can, cannot, should or should not remove, and in the context of the kind of mess I'm experiencing, how can I trust they will not remove anything they shouldn't remove, or leave behind something they should remove? |
You can see/choose what it removes, in theory they watch what the uninstaller does then searches for any relevant information (+ the information its told to look for if its a tricky software), but since you already know what to look for this should be perfect for you. Test it in Sandbox/VM ;) |
The current situation is that the app's auto-updates itself, but most of the time |
Thank you for taking the time to report this issue. In the amount of time the issue has been open, there have been several updates to the WinGet CLI which may have helped mitigate some of these concerns. I encourage you to try out the latest version of the CLI and see if your problem still persists. If additional problems persist, it would be extremely helpful if a new issue could be opened. Thanks! Close with reason: Stale; |
Brief description of your issue
With the latest version of "4k Video Downloader" (
OpenMedia.4KVideoDownloader
) installed, i.e. version 4.20.0.4740,winget
tells me I have version 4.19.4.4720 installed, and that there's an upgrade available.Steps to reproduce
(Probably?) Install version 4.19.4.4720 of "4k Video Downloader", and upgrade to 4.20.0.4740.
Then invoke
winget upgrade
.Expected behavior
"4k Video Downloader" should not be listed by
winget upgrade
, andwinget list
should list it as version 4.20.0.4740Actual behavior
winget upgrade
reports:Note: this is probably not
winget
's fault -- kind of. The app shows up twice under theUninstall
tree:HKEY_LOCAL_MACHINE__SOFTWARE__Microsoft__Windows__CurrentVersion__Uninstall_OpenMedia.4KVideoDownloader_202203101546.zip
Environment
The text was updated successfully, but these errors were encountered: