From: Jam <ja...@ti...> - 2010-12-12 22:57:02
|
On Monday 13 December 2010 06:15:33 ltsp-developer- re...@li... wrote: > I'm attaching a patch that removes the LTSP written "fork twice to > daemonize" code handed down to us by His Holiness St. Stevens, and > substitutes in the standard python daemon object. It also adds a "-d" > or "--debug" switch, that stops the process from backgrounding, so you > can see stderr and stdout messages, useful for, well, debugging. > > The only minor problem is that it now puts another dependency on the > "python-daemon" package (in debian based distros, not sure what it is > for our RPM distro maintainers). > > What do you all think? I think it clarifies and shortens the code > tremendously. And the less code we have to maintain... > > Let me know your thoughts. If the general consensus is to go for it, > I'll commit it upstream. Scott the fork-twice is the right-way (tm) to do it. Change for other reasons, but not to get rid of this behaviour. (Technical stuff about controlling terminal etc, and BTW what kde is NOT doing so eg launch k3b, kmail etc in a terminal and the app daemonises but leaves heaps of ongoing mouse droppings in the terminal) James |