I could not find a way to re-open the ticket, so I made a new one and included the patch: https://merb.lighthouseapp.com/projects/7433-merb/tickets/1297-sass-not-properly-loaded-by-merb-haml
..tony.. On Tue, Oct 13, 2009 at 10:52 AM, Jacques Crocker <[email protected]>wrote: > Go ahead and reopen. We should get this fixed for 1.1 Can you submit a > patch? I'll get it merged in > Thanks! > > > On Oct 13, 2009, at 10:49 AM, Tony Mann 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-work-out-of-the-box-needs-require-sass> > 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 -~----------~----~----~----~------~----~------~--~---
