Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Don't eagerload the app directory #8150

Closed
wants to merge 1 commit into from

3 participants

@thiagofm

From issue: #8146

@steveklabnik
Collaborator

This doesn't merge cleanly.

@thiagofm thiagofm closed this
@thiagofm

@steveklabnik thanks, will take a deeper look.

@rafaelfranca
Owner

Also this is not the proper fix, the app directory needs to be eager loaded by default, until you say that you don't want

@thiagofm

@rafaelfranca alright, thanks for the info.

@adammathys adammathys referenced this pull request from a commit in freerunningtech/sunspot
@adammathys adammathys Add to_s to Rails.version
Due to rails/rails#8150.

Change-Id: Id81d89bf60ad6dc47057e65df5e3939a6427e80e
5c12858
@adammathys adammathys referenced this pull request in sunspot/sunspot
Closed

Add to_s for Rails.version #389

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Nov 8, 2012
  1. @thiagofm
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  railties/lib/rails/engine/configuration.rb
View
2  railties/lib/rails/engine/configuration.rb
@@ -38,7 +38,7 @@ def generators #:nodoc:
def paths
@paths ||= begin
paths = Rails::Paths::Root.new(@root)
- paths.add "app", :eager_load => true, :glob => "*"
+ paths.add "app", :eager_load => false, :glob => "*"
paths.add "app/assets", :glob => "*"
paths.add "app/controllers", :eager_load => true
paths.add "app/helpers", :eager_load => true
Something went wrong with that request. Please try again.