Hi Donnie:

Thanks for raising this issue and the patch. Locally, I've patched it 
using ruby's File.basename and it seems to work for your use case and 
supports other file extensions.

I'm working on another prerelease now.

Regards,
Bradley Taylor

[EMAIL PROTECTED] wrote:
> Hello,
> 
> Thank you for your development of Mongrel, great job.  :)
> 
> I am using mongrel_cluster, but I have a problem where my rails
> application name includes a "." such as mydomain.com, which causes an
> error when generating the PID file for mongrel.
> 
> I have attached below a diff of the current working copy in the trunk
> and the changes I made to overcome this problem.  I kept your convention
> of assuming the file ends with ".pid" in my changes.
> 
> Hope this helps.
> __
> Donnie Jones
> 
> 
> 
> svn diff
> Index: init.rb
> ===================================================================
> --- init.rb     (revision 519)
> +++ init.rb     (working copy)
> @@ -24,7 +24,7 @@
>          }
>          conf = YAML.load_file(@config_file)
>          @options.merge! conf if conf
> -        @pid_file = @options["pid_file"].split(".")
> +        @pid_file = @options["pid_file"].split(".pid")
> 
>          start_port = end_port = @only
>          start_port ||=  @options["port"].to_i
> @@ -34,7 +34,7 @@
>        end
> 
>        def port_pid_file(port)
> -        "[EMAIL PROTECTED]@pid_file[1]}"
> +                               "[EMAIL PROTECTED]"
>        end
> 
>        def start
> 
> 
> _______________________________________________
> Mongrel-users mailing list
> [email protected]
> http://rubyforge.org/mailman/listinfo/mongrel-users

_______________________________________________
Mongrel-users mailing list
[email protected]
http://rubyforge.org/mailman/listinfo/mongrel-users

Reply via email to