[Package Issue]: Guru3D.Afterburner.Beta about winget-pkgs HOT 8 CLOSED

Issue is resolved on my end. Thanks to all!

from winget-pkgs.

Not sure exactly what's causing the correlation failure here, but I believe the simple solution would be to get rid of the DisplayVersion mapping and simply use PackageVersion: 4.6.6 Beta 3. DisplayVersion isn't necessarily required here since the version is also marketed as 4.6.6 (Beta 3) on the publisher's website. Also, the inclusion of DisplayVersion also means that we can't include multiple 4.6.6 Beta versions since they all would need to use the same 4.6.6 DisplayVersion, something that isn't allowed by the validation pipelines / CLI by design.

from winget-pkgs.

This contains an apps and features entry, so it should not be the case:

AppsAndFeaturesEntries:
  - DisplayName: MSI Afterburner 4.6.6 Beta 3
    DisplayVersion: 4.6.6 Beta 3

https://github.com/microsoft/winget-pkgs/blob/master/manifests/g/Guru3D/Afterburner/Beta/4.6.6/Guru3D.Afterburner.Beta.installer.yaml

from winget-pkgs.

Interesting. Something's not right though as when I do winget list I get this:

Name                                      Id                                         Version           Available Source
MSI Afterburner 4.6.6 Beta 3              Guru3D.Afterburner.Beta                    4.6.6 Beta 3      4.6.6     winget

from winget-pkgs.

@stephengillie, what would you recommend?

from winget-pkgs.

Verify:

Repro:

• Ideally, the PackageVersion should be updated to match the application: v4.6.6.16381 Beta 3
• But this doesn't cover the "upgrade always available" situation. It seems like the package manager is comparing the Registry data with the PackageVersion, and ignoring the DisplayVersion.
• Why it's doing this and how to fix, I'm not sure. I'm guessing that maybe the DisplayVersion isn't parsing the spaces in the version number correctly - maybe adding quotes around it will help parsing?

@mdanish-kh and @Trenly, is there anything I missed here?

from winget-pkgs.

@elisimpson PR containing the mentioned changes was merged into the repo. Could you please check if you are still encountering the issue?

from winget-pkgs.

Close with reason: Issue resolved by removing DisplayVersion;

from winget-pkgs.