-
Notifications
You must be signed in to change notification settings - Fork 711
Does not work with AspNetCore.MVC 1.1.1 #78
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
Comments
+1 for ASP.NET Core 1.10 support (and 1.1.1 if required). |
I have some other immediate changes that need to go out this week too. I'll re-hash this. This looks like a simple assembly binding redirection misconfiguration. Which toolset are you using - Visual Studio 2015 or 2017? I want to make sure I can reproduce your setup and configuration. |
2015 |
To be clear (since you are both reporting it):
Don't get me wrong, I believe you. It seems very bizarre that a revision/patch version would cause this kind of failure. It would seem the ASP.NET team may have introduced a breaking change in package itself. A somewhat similar change was asked for in #66. I'm wondering if there is an ASP.NET Core package change and these two issues are related. I'll let you know what I find. |
I've done some more investigation into this issue. I'm not really sure the cause. I'll be updating the references to 1.1.0 and Microsoft.AspNetCore.Mvc.Core. The full MVC package isn't needed as mentioned in #66. The only thing that kind of leaped out as a possible culprit was the Microsoft.AspNetMvc.Mvc 1.1.0 updated it's references to NETStandard.Library 1.6.1 (from 1.6.0). Is 1.1.1 specifically needed or is 1.1.0 sufficient? I want to avoid publishing multiple packages until I have to. I also want to target the lowest supported version (e.g. 1.1.0) and let developers choose when they want to update to newer versions (e.g. 1.1.1). This is important for other library and tool authors as well. |
https://blogs.msdn.microsoft.com/dotnet/2017/01/30/january-2017-update-for-asp-net-core-1-1/
So you don't need to support |
Perfect! These are the details I was looking for. Thanks. Updating to 1.1.1 only. |
The text was updated successfully, but these errors were encountered: