-
Notifications
You must be signed in to change notification settings - Fork 33
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
Does not work as described in readme #53
Comments
The default targets file that is supplied with NuGet package should work with Release configuration. Can you tell me which .NET Framework version are you using, so I can try to reproduce it? |
That may be my mistake as I assumed it would apply to both debug and release and I'm not sure if I actually tested the release build. |
You can do just like I did it in default targets file. It will include the executable itself and all the DLL files in the output directory. <Target Name="ILRepack" AfterTargets="Build">
<ItemGroup>
<InputAssemblies Include="$(OutputPath)$(TargetName)$(TargetExt)"/>
<InputAssemblies Include="$(OutputPath)*.dll" Exclude="$(OutputPath)$(TargetName)$(TargetExt)"/>
</ItemGroup>
<ILRepack
Parallel="true"
DebugInfo="true"
AllowDuplicateResources="false"
InputAssemblies="@(InputAssemblies)"
TargetKind="SameAsPrimaryAssembly"
KeyFile="$(KeyFile)"
OutputFile="$(OutputPath)$(TargetName)$(TargetExt)"
/>
</Target> |
Also does not work by default for me, does nothing. Using the recommended targets file outputs " not recognized". I tried everything recommended and I'm giving up on this one. |
@Webslug Is it possible to provide a project where I can reproduce this on my PC? |
Never mind, sorry I got it working. Apologies. |
You cannot simply install the NuGet package and it will automatically merge all of the dependencies.
it does nothing unless you create an ILRepack.targets file, which (as far as I can tell) requires manually defining the path of each dependency to merge.
It would be nice if it worked automatically, like the readme says, or at least was able to automatically detect the dependencies to merge, since it's easy to forget adding a newly added dependency to the targets file.
The text was updated successfully, but these errors were encountered: