-
Notifications
You must be signed in to change notification settings - Fork 220
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
"Specified method is not supported" on net5.0 #273
Comments
+1 |
Related to #236 |
Blocked by gluck/il-repack#273
bump, I believe this is fixed in Cecil now. Would be great to have an update for this as it's blocking .net 5 support for several of my packages. |
actually, I rolled back my .Net 5 support and am still seeing the error in ILRepack 2.0.13.
|
This saddens me... ILRepack is using an old version of Cecil, which throws an exception when being used on .NET 5 gluck/il-repack#236 gluck/il-repack#273 It's supposedly fixed in the up-to-date versions of Cecil though jbevain/cecil#669 On the other hand... it's already fixed in a fork of ILRepack (but that one got archieved 16 days ago and no new nuget package has been made available 😭) peters/il-repack#2
This saddens me... ILRepack is using an old version of Cecil, which throws an exception when being used on .NET 5 gluck/il-repack#236 gluck/il-repack#273 It's supposedly fixed in the up-to-date versions of Cecil though jbevain/cecil#669 On the other hand... it's already fixed in a fork of ILRepack (but that one got archieved 16 days ago and no new nuget package has been made available 😭) peters/il-repack#2
Is there any progress on this issue? |
Is there any update? |
I recommend the following alternative: add this to your csproj
and then place lines like the following in the folder where the csproj lives (the filename must be
This worked for me on .NET Core all the way to 6.0 and probably beyond. |
Any chance this can work in .net 6.0? Works fine locally in VS 2022, but get this error in Azure Devops. vmimage: windows-latest |
I have the exact same issue as @aalmada Edit: I should also mention to be clearer, that building from VS2022 succeeds and I can see the two name spaces when opening the combined DLL in a decompiler (e.g., dotPeek), but VS cannot pickup the methods in the combined DLL. I then attempted to
Line 31 from the project file shows ILRepack is not understood?
The project is referencing
|
I could not get ILRepack to work, but there was a more recent nuget that did the trick: https://www.nuget.org/packages/ILRepack.Lib.MSBuild/2.1.18/ However, VS2022 still will not recognize the different DLL namespaces that were merged although dotPeek has no problem reading them and their contents. EDIT: I eventually got VS2022 to recognize the merged namespaces. In my case, it has something to do with the new console template for This other repack also works: https://github.com/ravibpatel/ILRepack.Lib.MSBuild.Task (It is a little nicer, in my opinion, as you are able to modify targets and post build task.) |
I have the same issue with GitHub Actions. It works fine locally using ILRepack.MSBuild.Task on Visual Studio 2022. Action report: https://github.com/NetFabric/NetFabric.Hyperlinq.Analyzer/actions/runs/5542153526/jobs/10116466992?pr=66 |
This should work fine with the latest ILRepack version: https://www.nuget.org/packages/ILRepack/2.0.23 |
I'm using
ILRepack.MSBuild.Task 2.0.1
on.NET 5.0.100-preview.7
with target frameworknetstandard2.0
.I'm getting the following error when I run
dotnet build
on the command line:I don't get the error when I build inside Visual Studio.
I reported the issue on Cecil (jbevain/cecil#669) but it seems the issue is with ILRepack.
The text was updated successfully, but these errors were encountered: