-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
System.Text.Json: Method not found System.Text.Encodings.Web.TextEncoder.FindFirstCharacterToEncodeUtf8 stil happens in NuGet 5.0.0 ans 5.0.1 #49940
Comments
Tagging subscribers to this area: @eiriktsarpalis, @layomia Issue Details#31326 still happens in System.Text.Json Nuget 5.0.0 and 5.0.1
|
@ericstj Is this the same root cause as #49211? See also your earlier comment at #1460 (comment). |
@GrabYourPitchforks yes, same error |
There's also in issue on the xamrin.ios board: dotnet/macios#10912 |
Tagging subscribers to this area: @GrabYourPitchforks Issue DetailsStill happens with:
Background info, because this was discussed in another thread.
https://gist.github.com/juwens/d04e750ac0ad352f085da571b6a5482f
|
Keeping this open for now as tracking, but if it's a true dupe we can resolve. |
@juwens can you try this workaround:
That should confirm the issue and unblock you if necessary. |
Actually, the version should be 4.5.4 (the highest version in NuGet; possibly referenced from other NuGets):
|
With IncludeAssets="None"
I get a build error
Without IncludeAssets="None"
I get the exception again
|
While I'm currently creating a sample app, I noticed that I'm having trouble to reproduce the problem. |
It's exactly the same error as the ones linked from dotnet/macios#10928. I do have a repro app, several of them in fact. |
The triggering combination is usually a library that targets netstandard2.0 and references NuGet package System.Memory. This library can be a local one or a NuGet one. Once you include this library from Xamarin.Mac/iOS project it incorrectly overrides the built-in System.Memory.dll (version 4.0.99.0) with the one from the NuGet (version 4.0.1.1). The one from the NuGet is not meant to be used with Xamarin and causes errors at runtime for certain usage (but not always, some methods are self contained). |
Thanks a lot :) |
@filipnavara all of our shared libs are netstandard2.0 Do you have another hint where to look at? I'm somewhat lost. I've greped for
|
@juwens we understand what's going on and @filipnavara's fix should address this. One thing that seems to happen (occured for me when I tried to reproduce this) was that Xamarin's |
Still happens with:
Background info, because this was discussed in another thread.
Related: #1460 #31066 #31326
https://gist.github.com/juwens/d04e750ac0ad352f085da571b6a5482f
The text was updated successfully, but these errors were encountered: