FooBarWidget / passenger
watch download tarball
public
Forks61Right
Watchers800Right
Description: Phusion Passenger (mod_rails)
Homepage: http://www.modrails.com/
Clone URL: git://github.com/FooBarWidget/passenger.git
Click here to lend your support to: passenger and make a donation at www.pledgie.com !

Comments for FooBarWidget's passenger   feed

FooBarWidget commented on FooBarWidget/passenger Tue Jun 02 14:18:31 -0700 2009
Comment in 3fbcdff:

I don't think it's worth it to tell them. I don't know in what way APR is broken and I can't make a good test case to reproduce the problem, I just know that the problems went away after I rewrote the stuff with my own code.

rdp commented on FooBarWidget/passenger Tue Jun 02 09:01:27 -0700 2009
Comment in 3fbcdff:

did you tell the apr guys their stuff stinks?

atnan commented on FooBarWidget/passenger Thu May 14 06:40:47 -0700 2009
Comment in 5c7e51a:

Hmm...I didn't know that a2enmod/a2ensite/a2dismod/a2dissite was a Debian concoction. You learn something new every day. Sure as hell beats appending to the bottom of configuration files, though.

simonmenke commented on FooBarWidget/passenger Thu May 14 06:16:49 -0700 2009
Comment in f085919:

I guess I can delete my fork then ;)

FooBarWidget commented on FooBarWidget/passenger Thu May 14 01:00:31 -0700 2009
Comment in 5c7e51a:

What dsturnbull said.

The documentation doesn't suggest editing the "primary" Apache configuration file, just "the" Apache configuration. Whatever the correct configuration file should be is up to the reader.

FooBarWidget commented on FooBarWidget/passenger Thu May 14 00:59:05 -0700 2009
Comment in f085919:

Yes. Though you may have to call 'Gem.clear_paths' in your app before it actually has any effect.

dsturnbull commented on FooBarWidget/passenger Thu May 14 00:02:27 -0700 2009
Comment in 5c7e51a:

Apache packages are rarely configured as in debian and ubuntu

atnan commented on FooBarWidget/passenger Wed May 13 22:54:51 -0700 2009
Comment in 5c7e51a:

Why do you suggest editing the primary Apache configuration file, rather than putting Passenger module configuration files in the mods_available so that a2enmod and a2dismod can be used for enabling and disabling the module?

simonmenke commented on FooBarWidget/passenger Wed May 13 12:50:44 -0700 2009
Comment in f085919:

Does this allow me to specify a different GEM_PATH / GEM_HOME for each virtual host?

labria commented on FooBarWidget/passenger Sat Apr 18 03:12:25 -0700 2009
Comment in 184cb20:

This seems to break “rake package” for me, GCC 4.0.1

labria commented on FooBarWidget/passenger Fri Apr 17 04:32:30 -0700 2009
Comment in ae64fa7:

Thanks!

libc commented on FooBarWidget/passenger Thu Apr 16 18:47:31 -0700 2009
Comment on bin/passenger-install-nginx-module L125 in dfd973b:

I believe, you’ve mistaken @pcre_is_installed with @pcre_source_dir here

FooBarWidget commented on FooBarWidget/passenger Thu Apr 02 08:08:16 -0700 2009
Comment in 1ae5960:

Oops, you’re right. That’s a typo.

Roman2K commented on FooBarWidget/passenger Wed Mar 11 18:09:22 -0700 2009
Comment on lib/phusion_passenger/abstract_request_handler.rb L375 in 1ae5960:

Congratulations on the copious documentation for this workaround.

However, why do you define the sync= singleton method within an instance_eval block? I might be wrong, but I think it’s unnecessary since client is a local variable.

uipoet commented on FooBarWidget/passenger Wed Feb 18 16:33:25 -0800 2009
Comment in 29f1592:

Fantastic! So much better than a hacked fastthread placebo. I believe this makes passenger truly 1.9.1 compatible. Looking forward to the official gem release.

uipoet commented on FooBarWidget/passenger Wed Feb 18 16:31:37 -0800 2009
Comment in 796b98b:

Thank you for the very fast turn-around on this! Can’t wait to fire up passenger on my 1.9.1 tonight. :)

joshpencheon commented on FooBarWidget/passenger Wed Feb 18 15:54:56 -0800 2009
Comment in 87db9ab:

Brilliant, thanks guys. :-)

FooBarWidget commented on FooBarWidget/passenger Sat Feb 14 08:09:12 -0800 2009
Comment in 0e43297:

Thanks for the comments nobu.

I use write instead of print because print will append a newline to the output.

I am aware that it is claimed that fastthread’s functionality is bundled starting from a certain patchlevel of 1.8.6. But I once ran into a threading bug in 1.8.6, which seemed to be solved by loading fastthread.

As for the indentation, Github doesn’t properly display tabs.

nobu commented on FooBarWidget/passenger Wed Feb 04 16:58:59 -0800 2009
Comment in 0e43297: 
  • RSTRING_PTR()
    is not a direct replacement of 
    rb_str2cstr()
    . Use 
    StringValueCStr()
    instead.
  • Why you use 
    write
    instead of 
    print
    ? The latter can deal with multiple arguments. You don’t need 
    body.each
    , as you can write 
    output.print(*body)
    .
  • ‘fastthread’ is bundled with 1.8.6, and its gem has been left with some bugs. 
    $“.grep(%r”(?:\A|/)thread\.(?!rb\z)\w+\z")0
    returns true value if the extension library verion of ‘thread’.
  • Some indentation seem broken :)
atnan commented on FooBarWidget/passenger Thu Jan 22 17:01:44 -0800 2009
Comment in 0c66e05:

@FooBarWidget: Ah, sounds good. Thanks for the clarification.