Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

exit parent without triggering atexit() processing #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Maxime2
Copy link
Contributor

@Maxime2 Maxime2 commented Jul 30, 2015

It is better to use _exit() on parent termination as it doesn't trigger atexit() processing which would be processed when daemon terminates normal way.

@rustyrussell
Copy link
Owner

Maxim Zakharov [email protected] writes:

It is better to use _exit() on parent termination as it doesn't trigger atexit() processing which would be processed when daemon terminates normal way.
You can view, comment on, or merge this pull request online at:

#26

Thanks!

You could have done these as all one pull request BTW :)

Cheers,
Rusty.
PS. I prepended "daemonize:" to the subjects for the two I applied already.

@dgibson
Copy link
Collaborator

dgibson commented Jul 31, 2015

On Wed, Jul 29, 2015 at 08:15:19PM -0700, Rusty Russell wrote:

Maxim Zakharov [email protected] writes:

It is better to use _exit() on parent termination as it doesn't trigger atexit() processing which would be processed when daemon terminates normal way.
You can view, comment on, or merge this pull request online at:

#26

Thanks!

You could have done these as all one pull request BTW :)

Cheers,
Rusty.
PS. I prepended "daemonize:" to the subjects for the two I applied already.

Rusty, I'm not seeing any of these. Which tree did you apply them to?

David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT the other
| way around!
http://www.ozlabs.org/~dgibson

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants