Hampton, I agree the workaround is easy. But the Merb docs specifically say that you do not need to load sass yourself, thus the current behavior is a bug. And the fix is trivial.
..tony.. On Wed, Oct 14, 2009 at 5:21 AM, hampton catlin <[email protected]> wrote: > > To get around this, just do > > require 'sass' > > as one of the first things in your init.rb file. > > On Oct 14, 12:06 am, Tony Mann <[email protected]> wrote: > > My guess is that somewhere in your call chain you are loading sass, which > > avoids the problem. > > In any case, if you look at sass.rb, it checks to see if Merb.plugins is > > defined, and then loads sass/plugins itself. So clearly the intent was > that > > a plugin should load sass.rb, not plugins/sass.rb. > > > > ..tony.. > > > > On Tue, Oct 13, 2009 at 3:21 PM, Nicholas Orr > > <[email protected]>wrote:> > Hmm, > > > I don't have an issue compiling sass :/ > > > In my gem file I have > > > > > gem "haml", "2.2.0" > > > gem "merb-haml", merb_gems_version > > > > > gem "chriseppstein-compass", "0.8.16", :require_as => "compass" > > > > > I also use Compass so maybe the combo of all three have everything > work?? > > > In my project I also wanted sass compiled from string input as well as > > > files on the filesystem and made additions that I load in init.rb > > >http://github.com/MagnitudeCMS/magnitudecms/blob/master/config/init.r. > .. > > > > > Maybe you are leaving something out as to what you are doing... > > > > > (I'm using Merb 1.1 with bunlder for above) > > > > > Nick > > > > > On Wed, Oct 14, 2009 at 4:49 AM, Tony Mann <[email protected]> > wrote: > > > > >> Supposedly in Merb 1.0.12 you do not need a specify a dependency on > Sass > > >> as long as you have a dependency on merb-haml. However, this is not > correct. > > >> If you don't explcitly require sass.rb, you will get the error: > > > > >> NameError: uninitialized constant Sass::VERSION > > > > >> when Sass tries to compile your stylesheets. > > > > >> The problem is that merb-haml.rb requires "sass/plugin", when it > should > > >> require "sass". Sass::VERSION is defined in sass.rb. > > > > >> I was about to file this as a ticket in Lighthouse when I saw this > closed > > >> ticket< > https://merb.lighthouseapp.com/projects/7433/tickets/1073-sass-doesnt...> > which > > >> describes the same problem. Should I reopen this ticket or file a new > one? > > > > >> The workaround for now is to explicitly require Sass. The right fix is > to > > >> change merb-haml to require sass and not sass/plugin. > > > > >> ..tony.. > > > > > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "merb" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/merb?hl=en -~----------~----~----~----~------~----~------~--~---
