-
-
Notifications
You must be signed in to change notification settings - Fork 260
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
winget support would be great #848
Comments
Here's the automation option I had in mind: https://github.com/vedantmgoyal2009/winget-pkgs-automation (apparently now being replaced by https://github.com/marketplace/actions/winget-releaser, example, source) I'll see if I can figure this out on my own time and will probably send a PR to you, if successful. |
many thanks @exoosh , if you find out how to do it that would be great. I have assigned it to you, but decline if you want to |
I will test this first in a repo of my own just to make sure I get it right. Then will "port" it over here with a PR. In all likelihood there will be some secrets involved that need to be created for the PRs to be posted to |
moveing this back to version 2.4 from version 2.3 - unless this is done @exoosh ? No problem either way |
Apologies. No it's not yet done. But I have indeed now dabbled with Windows only has the For the record the steps are probably going to be as follows:
Alas, there's a catch here. There is currently no OWASP/ThreatDragon manifest. But there needs to be one for Therefore I suggest you ping me here and I'll do the creation of that first manifest so that the automated action can run next time around. How does that sound? |
that sounds very good to me, thanks @exoosh , and agreed, just one asset and one architecture |
Cool, alright. So to summarize: for the upcoming release I will create the manifest more or less manually and for subsequent ones the GitHub action should take over. |
@jgadsden I'm silly, sorry. We can do the WinGet manifest right now (and get automation in place for the upcoming version before it gets released), I'd just like to get your feedback on my choices for the data to be entered. Please pay particular attention to the following fields below:
Also note that some items are populated based on your project settings. E.g. the tags. Created with Komac 2.6.0:
Draft version of manifest filesIn case I somehow don't respond, the following files ought to go under the name given in their subsection titles respectively into https://github.com/microsoft/winget-pkgs/tree/master/manifests/o/OWASP into a subdirectory
|
Hello @assarbad , thanks for all this - certainly it looks practical Some suggestions but I am not 100% sure:
|
I can create tokens sometime this week - are we able to try it out with a test tag? say 2.3.0-RC1 ? |
Generally yes, but I'd recommend putting that into a separate Github action (i.e. any pre-release channels). At least I wouldn't want to get pre-release versions on a channel I expect release versions on. Let me get back to you in the evening on my own time. |
Cool, I'll do these changes to the files above. I saw there was an older I'll have to survey existing RC versions on WinGet to get an idea what our options are. btw: some of the values are extracted by Komac, in particular some of those values I gave on the command line seem to have been overwritten by something else already. So my guess would be it extracts these from the installer file meta data or so. |
If we were talking about Windows Installer, we could logically separate the RC from the release versions (via |
agreed @assarbad , if I can help then say, but otherwise will follow what you think |
@assarbad did you want to manually create a winget installer for version 2.2.0 ? just to see if everything is in place? |
@jgadsden |
- This is meant to automate releases to winget-pkgs - Resolves OWASP#848 and should work as soon as microsoft/winget-pkgs#184453 has been merged (baseline manifest for the 2.2.0 version) - DO NOT MERGE BEFORE PREREQUISITES ARE IN PLACE (see below) ## Instructions for prerequisites The `WINGET_TOKEN` needs to be created and registered in the repo _which does the release_. For good measure we need to clone `winget-pkgs` on the same user (or org) or set via `fork-user` it as per the README for [winget-releaser](https://github.com/vedantmgoyal9/winget-releaser). Configure the [Pull app](https://github.com/apps/pull) to keep the `winget-pkgs` fork in sync with its upstream. That `winget-pkgs` fork is going to be the _source_ of the PR filed at `microsoft/winget-pkgs` and the PRs are being opened on behalf of the owner of the used token. 1. Head to https://github.com/settings/tokens and create a new (classic) personal access token with _only_ the `public_repo` scope activated for it 2. In https://github.com/OWASP/threat-dragon/settings/secrets/actions create a secret named WINGET_TOKEN (as per vedantmgoyal9/winget-releaser) 3. Merge this PR 4. Create a (subsequent) release
Describe what problem your feature request solves:
It would make the installation and upgrading the software on modern Windows very very easy.
Describe the solution you'd like:
I'd love to be able to say
winget install --id OWASP.ThreatDragon
or similar -- and of course subsequentlywinget upgrade --id OWASP.ThreatDragon
-- in order to install and upgrade the software.PS: I think there is a way to integrate this with GitHub automation, but I need to dig that up again and will comment once I find it. That way if the assets in your releases follow a particular schema a PR will automatically go to microsoft/winget-pkgs.
The text was updated successfully, but these errors were encountered: