> One option we have is to have "packet_worker_runner" as installed executable when user installs packet gem.
The more I think about the idea of a packet runner, the more I like it. One reason I like it: it solves a problem with the Windows port I wrote. The Windows implementation needs to know what process to launch to create a new worker. It currently relaunches PACKET_APP, but I was concerned that might be a misuse of PACKET_APP and it forces the author of a PACKET_APP to implement code like what I wrote in script/backgroundrb to handle a "worker parameter. If there's a separate app whose only purpose in life is to be the runner, no one else has to write code to handle those parameters. - Brian On Sun, Apr 13, 2008 at 6:53 PM, hemant kumar <[EMAIL PROTECTED]> wrote: > > On Sun, 2008-04-13 at 16:41 -0700, Paul Kmiec wrote: > > > This help significantly in improving stability of > > BackgrounDRb and code > > > written on top of packet. So what you guys think? Shall we > > go for this > > > change? > > > > It sounds good to me. Anything that helps stability and > > improves > > memory usage will always get my vote. > > > > I agree. > > > > Paul > > Okay folks, > > I have updated git repo of just packet with latest changes wherein > I have removed fork in favour of fork and exec: > > http://github.com/gnufied/packet/tree/master > > The only catch is, now we need some kinda runner to run our workers > since exec takes argument in form of string only. For example: > > exec "packet_worker_runner > ##{t_worker_name}:{worker_read_end.fileno}:#{worker_write_end.fileno}:#{option_dump_length}" > > One option we have is to have "packet_worker_runner" as installed > executable when user installs packet gem. > > Any other ideas? > > > > _______________________________________________ > Backgroundrb-devel mailing list > [email protected] > http://rubyforge.org/mailman/listinfo/backgroundrb-devel > -- Brian
_______________________________________________ Backgroundrb-devel mailing list [email protected] http://rubyforge.org/mailman/listinfo/backgroundrb-devel
