search

gluck / il-repack

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

Possible Internalize regression #357

Closed Aragas closed 5 months ago

Aragas commented 5 months ago

Good day! It seems that there's a regression in v2.0.29 with internalize. Simply setting the --internalize flag triggers an error "Input file not found: --internalize" Based on the recent commits, I believe the idea was to add an optional list of files with the flags, but it might not work now when no files are provided at all

mattleibow commented 5 months ago

Not sure if this is related, but the Mono.Cecil types are no longer internal/hidden. They are now conflicting with my Mono.Cecil package reference.

mattleibow commented 5 months ago

Seems they are public from 2.0.24. At least that is the version I tested. 2.0.18 was still internal.

KirillOsenkov commented 5 months ago

OK will look into internalizing Cecil, sorry about all the regressions

KirillOsenkov commented 5 months ago

Filed #358 to track it separately

KirillOsenkov commented 5 months ago

Published 2.0.30, please try it out: https://www.nuget.org/packages/ILRepack/2.0.30 https://www.nuget.org/packages/ILRepack.Lib/2.0.30