aspnet / DependencyInjection

[Archived] Contains common DI abstractions that ASP.NET Core and Entity Framework Core use. Project moved to https://github.com/aspnet/Extensions
Apache License 2.0
873 stars 318 forks source link

Disable IL backend by default #636

Closed pakrym closed 6 years ago

pakrym commented 6 years ago

https://github.com/aspnet/Home/issues/3054 seems to be severe and can affect existing customers.

Eilon commented 6 years ago

So we're keeping the feature in general? E.g. no need to remove https://github.com/aspnet/DependencyInjection/blob/2d3decdfe47b7e685c41b2bf082ac138538c2cc0/src/DI/ServiceProvider.cs#L35 ?

davidfowl commented 6 years ago

So we're keeping the feature in general? E.g. no need to remove

It's not being removed completely, we're just removing the default for the "compiled" mode. Before it would default to ref emit based on the platform.

Eilon commented 6 years ago

OK