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

[Package Issue]: OPenRCT2.OpenRCT2 upgrade loop with 0.4.6 #139970

Closed
2 tasks done
markusdd opened this issue Feb 18, 2024 · 18 comments
Closed
2 tasks done

[Package Issue]: OPenRCT2.OpenRCT2 upgrade loop with 0.4.6 #139970

markusdd opened this issue Feb 18, 2024 · 18 comments
Labels
Area-External Installer-Issue Issue with the package's installer. Issue-Bug It either shouldn't be doing this or needs an investigation.

Comments

@markusdd
Copy link

Please confirm these before moving forward

  • I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
  • I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue

Installation issue.

Brief description of your issue

identical to issue #94743 , 0.4.6 PR is #118921

There is an update loop from 0.4.5 to 0.4.6

Steps to reproduce

try upgrade 0.4.5 to 0.4.6

Actual behavior

upgarde runs, but package again gets offered as updateable

Expected behavior

upgrade, and do not offer for upgrade again

Environment

winget --info
Windows-Paket-Manager v1.6.3482
Copyright (c) Microsoft Corporation. Alle Rechte vorbehalten.

Windows: Windows.Desktop v10.0.22631.3155
Systemarchitektur: X64
Paket: Microsoft.DesktopAppInstaller v1.21.3482.0

WinGet-Verzeichnisse
-----------------------------------------------------------------------------------------------------------------------
Protokolle                                   %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\Local…
Benutzereinstellungen                        %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\Local…
Verzeichnis für portierbare Links (Benutzer) %LOCALAPPDATA%\Microsoft\WinGet\Links
Portables Linkverzeichnis (Computer)         C:\Program Files\WinGet\Links
Portierbarer Paketstamm (Benutzer)           %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portierbarer Paketstamm                      C:\Program Files\WinGet\Packages
Portierbares Paketstamm (x86)                C:\Program Files (x86)\WinGet\Packages
Installationsprogrammdownloads               F:\Downloads

Links
-----------------------------------------------------------------------------------------
Datenschutzerklärung              https://aka.ms/winget-privacy
Lizenzvereinbarung                https://aka.ms/winget-license
Hinweise von Drittanbietern       https://aka.ms/winget-3rdPartyNotice
Startseite                        https://aka.ms/winget
Windows Store-Nutzungsbedingungen https://www.microsoft.com/en-us/storedocs/terms-of-sale

Administratoreinstellung                  Status
-----------------------------------------------------
LocalManifestFiles                        Deaktiviert
BypassCertificatePinningForMicrosoftStore Deaktiviert
InstallerHashOverride                     Deaktiviert
LocalArchiveMalwareScanOverride           Deaktiviert

Screenshots and Logs

No response

@markusdd markusdd added the Issue-Bug It either shouldn't be doing this or needs an investigation. label Feb 18, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Triage This work item needs to be triaged by a member of the core team. label Feb 18, 2024
Copy link
Contributor

Hi I'm an AI powered bot that finds similar issues based off the issue title.

Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it. Thank you!

Closed similar issues:

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

@markusdd

This comment was marked as resolved.

@mdanish-kh
Copy link
Contributor

Can you share the output of winget list openrct2?

@microsoft-github-policy-service microsoft-github-policy-service bot removed the Needs-Triage This work item needs to be triaged by a member of the core team. label Feb 18, 2024
@markusdd
Copy link
Author

markusdd commented Feb 18, 2024

Can you share the output of winget list openrct2?

sure, here you go:

PS C:\> winget list openrct2
Name                         ID                Version Verfügbar Quelle
-----------------------------------------------------------------------
OpenRCT2 0.4.5master-76ca840 OpenRCT2.OpenRCT2 0.4.5   0.4.6     winget

I think this is very much related to what we saw last time, the ID string is not just the version but some previous version with a branch and hash name.

Very much looks like the issue we had back then in the issue linked above.

EDIT: Whats new here is that the string says still 0.4.5, and when I launch the game it says 0.4.8 is already a available.

Packaging issue in general?

@markusdd
Copy link
Author

Just cross-checked the releases-section, the winget repo is indeed behind a lot: https://github.com/OpenRCT2/OpenRCT2/releases

@markusdd
Copy link
Author

ok so I just think the hash is wrong in the latest 0.4.6 commit here.
grafik

it's supposed to be b40b5da, but the PR mentioned above has this:
grafik

And all later versions have not been added. any automation that broke?

@mdanish-kh
Copy link
Contributor

That's not the issue I believe; It appears to be an installer issue. If you install the package manually from https://github.com/OpenRCT2/OpenRCT2/releases/download/v0.4.6/OpenRCT2-0.4.6-windows-installer-x64.exe in a clean VM or sandbox, it'll actually write an older version 0.4.5master-76ca840 to the Registry / Control Panel / Apps&Features. This should be reported to the publisher to update the DisplayVersion string written to registry.

[Policy] Installer-Issue
[Policy] Area-External

@microsoft-github-policy-service microsoft-github-policy-service bot added Area-External Installer-Issue Issue with the package's installer. labels Feb 18, 2024
@markusdd
Copy link
Author

ok, but what about the later versions?
There surely won't be a fix for 0.4.6 anymore as it is outdated anyway.

I'll try installing 0.4.7 and see what that reports

@mdanish-kh
Copy link
Contributor

Looks like 0.4.8 is writing the correct version to the Control Panel. I'll make a PR for that and remove 0.4.6

@markusdd
Copy link
Author

0.4.7 works as well.

any idea why they have not been added for so long? Isn't this automatic?

@mdanish-kh
Copy link
Contributor

0.4.7 works as well.

any idea why they have not been added for so long? Isn't this automatic?

Most packages are updated by the community. If you find a package outdated, you can create an issue for it to let contributors know that it's outdated or even better if you wish, you can submit a PR yourself. If you know the publisher, you can reach out to them to use manifest creator tools like wingetcreate / Komac and set-up CI that can automate the releases to the community repository.

@markusdd
Copy link
Author

did you use Komac just now to create the PR so quickly?

I have no issue to submit myself if I know how to do it.

@mdanish-kh
Copy link
Contributor

It was this one-liner 😄

wingetcreate update OpenRCT2.OpenRCT2 --version 0.4.8 --urls "https://github.com/OpenRCT2/OpenRCT2/releases/download/v0.4.8/OpenRCT2-0.4.8-windows-installer-win32.exe" "https://github.com/OpenRCT2/OpenRCT2/releases/download/v0.4.8/OpenRCT2-0.4.8-windows-installer-x64.exe" "https://github.com/OpenRCT2/OpenRCT2/releases/download/v0.4.8/OpenRCT2-0.4.8-windows-installer-arm64.exe" --submit --replace

Made a search with wingetcreate show OpenRCT2.OpenRCT2 -i first to get the old URLs and replace 0.4.6 with 0.4.8

@markusdd
Copy link
Author

I see. very helpful!

Will suggest this over at OpenRCT2

@mdanish-kh
Copy link
Contributor

I have no issue to submit myself if I know how to do it.

You can learn how to contribute by going through the contributing docs in this repo, or if you prefer video format, you can check out these Open at microsoft episodes

@mdanish-kh
Copy link
Contributor

@markusdd The PR got merged, has this issue been resolved for you?

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback This needs a response from the author. label Feb 20, 2024
@markusdd
Copy link
Author

I just tried and yes, it updated from 0.4.7 to 0.4.8 and does not get re-offered.

I opened the suggestion ticket over at OpenRCT2 to provide the manifests.

Thank you!

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs-Attention This work item needs to be reviewed by a member of the core team. and removed Needs-Author-Feedback This needs a response from the author. labels Feb 20, 2024
@markusdd
Copy link
Author

closing as resolved as far as winget goes.

@microsoft-github-policy-service microsoft-github-policy-service bot removed the Needs-Attention This work item needs to be reviewed by a member of the core team. label Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-External Installer-Issue Issue with the package's installer. Issue-Bug It either shouldn't be doing this or needs an investigation.
Projects
None yet
Development

No branches or pull requests

2 participants