MrDys / blacklight

Blacklight Plugin
http://projectblacklight.org/
Other
1 stars 1 forks source link

config/initializers/sass requires sass-rails, even in production #563

Open MrDys opened 12 years ago

MrDys commented 12 years ago

CODEBASE-392: sass-rails required in production

jcoyne: is config/initializers/sass.rb something BL generates? james_s: yeah. jcoyne: Hmm. that seems to require 'compass' even in production mode. james_s: Oh, that's a good point. jcoyne: but my gemfile looks like this: jcoyne: # Gems used only for assets and not required jcoyne: # in production environments by default. jcoyne: group :assets do jcoyne: gem 'sass-rails', '~> 3.1.5' james_s: I'll file a bug to investigate this. My guess is that we should wrap that in a: james_s: if Rails.env.development? || Rails.env.test? jcoyne: Yeah, I keep getting LoadError: no such file to load -- compass jcoyne: i production mode james_s: Right. james_s: Try putting that if statement around lines 1-3. jcoyne: That works, but it seems like there is a better conditional to use. jcoyne: see application.rb line 5 jcoyne: some people want to compile assets in production. jcoyne: some will precompile james_s: Right. Hmm. james_s: This might change as well with how the new compass-rails.

MrDys commented 12 years ago

Original reporter: tastyhat