Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Ganache processes do not close when quiting in Mac dock #1129

Closed
mikeseese opened this issue Feb 19, 2019 · 2 comments
Closed

Ganache processes do not close when quiting in Mac dock #1129

mikeseese opened this issue Feb 19, 2019 · 2 comments

Comments

@mikeseese
Copy link
Contributor

Moving this from @balexander4's comment from https://github.com/trufflesuite/ganache/issues/1055#issuecomment-464231304

This is occurring for me again when I close Ganache using the dock on Darwin, (right clicking and hitting quit, instead of using the x). The processes remain open running in the background causing the port error issues mentioned above.

Steps to reproduce:

  1. Open Activity Monitor to view open processes for Electron.
  2. Open a new instance of Ganache 2.0.0.2 and choice an existing workspace.
  3. Right click on the Ganache icon and select Quit to close. Do not use the left hand corner red X to close Ganache.
  4. View open processes in the Activity Monitor. You should still see Electron processes running on the Activity Monitor even after Ganache has been closed.
  5. Reopen Ganache and notice that there is an error about ports being already open on the Server tab. "The port is used by another application; please change it"

Running:
Mac High Sierra 10.13.6
Ganache 2.0.0.2

@honestbonsai
Copy link
Contributor

honestbonsai commented Feb 19, 2019

Can reproduce the issue on Mac Mojave 10.14.2, commit f0bd098

app.on("window-all-closed", ...) isn't triggered by closing from the dock

@mikeseese mikeseese changed the title Ganache processes do not close when quiting in Mac doc Ganache processes do not close when quiting in Mac dock Feb 19, 2019
@honestbonsai
Copy link
Contributor

honestbonsai commented Feb 19, 2019

Wild guess based on https://electronjs.org/docs/api/app#event-window-all-closed

Listen for will-quit event rather than window-all-closed to capture closing from dock

Edit: Yup, just tested. Closing from dock will emit the will-quit event. PR on its way.

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

No branches or pull requests

2 participants