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

Properly shut down processes #41

Closed
martinklepsch opened this issue Feb 22, 2017 · 8 comments
Closed

Properly shut down processes #41

martinklepsch opened this issue Feb 22, 2017 · 8 comments

Comments

@martinklepsch
Copy link

After debugging #40 I noticed that I still have lots of processes in the run state when looking at the *Process List* buffer. It does not seem to matter what/if any dependencies or code is loaded.

Probably my fault but is there a way to properly kill processes? Checked in M-x but couldn't find anything... (I'm using CMD + K now.)

@bbatsov
Copy link
Member

bbatsov commented Mar 1, 2017

Can I see a screenshot of your *Process list* buffer? I'm not sure what exactly is leaking for you.

@martinklepsch
Copy link
Author

This is what it looks like after I started an inf-clojure process and then killed it. (I ran the list-processes command after I killed the process to ensure the listing is up-to-date.)

screen shot 2017-03-01 at 17 18 23

@bbatsov
Copy link
Member

bbatsov commented Mar 2, 2017

Seems that the problem is that 2 processes get created when you invoke something like lein repl or boot repl and just one of them gets associated with the inf-clojure buffer for obvious reasons (the buffer is just a presentation of this process). Not sure what we can do about the second process, though...

@bbatsov
Copy link
Member

bbatsov commented Mar 5, 2017

Forget this - it's just one process. I'm really puzzled why it doesn't die as Emacs asks to kill it when you kill the inf-clojure buffer.

@bbatsov bbatsov closed this as completed in 862699e Mar 5, 2017
@martinklepsch
Copy link
Author

martinklepsch commented Mar 5, 2017

👍 — happy I didn't report garbage haha. If I now kill a buffer with CMD+k, should this also correctly terminate the process or is it intended that's people use the new quit command from now on?

@bbatsov
Copy link
Member

bbatsov commented Mar 5, 2017

If I now kill a buffer with CMD+k, should this also correctly terminate the process or is it intended that's people use the new quit command from now on?

You should use the new command. I've got no idea why Control+k is not working.

@martinklepsch
Copy link
Author

Cool, thanks! And thanks for taking the time to look at this, just made another dash on the "Beers to buy for Bozhidar" dash list. 😄

@bbatsov
Copy link
Member

bbatsov commented Mar 5, 2017

You're welcome! At ClojureD a few people approached me asking for some inf-clojure improvements, because it was becoming a popular way to work with self-hosted ClojureScript REPLs like planck and lumo and I decided to give it a bit of love.

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

No branches or pull requests

2 participants