Mongrel users - Nov 2007 - find_pid broken in mongrel_cluster 1.0.4

The find_pid method no longer returns a nil by default which causes
some breakage in cases where the pid can''t be found since the local
variable ''pid'' becomes the entire output of the
''ps'' command.  Can you
add this return back in so that it behaves the way it did earlier?

thanks,
eric

I fixed this on my local builds by adding a `return nil` at the bottom
of the function.

Is there a place to put patches like this?  The rubyforge page for
mongrel cluster seems dead.

On 11/6/07, Eric Kolve <ekolve at adready.com>
wrote:
> The find_pid method no longer returns a nil by default which causes
> some breakage in cases where the pid can''t be found since the
local
> variable ''pid'' becomes the entire output of the
''ps'' command.  Can you
> add this return back in so that it behaves the way it did earlier?
>
> thanks,
> eric
> _______________________________________________
> Mongrel-users mailing list
> Mongrel-users at rubyforge.org
> http://rubyforge.org/mailman/listinfo/mongrel-users
>

On Tue, 6 Nov 2007, Eden Li wrote:
> I fixed this on my local builds by adding a `return nil` at the bottom
> of the function.
>
> Is there a place to put patches like this?  The rubyforge page for
> mongrel cluster seems dead.
Put this in mongrel tracker/patches page, please.

Here is a direct link:
http://rubyforge.org/tracker/?atid=5147&group_id=1306&func=browse

Thank you guys for the report and the patch :D

filipe {
  @             icewall.org
  GPG        1024D/A6BA423E
  Jabber  lautert at jabber.ru
}
>
> On 11/6/07, Eric Kolve <ekolve at adready.com> wrote:
>> The find_pid method no longer returns a nil by default which causes
>> some breakage in cases where the pid can''t be found since the
local
>> variable ''pid'' becomes the entire output of the
''ps'' command.  Can you
>> add this return back in so that it behaves the way it did earlier?
>>
>> thanks,
>> eric
>> _______________________________________________
>> Mongrel-users mailing list
>> Mongrel-users at rubyforge.org
>> http://rubyforge.org/mailman/listinfo/mongrel-users
>>
> _______________________________________________
> Mongrel-users mailing list
> Mongrel-users at rubyforge.org
> http://rubyforge.org/mailman/listinfo/mongrel-users
>