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

osx: quiting app does not always kill ipfs #12

Closed
jbenet opened this issue Jun 17, 2015 · 4 comments
Closed

osx: quiting app does not always kill ipfs #12

jbenet opened this issue Jun 17, 2015 · 4 comments
Labels
kind/bug A bug in existing code (including security flaws)
Milestone

Comments

@jbenet
Copy link
Member

jbenet commented Jun 17, 2015

go-ipfs is notoriously annoying to kill, because it tries to shut down gracefully first.

  • i think electron-app or node-ipfsd-ctrl should make sure ipfs is killed by waiting + checking.
  • separately, we should make go-ipfs always respect sigterm. Fix Daemon Exit  kubo#1384
@jbenet
Copy link
Member Author

jbenet commented Jun 18, 2015

another note, if the electron-app crashes (or is force quit) it doesnt also bring down the ipfs daemon. it looks like ipfsd-ctl is liberating its process child or something (i.e. normally osx will kill children when parent proc dies)

@krl
Copy link
Contributor

krl commented Jun 21, 2015

this is now addressed in node-ipfsd-ctl, except maybe the crash issue. i'll investigate

@daviddias daviddias added this to the CCC milestone Aug 11, 2015
@daviddias
Copy link
Member

Referenced here: #25 (comment)

@daviddias daviddias added the kind/bug A bug in existing code (including security flaws) label Aug 11, 2015
@daviddias
Copy link
Member

Fixed with 70efae8 thanks to @jbenet subcomandante woo!

hacdias added a commit that referenced this issue Aug 29, 2019
# This is the 1st commit message:

try gh actions

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #2:

add build

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #3:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #4:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #5:

add gh token

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #6:

fix names

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #7:

add chrome-log driver

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #8:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #9:

xvfb?

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #10:

install xvfb

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #11:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #12:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #13:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #14:

add screensize

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #15:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #16:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #17:

xvfb-maybe

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #18:

chore!

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>
hacdias added a commit that referenced this issue Sep 2, 2019
# This is the 1st commit message:

try gh actions

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #2:

add build

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #3:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #4:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #5:

add gh token

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #6:

fix names

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #7:

add chrome-log driver

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #8:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #9:

xvfb?

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #10:

install xvfb

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #11:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #12:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #13:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #14:

add screensize

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #15:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #16:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #17:

xvfb-maybe

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #18:

chore!

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>
hacdias added a commit that referenced this issue Sep 2, 2019
# This is the 1st commit message:

try gh actions

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #2:

add build

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #3:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #4:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #5:

add gh token

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #6:

fix names

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #7:

add chrome-log driver

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #8:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #9:

xvfb?

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #10:

install xvfb

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #11:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #12:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #13:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #14:

add screensize

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #15:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #16:

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #17:

xvfb-maybe

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>

# This is the commit message #18:

chore!

License: MIT
Signed-off-by: Henrique Dias <hacdias@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug A bug in existing code (including security flaws)
Projects
None yet
Development

No branches or pull requests

3 participants