Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

Add reference to Microsoft.NETCore.Platforms package to workaround A SDK issue #808

Merged
merged 1 commit into from
Oct 22, 2019

Conversation

Lxiamail
Copy link
Member

There is a SDK issue in .NET Core 2.* (https://github.com/dotnet/cli/issues/12341). The issue results with ilasm.exe is not copied to Microsoft.Fx.Portability.MetadataReader.Tests\net461\win7-x86 output directory as it should be. Add reference to Microsoft.NETCore.Platforms package to workaround the issue. The issue has been fixed in .NET Core 3.0 SDK (verified on 3.0.100). The reference should be removed once build machine moves to .NET Core 3.0.

Copy link
Contributor

@marklio marklio left a comment

Choose a reason for hiding this comment

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

Seems fine. FWIW, the model for "moving the build machine to .NET Core 3" is to explicitly choose the 3.0 SDK as part of the build process.

@Lxiamail
Copy link
Member Author

The test failure is not related with this PR and has been fixed.

@Lxiamail Lxiamail closed this Oct 21, 2019
@Lxiamail Lxiamail deleted the workaroundilasmSDKissue branch October 21, 2019 21:22
@Lxiamail Lxiamail restored the workaroundilasmSDKissue branch October 22, 2019 17:09
@Lxiamail Lxiamail reopened this Oct 22, 2019
@Lxiamail Lxiamail merged commit ee03621 into dev Oct 22, 2019
@Lxiamail Lxiamail deleted the workaroundilasmSDKissue branch October 25, 2019 19:41
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