-
Notifications
You must be signed in to change notification settings - Fork 276
rails 4 beta support #59
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
👍 |
No issues with this branch so far for me :) |
I hope to get this merged soon been super busy latley On Fri, Sep 21, 2012 at 12:36 PM, Ian MacLeod notifications@github.comwrote:
|
👍 |
What's missing for getting merged in? |
My free time =( Sent from my iPhone On Oct 13, 2012, at 12:46 AM, Olivier Lacan notifications@github.com wrote:
|
@slbug Any chance of an update? Seems |
Will try soon. Need to dig into sprockets integration |
+1 |
1 similar comment
👍 |
👍 |
+1 |
I need this fix too. |
@slbug when I use your fork on edge Rails, and sprockets-rails 2.0.0.rc1, I get |
@slbug problem is that you fork stable branch, not master, where there isn’t static compiler patch. |
@ai, i've used master branch, but static compiler was removed after i created this pull request. And atm i don't have time to update it. |
@slbug after I fix static compiler issue I have abother problem: |
The second problem in Rails 4 and Compass is that Sass template compiler is now directly in the I create really quick hack to enable Compass in Rails 4. |
Quick, potentially obselete comment: whatever the solution with integration Rails 4 is, Compass ought to provide a generalized interface for files generated in the compilation process, and not just particularly for sprite images. I had enough of a wretched time mimicking the Rails 3 + compass-rails hack to get something similar to auto-sprite-generation working, so whatever the Rails 4 integration, I'd just make sure Compass plugins have some api for registering their mid-compile generated assets. |
Just want to keep everyone in the loop this is on my list of things to get done. We are trying to set up a hack weekend to get the next version of compass out the door. Once that has happened i will get rails 4 support released |
Awesome, I'm having no luck with the hack posted by ai. |
omg when will this be merged :D |
@milgner tried your branch and it works for me. thanks! |
def write_manifest_with_sprites(manifest) | ||
write_manifest_without_sprites(manifest.merge(self.class.generated_sprites)) | ||
if CompassRails.rails4? | ||
require 'sprockets/rails/static_compiler' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this no longer exists in sprockets-rails.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any suggestions what to do about this? It's causing errors for me.
👍 where is rails4 support? :( |
@milgner, or others: Any instructions for how to get this working with Rails 4? I've tried to integrate the various branches here without success (I blame myself). One of the first issues I'm seeing is a warning when starting the Rails server: 'Unsupported rails environment for compass' (Compass is a dependency of the Foundation framework, which I'm attempting to use with Rails 4). |
@narmy I found this ref did it for me: https://github.com/milgner/compass-rails/commit/1749c06f15dc4b058427e7969810457213647fb8 |
Have you tried remove it from the assets group? I'm not 100% sure if it Now, according to the (still in progress) upgrade Rails 4.0 removed the assets group from Gemfile. You'd need to remove that On Mon, Jul 1, 2013 at 11:53 PM, Alexandre Gaudencio <
José Duarte |
@jduarte Excellent, thanks for the suggestion 👍 |
For those of you who have this working I have 2 questions.
|
Those are the two things I was having problems with. Specifically, one sprite worked ok, but subsequent sprites didn't work. And then the production compile didn't seem like it was going to "just work". So I rolled back to Rails 3.2 for now. |
Ihave spent about 2 hours now digging into this. I got everything working but sprites and precompile. since the new precompile method uses a rake task i have no way at getting at the manifest instance to append the compiled sprites other then this hack: manifest = nil
ObjectSpace.each_object(::Sprockets::Manifest) { |o| manifest ||= o}
manifest.assets[logical_path.to_s] = asset.digest_path Im not ok with shipping this with compass rails since well its gross and could have unexpected results if things were running threads so... unless someone knows how to get a handle to the current objects in a rake task or sprockets exposes an interface for adding objects to the manifest on the fly I'm not sure i can get this working properly |
link to what i was working on https://github.com/Compass/compass-rails/blob/rails4-hack/lib/compass-rails/railties/4_0.rb#L73-L77 |
Also reporting that
Work for me as well! |
@frgooall I can guarantee these 2 things are not working #59 (comment). you are going to have a bad time when you try and deploy |
Okay, thanks for the heads up! does the Rails4 branch work? |
@scottdavis still, have a problem with your branch: |
@reizenikker can you be more descriptive |
@scottdavis i will try. |
@reizenikker could you set up a small test project that duplicates this error? |
of course Ivan Nastyukhin On Thursday, July 4, 2013 at 12:03 AM, Scott Davis wrote:
|
@scottdavis |
@scottdavis I'd be happy to try and pick up where you left off on this rails4-hack branch if you want to point me in the right direction... |
Just needs test cases added and the rails4 branch merged in Sent from my iPhone On Jul 12, 2013, at 11:22 AM, Scott M notifications@github.com wrote:
|
This worked for me as well: gem 'compass-rails', github: 'milgner/compass-rails', branch: 'rails4' |
Thanks for your work on this Scott. Rails dev student. Looking forward to when I know enough to be able to contribute to projects like this. |
Please file any bugs under a new issue |
Thanks @scottdavis! |
+1 |
Thanks 👍 |
Thanks for your work. |
Thanks! |
Thanks +1 |
I think it's time to do it :)