8000 Update WinGet package as part of release workflow by electricbrass · Pull Request #1247 · odamex/odamex · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update WinGet package as part of release workflow #1247

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

Open
wants to merge 2 commits into
base: stable
Choose a base branch
from

Conversation

electricbrass
Copy link
Contributor
@electricbrass electricbrass commented Apr 16, 2025

There is an existing WinGet package for Odamex, but it lags behind in updates by quite a bit. This PR is to make this package something officially maintained and updated as part of releases. When a new release is made, a PR is automatically made to https://github.com/microsoft/winget-pkgs with an updated manifest for the new version. Here is an example of such a PR: microsoft/winget-pkgs#248963 (note that once microsoft/winget-pkgs#243548 is merged, the new manifests created by this will be based on that one, rather than the 10.6.0 manifest that is currently the latest. This one properly indicates that Odamex is available for both x86 and x64, and allows installing per user or for all users)

For this to work, a fork of https://github.com/microsoft/winget-pkgs will need to be made under the odamex organization, and this repository will need a GitHub Personal Access Token (classic) with permissions set to public_repos as a secret with the name WINGET_TOKEN.
If there is an odamex service account, the access token should probably be from that, rather than someones personal account.

@electricbrass electricbrass marked this pull request as draft April 17, 2025 07:52
@electricbrass
Copy link
Contributor Author

I'm going to look into how GitHub Apps work, and see if we can use that as a solution for the PAT.

@kris6673
Copy link

Until you get this working, I have an automation set up to run every 4 hours that checks if there is a new version available, that's not been added to the winget repo :)
I'll remove it when this gets merged.

@electricbrass
Copy link
Contributor Author

This will not work with a GitHub App, but I did just remember that @odamex-system exists. So that should work for what's needed here.

@electricbrass electricbrass marked this pull request as ready for review April 17, 2025 16:49
@electricbrass electricbrass added this to the Odamex 11.1.0 milestone Jun 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0