Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Move Thread to shared CoreLib #35462

Merged
merged 1 commit into from
Feb 21, 2019
Merged

Conversation

filipnavara
Copy link
Member

Rather than having Thread partially live in CoreFX and call into an RuntimeThread type in CoreLib that in turn calls into an Thread type in CoreLib, we're just moving Thread to live in CoreLib. Discussed in dotnet/coreclr#22032.

To start that, this PR moves Thread and its dependencies from their current locations into the shared CoreLib. Follows the process established by #34654.

cc: @jkotas, @marek-safar, @stephentoub

Copy link
Member

@stephentoub stephentoub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks :)

@filipnavara filipnavara changed the title WIP: Move Thread to shared CoreLib Move Thread to shared CoreLib Feb 20, 2019
@stephentoub stephentoub merged commit 694eeba into dotnet:master Feb 21, 2019
Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/coreclr that referenced this pull request Feb 21, 2019
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/corert that referenced this pull request Feb 21, 2019
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/mono that referenced this pull request Feb 21, 2019
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
jkotas pushed a commit to dotnet/coreclr that referenced this pull request Feb 21, 2019
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
jkotas pushed a commit to dotnet/corert that referenced this pull request Feb 21, 2019
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
marek-safar pushed a commit to mono/mono that referenced this pull request Feb 21, 2019
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
@karelz karelz added this to the 3.0 milestone Mar 18, 2019
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
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.

3 participants