Skip to content

Conversation

@jonathanpeppers
Copy link
Member

This feature flag is now on by default in .NET 6.0.100-preview.4.21253.14 and higher.

We should stop generating EnableWorkloadResolver.sentinel, as it strongly ties our installers to a specific version of the .NET SDK.

@jonathanpeppers jonathanpeppers force-pushed the remove-enableworkloadresolver.sentinel branch from 3b0ea8e to 24311f7 Compare May 5, 2021 12:58
@jonathanpeppers
Copy link
Member Author

I think we actually need a newer preview 5 build for this:

error NETSDK1139: The target platform identifier android was not recognized.

This feature flag is now on by default in .NET 6.0.100-preview.4.21253.14 and higher.

We should stop generating `EnableWorkloadResolver.sentinel`, as it strongly ties our installers to a specific version of the .NET SDK.
@jonathanpeppers jonathanpeppers force-pushed the remove-enableworkloadresolver.sentinel branch from 24311f7 to ad7ca58 Compare May 7, 2021 21:31
@jonathanpeppers jonathanpeppers requested a review from pjcollins May 10, 2021 13:11
@jonathanpeppers jonathanpeppers marked this pull request as ready for review May 10, 2021 13:12
@jonathanpeppers jonathanpeppers requested a review from jonpryor as a code owner May 10, 2021 13:12
@jonathanpeppers
Copy link
Member Author

Only test failures are related to <Lint/>.

And this one seems like it is maybe a random issue: System.Threading.ThreadPools.Tests.ThreadPoolTests.QueueRegisterPositiveAndFlowTest

@jonathanpeppers jonathanpeppers merged commit 2538c23 into dotnet:main May 10, 2021
@jonathanpeppers jonathanpeppers deleted the remove-enableworkloadresolver.sentinel branch May 10, 2021 19:32
jonathanpeppers added a commit to jonathanpeppers/maui that referenced this pull request May 13, 2021
Context: dotnet/android#5909

This file is no longer needed in .NET 6 Preview 4 and higher. We should stop creating it now.
rmarinho pushed a commit to dotnet/maui that referenced this pull request May 13, 2021
Context: dotnet/android#5909

This file is no longer needed in .NET 6 Preview 4 and higher. We should stop creating it now.
lytico pushed a commit to lytico/maui that referenced this pull request May 15, 2021
Context: dotnet/android#5909

This file is no longer needed in .NET 6 Preview 4 and higher. We should stop creating it now.
@github-actions github-actions bot locked and limited conversation to collaborators Jan 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants