-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
D4koon.WhatsappTray version 1.9.0.0 #82559
D4koon.WhatsappTray version 1.9.0.0 #82559
Conversation
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
@microsoft-github-policy-service agree |
There is a related feature request in WhatsappTray project: D4koon/WhatsappTray#145 |
Hello @wingetbot! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
medvedev125, The check-in policies require a moderator to approve PRs from the community. Our moderators are community volunteers, please be patient and allow them sufficient time to review your submission. Template: msftbot/requiresApproval/moderator |
- Architecture: x64 | ||
InstallerType: inno | ||
InstallerUrl: https://github.com/D4koon/WhatsappTray/releases/download/v1.9.0/WhatsappTrayV1.9.0.0.exe | ||
InstallerSha256: 6535FA20F12FD21C0ABD104E5594D4F0AE82528D4274D3470D9C1FBB3C403A18 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
InstallerSha256: 6535FA20F12FD21C0ABD104E5594D4F0AE82528D4274D3470D9C1FBB3C403A18 | |
InstallerSha256: 6535FA20F12FD21C0ABD104E5594D4F0AE82528D4274D3470D9C1FBB3C403A18 | |
ProductCode: "{1237D679-445B-4101-B7EC-B5081C5FA4A3}_is1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why we should add the ProductCode field? Where do you get this ID? And why "_is1"?
There is not in manifest of other apps. For example: https://github.com/microsoft/winget-pkgs/blob/master/manifests/w/WhatsApp/WhatsApp/2.2236.10/WhatsApp.WhatsApp.installer.yaml
I created this commit with the wingetcreate util. And if this field is obligated, than it's a bug in the wingetcreate.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why we should add the ProductCode field?
It's required for this package because the developer didn't add the publisher for this application.
It's used so that WinGet can properly map the application to winget list
and make winget upgrade
work.
You can take a look at https://github.com/microsoft/winget-pkgs/blob/master/doc/manifest/schema/1.2.0/installer.md
winget-create already does the ProductCode
automatically for MSI installers, but for EXE installers, there's really no easy way other than to install the application to a virtual machine (VM) and monitoring the Windows Registry.
Where do you get this ID?
It's taken from the Windows Registry.
You can take a look at https://github.com/ItzLevvie/winget-pkgs-validate-and-install/blob/f4db043f8c11953c96c7976b90ec486f992933f6/Community%20Moderators%20-%202022-10-02%20-%20development.ps1#L185-L200
And why "_is1"?
That's just what Inno Setup does — which is what this application is wrapped around with to create an installer.
Hello @medvedev125, The package manager bot determined changes have been requested to your PR. Template: msftbot/changesRequested |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see #82559 (comment)
Hello @medvedev125, The package manager bot determined changes have been requested to your PR. Template: msftbot/changesRequested |
Close with reason: Replaced by PR 82817 to avoid having the pull request be stale; |
winget validate --manifest <path>
?winget install --manifest <path>
?Note:
<path>
is the name of the directory containing the manifest you're submitting.Microsoft Reviewers: Open in CodeFlow