-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
CoreFx test utility depends on Microsoft.DotNet.PlatformAbstractions #25681
Comments
We cannot depend on the latest PlatformAbstractions package which is built from core-setup as it adds a cycle which we cannot have in our source-build. To fix this temporarily we are pinning to the last stable version 2.0.4. See issue https://github.com/dotnet/corefx/issues/28620.
See dotnet/corefx@fa9fe16 for how I plan to pin it for now. |
I would absolutely LOVE for this to happen. However, I don't think it should go wholesale into corefx. I think pieces of it should be removed altogether (I'm looking at you HashCodeCombiner and Platform). And some things should be folded into ApplicationEnvironment can probably be deleted as well now that we have So thinking about it more - probably move RuntimeEnvironment into RuntimeInformation, and delete the rest. Obviously the library will still live in servicing branches, if we ever need to service the already released versions of it. And users can continue to use the already released versions all they want. |
We cannot depend on the latest PlatformAbstractions package which is built from core-setup as it adds a cycle which we cannot have in our source-build. To fix this temporarily we are pinning to the last stable version 2.0.4. See issue https://github.com/dotnet/corefx/issues/28620.
We cannot depend on the latest PlatformAbstractions package which is built from core-setup as it adds a cycle which we cannot have in our source-build. To fix this temporarily we are pinning to the last stable version 2.0.4. See issue https://github.com/dotnet/corefx/issues/28620.
[if former offsets the latter] maybe we can try to think it in light of the proposed API at https://github.com/dotnet/corefx/issues/27417 for public consumption when bringing over |
Looks like we have a few issues that are tracking similar things https://github.com/dotnet/corefx/issues/16629, https://github.com/dotnet/corefx/issues/17452, I hope we can tackle a number of them as part of moving these APIs into corefx. |
@krwq @tarekgh @eerhardt added a dependency on Microsoft.DotNet.PlatformAbstractions to CoreFx.TestUtilities in dotnet/corefx#24531. That cause a cycle in our build-from-source efforts so we have to figure out a solution to break that cycle.
For now I'm going to break the cycle by pinning to an existing version but we cannot do that in general because it requires us to have an existing binary which is bad for new-distro bring-up.
@eerhardt what is your thoughts on bring Microsoft.DotNet.PlatformAbstractions into corefx?
The text was updated successfully, but these errors were encountered: