unicorn_rails cleanup (possible fix for Rails3) pushed
normalperson at yhbt.net
Tue Jun 8 16:55:59 EDT 2010
Hongli Lai <hongli at phusion.nl> wrote:
> On Tue, Jun 8, 2010 at 9:20 PM, Eric Wong <normalperson at yhbt.net> wrote:
> > Thanks Hongli, so this only affects people who remove the
> > config.ru that Rails 3 creates for them? Yikes...
> No. The problem even occurs if you already have config.ru. But the
> thing is, Rails 3 has deprecated ActionController::Dispatcher a few
> weeks ago and replaced it with a stub. Rails::Rack::Static changed its
> interface and must be constructed differently. The only way to obtain
> a valid Rack endpoint for the app seems to be parsing
Actually, I made "unicorn_rails" completely bypass the "rails_builder"
method if there's a config.ru, so it should never hit the
> > Let me know if the edited patch below looks alright to you.
> Yes it looks fine. A bit overcomplicated regexp compared to using
> 'strip' but whatever works. :)
I just kept the regexp as-is, works for me.
I just managed to push this to git://git.bogomips.org/unicorn.git before
my Internet connection died on me earlier today. I've beefed up the
tests a bit but will probably do more later.
Eric Wong (4):
t0300: Rails 3 test actually uses unicorn_rails
tests: libify common rails3 setup code
unicorn_rails: fix requires for Ruby 1.9.2
tests: add Rails 3 test for the missing config.ru case
Hongli Lai (Phusion) (1):
Fix unicorn_rails compatibility with the latest Rails 3 code
More information about the mongrel-unicorn