less-rails breaking sass @import rule on rails 3 about less-rails HOT 5 CLOSED

I tried writing our import preprocessor (https://github.com/metaskills/less-rails/blob/master/lib/less/rails/import_processor.rb) in such a way that it only acts on less files. My goal was it would noop in all other situations. The preprocessor is critical for us as it is the only way for us to hook in dependencies on other less assets. Is there a way it should be written to be more friendly? Maybe @chriseppstein can help too?

from less-rails.

I just published less-rails v2.1.4 that touches the import preprocessor. Does that help this issue too?

from less-rails.

Maybe I missed it, but I didn't see any change that would cause the import processor to be selectively ran on less files.

The way that sass handles this is by making the imports resolved during template evaluation.

from less-rails.

I can not go that route since less's template evaluation is in the JavaScript. We have monkey patched that far down for our rails asset helpers, but found that doing so for import is much more difficult. Hence the preprocessor was our best choice.

Is this a good enough guard?
https://github.com/metaskills/less-rails/blob/master/lib/less/rails/import_processor.rb#L18

from less-rails.

@metaskills It works now with 2.1.4. You can close the issue.
Thanks.

from less-rails.