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

Version mismatch after latest update (reported by users who installed via winget) #18115

Open
simkoG opened this issue Nov 30, 2022 · 22 comments
Labels
Not an issue User error, problem unrelated to qBittorrent, feature already implemented, etc NSIS NSIS installer related OS: Windows Issues specific to Windows

Comments

@simkoG
Copy link

simkoG commented Nov 30, 2022

qBittorrent & operating system versions

qBittorrent: 4.4.5 x64
Operating system: Windows 11 Pro 22H2/22621.819 x64 (10.0.22621)

What is the problem?

Something happened and now qBittorent shows taht the installed version is v4.4.5 (64-bit) while in the Control Panel list I see 4.5.0 version.

Steps to reproduce

I am not sure. Maybe related to winget?

Additional context

image
image

Log(s) & preferences file(s)

qbittorrent.log

@thalieht
Copy link
Contributor

I am not sure. Maybe related to winget?

Most probably.

@sledgehammer999
Copy link
Member

Go to our site and download the official installer.

@ghost
Copy link

ghost commented Dec 3, 2022

Im having this same issue, in my case it installed but then reverted to 4.4.5 after closing and reopening the program.
Screenshot 2022-12-03 134538

@thalieht thalieht added the Waiting info Waiting for participants to supply more info/fulfill template requirements label Jan 11, 2023
@xavier2k6
Copy link
Member

@simkoG @Sliverin Is this still an issue with qBittorrent 4.5.2?

@Yentis
Copy link

Yentis commented Sep 4, 2023

Just had this happen when updating through Winget while qBittorrent was still running (and after restarting qBittorrent):
qbittorrent_tCr2D0BKbn
wingetui_9wlwV0Q2gL

@luzpaz luzpaz changed the title Version mismatch after latest update Version mismatch after latest update (reported by users who installed via winget) Sep 4, 2023
@luzpaz luzpaz added the OS: Windows Issues specific to Windows label Sep 4, 2023
@luzpaz
Copy link
Contributor

luzpaz commented Sep 4, 2023

What's the winget installer missing? Can someone let the downstream winget devs know ? (https://github.com/microsoft/winget-pkgs/pulls?q=is%3Apr+qbittorrent+is%3Aclosed)

@xavier2k6
Copy link
Member

@Yentis

  1. Did you issue install or upgrade command?
  2. Did you run winget under PowerShell or Command Prompt?
  3. Run as Administrator being used or not?
  4. Are you using Windows 10 or Windows 11 & which build version?

@Yentis
Copy link

Yentis commented Sep 4, 2023

  1. winget update --id qBittorrent.qBittorrent --exact --accept-source-agreements --disable-interactivity --accept-source-agreements --force via WingetUI (alias for upgrade)
  2. Used WingetUI which uses PowerShell
  3. Not running as Administrator, permissions were requested upon starting the update
  4. Windows 10 21H2 19044.3324

@xavier2k6
Copy link
Member

@Yentis Use winget upgrade qBittorrent.qBittorrent via PowerShell or Command Prompt

@Yentis
Copy link

Yentis commented Sep 4, 2023

That gives the same result as before.

@xavier2k6
Copy link
Member

xavier2k6 commented Sep 4, 2023

in my case it installed but then reverted to 4.4.5 after closing and reopening the program.

when updating through Winget while qBittorrent was still running

There lies the problem!

If qBittorrent is running when updating via winget......it'll update the correct version info via Windows Control Panel & under winget list qBittorrent.qBittorrent command but if you navigate to C:\Program Files\qBittorrent you will notice that qBittorrent.exe file didn't actually update to newer versions.......

Going forward, you will have to shutdown qBittorrent prior to updating with winget, update via our official installers from main website -> https://www.qbittorrent.org/download &/or report issue/findings upstream.

@xavier2k6 xavier2k6 added Not an issue User error, problem unrelated to qBittorrent, feature already implemented, etc and removed Waiting info Waiting for participants to supply more info/fulfill template requirements labels Sep 4, 2023
@xavier2k6
Copy link
Member

I will leave this issue open for a day or so....

@Yentis
Copy link

Yentis commented Sep 4, 2023

Makes sense that things go wrong if the app is still running, but maybe the silent installer should fail in the case of winget? So the user knows they have to close the app first.

@xavier2k6
Copy link
Member

but maybe the silent installer should fail in the case of winget? So the user knows they have to close the app first.

I didn't enable logging to see where or what went wrong....may do so when I have more time.

@luzpaz
Copy link
Contributor

luzpaz commented Sep 5, 2023

Maybe we can let the winget devs know about this and they can factor a warning message for users to make sure qbit isn't running while they update ? (this will help avoid future tickets like this from winget users)

@xavier2k6
Copy link
Member

xavier2k6 commented Sep 5, 2023

Ran winget with command winget install --verbose-logs qBittorrent.qBittorrent via PowerShell while qBittorrent was running & log is below:

WinGet-2023-09-05-14-43-47.919.log

I know if we run our own installer....it gives a warning to close running instance.

Screenshot 2023-09-05 151235

@denelon
Copy link

denelon commented Nov 1, 2023

WinGet was designed with automation in mind, and a requirement to support silent installation. If the installer could throw an error when the installer is executed in silent or silent with progress, that error could be mapped to an expected return code in the manifest informing the user that they need to close the application.

@xavier2k6
Copy link
Member

@sledgehammer999 Can you look in to this?

@xavier2k6 xavier2k6 added the NSIS NSIS installer related label Nov 2, 2023
@luzpaz
Copy link
Contributor

luzpaz commented Nov 17, 2023

soft reminder for @sledgehammer999

@soredake
Copy link

#18582

@soredake
Copy link

Same problem with virtualbox https://www.virtualbox.org//ticket/21951

@Chocobo1
Copy link
Member

Chocobo1 commented Jan 19, 2024

If the installer could throw an error when the installer is executed in silent or silent with progress, that error could be mapped to an expected return code in the manifest informing the user that they need to close the application.

I've submitted PR #20296 which will set error codes when encountered errors at installation. I reckon the rest is to map the error codes to specific actions on WinGet side so each situation will be handled accordingly.

Chocobo1 added a commit that referenced this issue Jan 22, 2024
* Set specific exit codes when NSIS installer failed in silent mode
  Related: #18115 (comment)
* GHA CI: treat NSIS warnings as errors

PR #20296.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Not an issue User error, problem unrelated to qBittorrent, feature already implemented, etc NSIS NSIS installer related OS: Windows Issues specific to Windows
Projects
None yet
Development

No branches or pull requests

9 participants