Re: [PATCH] Power control

Allmost 2 months and zero feedback. Thus its perfect? =)

//Anton

On 30 December, 2010 - Anton Lundin wrote:

> Second generation of this patch, done.
> 
> http://www.acc.umu.se/~glance/conserver/02-conserver-8.1.18-power-control.patch
> 
> Fixed the "multiplexing" so that the powercmd can't hang the whole
> Kiddie()-process. Its pretty much the same as its done for initcmd's.
> 
> There is probably still some fiddeling to be done before its perfect<tm>
> but its god enuff to take a look at.
> 
> Things on the todo:
> * processgroup for the powercmd like its done for initcmd's?
> * stdin for powercmd? currently its just close()'d
> * put the output from the powercmd in the log?
> * kill hanging powercmd's after $foo seconds?
> * maybee rename everything to ServerSite-kommands?
> 
> //Anton - Who is heading home to end this year with a big bang...
> 
> On 04 November, 2010 - Bryan Stansell wrote:
> 
> > There is certainly going to be a need to either redirect stdin/stdout of the command to the console or not (some "interactive" flag).  If not, stdin would probably be /dev/null and stdout could be anyone connected to the console (so everyone sees the output).  Maybe it should just be the console or nothing at all.  Something to think about... 
> > 
> > Bryan
> > 
> > On Nov 4, 2010, at 2:01 AM, Fabien Wernli wrote:
> > 
> > > On Wed, Nov 03, 2010 at 02:43:46PM -0700, Bryan Stansell wrote:
> > >> And I'm all for making it more generic as well...perhaps by making the 'k' command programmable - in the sense that you can associate a command with any character ([a-z0-9]) after 'k' and give it a label so you know what the command is supposed to do.
> > >> 
> > >> So, 'k' for 'k'ommand...or perhaps even '!' to invoke a command?  Harder to type, but consistent with vi, for example.
> > > 
> > > Great. I vote for '!'.
> > > Also, maybe it would be interesting to have a distinction between commands
> > > that output something or not on std(err|out).
> > > 
> > > _______________________________________________
> > > users mailing list
> > > users@conserver.com
> > > https://www.conserver.com/mailman/listinfo/users
> > 
> > 
> > _______________________________________________
> > users mailing list
> > users@conserver.com
> > https://www.conserver.com/mailman/listinfo/users
> 
> -- 
> Anton Lundin	+46702-161604



> _______________________________________________
> users mailing list
> users@conserver.com
> https://www.conserver.com/mailman/listinfo/users


-- 
Anton Lundin	+46702-161604