Skip to content

Fix fork bug #1

Merged
merged 2 commits into from Apr 17, 2011

2 participants

@apocalypse

I noticed my twirc script taking up 100% cpu and was wondering why. After some investigation, I knew what the problem was :)

When you daemonize, you have to inform the POE kernel that it has forked. This patch fixes this along with a small typo I spotted while debugging.

Thanks again for such a great module!

@semifor semifor merged commit bbb365c into semifor:master Apr 17, 2011
@semifor
Owner
semifor commented Apr 17, 2011

Thanks for the patches! I was completely unaware of the has_forked method. I've never run into the 100% CPU load issue you mentioned. Very happy you found and fixed it, though!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.