Skip to content
This repository was archived by the owner on Dec 14, 2018. It is now read-only.

* Remove support for updateable precompiled views. #2917

Closed
wants to merge 2 commits into from

Conversation

pranavkm
Copy link
Contributor

@pranavkm pranavkm commented Aug 5, 2015

  • Allow precompiled views to be served when source file does not exist in
    file system.
  • Cache results for views that do not exist on disk.

Fixes #2462 and fixes #2796.
Partially addresses #2551

* Allow precompiled views to be served when source file does not exist in
  file system.
* Cache results for views that do not exist on disk.

Fixes #2462 and fixes #2796.
Partially addresses #2551
@pranavkm
Copy link
Contributor Author

pranavkm commented Aug 5, 2015

cc @Alxandr @rynowak @loudej

@@ -12,7 +13,9 @@ public class Startup
public void ConfigureServices(IServiceCollection services)
{
// Add MVC services to the services container
services.AddMvc();
services
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@rynowak what's the best way to do this?

Copy link
Member

Choose a reason for hiding this comment

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

For now, make an extension method on IMvcBuilder and on IServiceCollection. The IServiceCollection part will go away soon.

@rynowak
Copy link
Member

rynowak commented Aug 6, 2015

😻

IAssemblyLoadContextAccessor loadContextAccessor,
IOptions<RazorViewEngineOptions> optionsAccessor)
: this(razorFileInfoCollections,
loadContextAccessor.GetLoadContext(typeof(CompilerCache).GetTypeInfo().Assembly),
Copy link
Member

Choose a reason for hiding this comment

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

would help readability to put this back in a private field. loadContextAccessor.GetLoadContext(RazorHostAssembly) > current

@rynowak
Copy link
Member

rynowak commented Aug 6, 2015

@rynowak
Copy link
Member

rynowak commented Aug 7, 2015

:shipit:

@pranavkm pranavkm closed this Aug 7, 2015
@pranavkm pranavkm deleted the prkrishn/precompilation branch August 7, 2015 18:12
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.

4 participants