setsid() is recommended from the daemonize code I've read:
http://www.itp.uzh.ch/~dpotter/howto/daemonize
os.chdir('/') is sort of recommended, but doesn't seem to be required.
I moved the '/dev/null' definition closer.
I think that addresses everything. If you could give it a once-over and then
send it via ec2land, I would appreciate it.
--
https://code.launchpad.net/~jameinel/launchpad/lp-service/+merge/40386
Your team Launchpad code reviewers is requested to review the proposed merge of
lp:~jameinel/launchpad/lp-service into lp:launchpad.
_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to : [email protected]
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help : https://help.launchpad.net/ListHelp
