-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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]: AnyDeskSoftwareGmbH.AnyDesk #67794
Comments
This looks like a bad manifest to me. Looking at the history, we would have to ask @OfficialEsco if they remember why they made the version start with "ad" in #14259. I just installed the latest on a sandbox and the The website lists it under the download button as "v7.0.13 (3.8 MB)", no "ad" involved. I don't see any reason why we should want to use a non-standard, non-sortable package version like this. It should be changed to be |
The "<" symbol means we evaluated the version, and it was less than one of the manifests we have in range. |
I was going to suggest that
As far as the version parsing goes, I'm thinking that if there are leading non-decimal characters followed by decimal characters in the first version part, we can ignore them? So AnyDesk, with only the one version at a time, isn't actually impacted by this much (until "ad 10.0.0" doesn't sort higher than "ad 9.9.9"). And this issue is more about the confusing @OfficialEsco and @jedieaston , you have more experience with the plethora of version numbers out there. Do you see any issue with my suggestion above? Any improvement? We obviously have to be careful not to disrupt the existing behavior, but I don't really see a problem affecting versions that don't start with numbers. |
AnyDesk exe is the only package which have this stupid Version schema, there was actually a period where we had AnyDesk.AnyDesk.MSI and AnyDesk.AnyDesk.EXE to work around this issue.. I don't see any issues in doing that, that could maybe also resolve the |
This also sounds good to me. |
Any update on this issue? |
It's a relatively low priority ask at this point as there are very few 👍 on the issue, and relatively few packages with this problem impacting upgrade. |
(maybe this will increase the priority?) AnyDesk 7.0.14 was released recently to fix another security vulnerability (the changelog doesn't mention any CVE for it, or other details yet) but winget still tries to "upgrade" from 7.0.14 to the old 7.0.13. side note: Firefox Developer edition also seems affected by a similar wrong version detection |
Currently blocked in validation for unknown pipeline issue reasons |
The hash must to be updated. But there is an other problem: There is a lack of separation of AnyDesk.exe and AnyDesk.msi versions. For example, while I use installation by "AnyDesk.exe", |
A simple note:
Check the official documentation for available parameters. |
Separating the package causes a bigger issue where both the MSI and EXE matches, the only difference is the stupid version name of the EXE which is |
The only difference is NOT the version name. For example, msi version installs at "C:\Program Files (x86)\AnyDeskMSI" while exe version installs at "C:\Program Files (x86)\AnyDesk". In installation folder, the msiversion adds "MSI" at the end of name ("AnyDeskMSI.exe"). As a result, it would be better to separate "AnyDesk.exe" version from regular msi version.
|
I checked the manifest and see that exe version is already in it. |
when I use same command at powershell, version number is correct. So, what can cause this issue? |
1 similar comment
when I use same command at powershell, version number is correct. So, what can cause this issue? |
Tested your script, same version number. |
I see it now. It is not reflected to |
I figured out a small difference between exe version and msi version: exe <-> msi You can use same config files on both, just don't forget to change .trace filename accordingly. |
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; |
Since version 9.0.1 this works very good . cmd => winget upgrade AnyDeskSoftwareGmbH.AnyDesk |
Please confirm these before moving forward
Category of the issue
Other
Brief description of your issue
Winget seems to be parsing the version information from the Windows Registry in an incorrect way.
Steps to reproduce
Actual behavior
See steps to reproduce and screen shot.
Expected behavior
Pull the version from the Registry without the carrot. (I think it's a carrot)
Environment
Screenshots and Logs
The text was updated successfully, but these errors were encountered: