gluck / il-repack

Open-source alternative to ILMerge
Apache License 2.0
1.16k stars 214 forks source link

Provide a dotnet global tool version of ilrepack #364

Closed kzu closed 1 month ago

kzu commented 2 months ago

A dotnet global tool of ilrepack would be much easier to install and use in a cross-platform manner.

This PR provides:

The workflows automatically detect and use two repository/org secrets:

An example run of the build workflow can be seen in action at https://github.com/kzu/il-repack/actions/runs/9846710011. And the published CI package for the global tool can be installed with:

dotnet tool install -g dotnet-ilrepack --prerelease --add-source https://kzu.blob.core.windows.net/nuget/index.json

And checked with: ilrepack --version.

KirillOsenkov commented 2 months ago

Thanks! I'll take a look soon. This would fix #289

ThaDaVos commented 1 month ago

Any ETA?

KirillOsenkov commented 1 month ago

I'm very busy these days but I'll try and take a look within a week or two.

KirillOsenkov commented 1 month ago

@kzu I'm not an admin on the repo so I can't set the NUGET_API_KEY to publish. I'm OK with building .nupkg locally and publishing those, since it's infrequent.

Is it the Environment Secret mentioned in the Actions here? image

image

Or maybe @gluck can help us set a secret in the Settings?

KirillOsenkov commented 1 month ago

I uploaded https://www.nuget.org/packages/dotnet-ilrepack/1.0.0 manually for now, could you please try it out?

MrJul commented 1 month ago

Not the OP, but I can confirm that the tool is working fine. Thanks!

kzu commented 1 month ago

@KirillOsenkov it should be the secret in settings, not in env... not sure both map equally 🤔 .

kzu commented 1 month ago

Published package works like a charm. If some day you the nuget secret in the repo, future publishing is just a matter of creating a release (for both packages) 👌