From: Scott B. <sba...@le...> - 2010-11-15 19:24:57
|
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. |