-
-
Notifications
You must be signed in to change notification settings - Fork 322
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
No version number reported in windows #503
Comments
Hey @leochras, You don't need to upgrade using winget, as the Portmaster updates itself. The things the installer does barely change, so you don't have to run it again. |
I see. While no damage can be done to the Portmaster this way, we should definitely avoid it. Do you know which version number winget uses in this case? We don't even use the "Safing.Portmaster" ID within the installer as far as I know. The winget installer also in not managed by us, but by @OfficialEsco and others. (Thanks again!) Extra note for @OfficialEsco here: we are currently testing a dependency of the Windows installer, so we would be shipping an updated installer in 2-4 weeks. That would be a perfect time to include fixes from this issue. Ref to winget packaging: |
WinGet still uses However we've had this chat before, and this might actually be the only application i agree having no DisplayVersion makes sense because the installation is pretty much never being touched. @leochras at this time these are your options
|
Thanks @OfficialEsco for chiming in here. Also, sorry @leochras, I didn't connect the dots before that you are now also involved with WinGet. Thinking about this more, I think it does make sense to include the version number, as there will be some (even if very vew) updates in the future, that require re-running the installer - or are the cleanest solution, even if possible differently. |
To be clear, I also think this is a problem with winget and not portmaster itself. Thank you both for your efforts! |
Assuming the original need was handled, this will be automatically closed now. |
Sorry, issue was closed by misconfigured bot. |
Hey, have you given this some more thought?
|
Thanks for the nudge - I've actually already done that with the latest version of the installer, but I had to withdraw that update again, as a dependency had a bug that prevented proper installation. |
The Portmaster Windows Installer is now out with version v0.8.8. Please check if the version number is recognized correctly. |
I can't seem to find the exercutable builds, downloading the newest version from your website, the app still doesn't report it's version number.. |
He did a sneaky non-public release, but since he said 0.8.8 its https://updates.safing.io/windows_amd64/packages/portmaster-installer_v0-8-8.exe edit: oh yeah https://updates.safing.io/latest/windows_amd64/packages/portmaster-installer.exe downloads 0.8.8 too |
|
Oh, right, I forgot to tag the version, my bad. Did that now. Hm. Bummer. I guess I'll have to look into it again. |
OHHHHHHHHH I see what you have done safing/portmaster-packaging@4de0b78 |
Yes, I had hoped for the NSIS installer system to be, well, more intelligent. |
Thanks again for the PR, @OfficialEsco, it is now released as version v0.8.8.1. It worked for me, but would be nice to get confirmation from you two! |
Already confirmed it works =) Update while typing: apparently https://updates.safing.io/windows_amd64/packages/portmaster-installer_v0-8-8.exe downloads 0.8.8.1 I'm guessing in the future you will build the installer and release it on GitHub, however until then could you include a direct download link to the installer in the release notes? However since you added the FileVersion and ProductVersion we could move over to https://updates.safing.io/latest/windows_amd64/packages/portmaster-installer.exe since IIRC all the installer just downloads the latest version anyways? (I would probably recommend you to add something to stop it from doing that as Privacy people can be.. difficult) |
What happened:
The most recent version of the portmaster installer still doesn't report the version number to windows:
What did you expect to happen?:
The installer should report the version of portmaster to windows.
This would make upgrades with winget possible. As of now, it only says that the version is "unknown" and this makes upgrading impossible.
How did you reproduce it?:
Debug Information:
Version 0.7.14
Platform: Microsoft Windows 11 Pro 10.0.22000 Build 22000
Status: Trusted
Resolvers: 2/2
No Module Error
Unexpected Logs
Goroutine Stack
The text was updated successfully, but these errors were encountered: