I would like to unify the ferret_start and ferret_stop scripts into a single script. My motivation is to make the scripts more like the other rails scripts, and to also make it fit into monitoring schemes better. I realize that some work was done recently to allow the scripts to be run from script/runner, but I think a unified script is a better approach. A unified script would allow more advanced command line processing without duplication. For example, being able to supply the RAILS_ENV on the command line, and overriding configuration options such as the PID file. Would there be any objection to me submitting patches that unified these two scripts? Here is what I would expect them to look like in the future (after my patches): script/ferretd -e production start script/ferretd -e production stop -- Posted via http://www.ruby-forum.com/.
Hi! sounds great, just go ahead :-) cheers, Jens On Mon, Oct 08, 2007 at 06:51:44PM +0200, Peter Jones wrote:> I would like to unify the ferret_start and ferret_stop scripts into a > single script. My motivation is to make the scripts more like the other > rails scripts, and to also make it fit into monitoring schemes better. > I realize that some work was done recently to allow the scripts to be > run from script/runner, but I think a unified script is a better > approach. > > A unified script would allow more advanced command line processing > without duplication. For example, being able to supply the RAILS_ENV on > the command line, and overriding configuration options such as the PID > file. > > Would there be any objection to me submitting patches that unified these > two scripts? Here is what I would expect them to look like in the > future (after my patches): > > script/ferretd -e production start > script/ferretd -e production stop > -- > Posted via http://www.ruby-forum.com/. > _______________________________________________ > Ferret-talk mailing list > Ferret-talk at rubyforge.org > http://rubyforge.org/mailman/listinfo/ferret-talk >-- Jens Kr?mer webit! Gesellschaft f?r neue Medien mbH Schnorrstra?e 76 | 01069 Dresden Telefon +49 351 46766-0 | Telefax +49 351 46766-66 kraemer at webit.de | www.webit.de Amtsgericht Dresden | HRB 15422 GF Sven Haubold, Hagen Malessa
I''ve tweaked the ferret server scripts a bit. Made them a little more modular. Peter, feel free to email me if you want to discuss things. Also, I''m learning about threading and forking. My tentative thought is that the use ''safefork'' (a while loop encapsulating a Kernel#fork) might be able to be improved upon. The examples in The Ruby Way by Fulton seem cleaner -- they don''t use Kernel#fork. Again, all of this is tentative; just thinking out loud. -------------- next part -------------- An HTML attachment was scrubbed... URL: http://rubyforge.org/pipermail/ferret-talk/attachments/20071022/9d461ad9/attachment.html
Can we have a look at the source? (sorry if I''m missing something and you already put it somewhere) On 10/23/07, David James <davidj503 at gmail.com> wrote:> I''ve tweaked the ferret server scripts a bit. Made them a little more > modular. Peter, feel free to email me if you want to discuss things. > > Also, I''m learning about threading and forking. My tentative thought is > that the use ''safefork'' (a while loop encapsulating a Kernel#fork) might be > able to be improved upon. The examples in The Ruby Way by Fulton seem > cleaner -- they don''t use Kernel#fork. Again, all of this is tentative; > just thinking out loud. > > _______________________________________________ > Ferret-talk mailing list > Ferret-talk at rubyforge.org > http://rubyforge.org/mailman/listinfo/ferret-talk >
Sorry for the top posting. I posted this to the ruby-forum site on the 25th of October but it doesn''t seem to have made it''s way to this mailing list. Here is the original post: --- I''ve attached my first set of changes. The attached archive includes a README file with information about what I''ve changed and why. These changes are only for Unix-like operating systems, for now. If you like the changes I''ve made, I''ll integrate the Windows code from the various scripts in the script directory. Let me know if you have any questions. --- The patches were originally attached to the forum posting. The URL to the patches is therefor: http://www.ruby-forum.com/attachment/780/patches.tar.gz Thanks. -- Peter Jones pmade inc. - http://pmade.com
Thanks Peter, I''ll have a look at these this evening. Jens On Mon, Nov 05, 2007 at 10:51:38AM -0700, Peter Jones wrote:> Sorry for the top posting. I posted this to the ruby-forum site on > the 25th of October but it doesn''t seem to have made it''s way to this > mailing list. Here is the original post: > --- > I''ve attached my first set of changes. The attached archive includes a > README file with information about what I''ve changed and why. > These changes are only for Unix-like operating systems, for now. If > you like the changes I''ve made, I''ll integrate the Windows code from > the various scripts in the script directory. > Let me know if you have any questions. > --- > The patches were originally attached to the forum posting. The URL to > the patches is therefor: http://www.ruby-forum.com/attachment/780/patches.tar.gz > Thanks. > -- > Peter Jones > pmade inc. - http://pmade.com > > _______________________________________________ > Ferret-talk mailing list > Ferret-talk at rubyforge.org > http://rubyforge.org/mailman/listinfo/ferret-talk >-- Jens Kr?mer webit! Gesellschaft f?r neue Medien mbH Schnorrstra?e 76 | 01069 Dresden Telefon +49 351 46766-0 | Telefax +49 351 46766-66 kraemer at webit.de | www.webit.de Amtsgericht Dresden | HRB 15422 GF Sven Haubold, Hagen Malessa
Hi Peter, works like a charm and looks great :-) Just merged this into trunk. Cheers, Jens On Mon, Nov 05, 2007 at 10:51:38AM -0700, Peter Jones wrote:> Sorry for the top posting. I posted this to the ruby-forum site on > the 25th of October but it doesn''t seem to have made it''s way to this > mailing list. Here is the original post: > --- > I''ve attached my first set of changes. The attached archive includes a > README file with information about what I''ve changed and why. > These changes are only for Unix-like operating systems, for now. If > you like the changes I''ve made, I''ll integrate the Windows code from > the various scripts in the script directory. > Let me know if you have any questions. > --- > The patches were originally attached to the forum posting. The URL to > the patches is therefor: http://www.ruby-forum.com/attachment/780/patches.tar.gz > Thanks. > -- > Peter Jones > pmade inc. - http://pmade.com > > _______________________________________________ > Ferret-talk mailing list > Ferret-talk at rubyforge.org > http://rubyforge.org/mailman/listinfo/ferret-talk >-- Jens Kr?mer http://www.jkraemer.net/ - Blog http://www.omdb.org/ - The new free film database
On Nov 7, 2007, at 3:05 PM, Jens Kraemer wrote:> works like a charm and looks great :-) > > Just merged this into trunk.Here''s an update to the capistrano tasks to work with the new script: http://projects.jkraemer.net/acts_as_ferret/ticket/190 Cheers, John