Skip to content

Conversation

akoeplinger
Copy link
Member

@akoeplinger akoeplinger commented Jun 23, 2021

Fixes #4237

See dotnet/designs#222 (comment) for context.

@mconnew I verified these packages restore using the net6.0 assets in the HelloAndroid sample from https://github.com/dotnet/maui-samples and at least a basic call to a SOAP webservice works.

@akoeplinger akoeplinger requested a review from mconnew June 23, 2021 16:29
@mconnew
Copy link
Member

mconnew commented Jun 23, 2021

@ericstj, can you comment on whether we need to add net6.0 to the Ref projects too? This work is to make the latest Xamarin platform which is based on .NET 6.0 (as opposed to mono) to use the WCF runtime assets from our packages instead of the previous Xamarin in-box mono based implementation.

@akoeplinger
Copy link
Member Author

Yes we do need them in the Ref projects too since the packages contain ref/MonoAndroid10/_._ right now as well, so we need the same explicit net6.0 asset as for lib.

@akoeplinger
Copy link
Member Author

@mconnew this should be good to go now, Eric gave his thumbs up :)

@ericstj
Copy link
Member

ericstj commented Jun 28, 2021

Yes, to state explicitly: please make parallel changes to TFM additions in ref/src. You need both here.

@mconnew mconnew merged commit 4d7e90e into dotnet:main Jul 8, 2021
@akoeplinger akoeplinger deleted the net6 branch July 9, 2021 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add net6.0 TFM to packages

3 participants