max-mapper / monu

menubar process monitor mac app [ALPHA]
https://github.com/maxogden/monu/releases
BSD 2-Clause "Simplified" License
1.11k stars 74 forks source link

read PIDs even if they arent in config.json #2

Open max-mapper opened 9 years ago

max-mapper commented 9 years ago

say you add a process with name 'foo' to config.json, start it, and then go to config.json and remove 'foo'.

now you are in a state where there is a foo.pid and foo.mon.pid, and the foo processes are likely running

IMO any PIDs that aren't referenced by 'config.json' should get SIGTERM'd and then deleted

mafintosh commented 9 years ago

I agree that you should sigterm any PIDs deferenced by config.json