On May 20, 2006, at 10:52 PM, Zed Shaw wrote:

> On Sat, 2006-05-20 at 21:53 -0300, Luis Lavena wrote:
>> On 5/20/06, Zed Shaw <[EMAIL PROTECTED]> wrote:
>>> * Patch from Bradley Taylor that gives mongrel_rails start the  
>>> options
>>> -u and -g.  These will change the uid/gid on UNIX (not WIN32!) to  
>>> the
>>> requested ones.
>>
>> Man, hate we clash parameters, soon we will be using -$ -% and -* to
>> set new options...
>>
>> Why better we don't start leaving these new options to the config  
>> file
>> and keep params to the minimum?

Since both mongrel_rails and mongrel_cluster support config file  
generation, we still need a way to specify what options to write to  
the config file.

>
> Damn, sorry about that man.  We can change those easily though.  I
> actually think I might change it to -M user:group since it doesn't  
> make
> a lot of sense to just change user.

I submitted a new patch that removes the short name options [-u,-g]  
and leaves the long names, --user and --group. I also added a check  
to Start::validate that requires that both user and group are  
specified together.  This should address both concerns.  The patch  
also switches the order of the calls to change_privilege by changing  
to the group then user.

Thanks,
Bradley

------
Rails Machine
Simplified Ruby on Rails application deployment
http://railsmachine.com




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

Reply via email to