Fix Microsoft.AspNetCore.Mvc.Dnx to avoid referencing System.IO/Reflection/Runtime

[kevinchalet]

Microsoft.AspNetCore.Mvc.Dnx currently references System.IO, System.Reflection and System.Runtime at the global dependency level, which causes weird issues on .NET Desktop: https://github.com/dotnet/cli/issues/1993#issuecomment-200512200

@pranavkm do you remember why you had to add these packages?

https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.Dnx/project.json#L20-L22

/cc @davidfowl

[pranavkm]

There was an issue with the CLI where it would add multiple entries to the project.lock.json (a package reference and a framework reference) which would cause dotnet build to fail. This was addressed in NuGet/NuGet.Client#389, but we haven't updated to a build of the CLI with the fix (it was a bit unstable when I checked last week). We're working on updating our packages to the newest CoreFx packages and perhaps that might resolve the the problem that surfaced the restore bug. We'd be able to address it once that's taken care of.

[kevinchalet]

Thanks for the explanation 👏

[Eilon]

@pranavkm also just parking this on your plate until we can verify.

[pranavkm]

Looks like updating the packages wasn't enough. We're working on updating the cli next which should allow us to remove this hack..

[pranavkm]

Fixed in 330e98d

[pranavkm]

Fixed in 330e98d