Skip to content

Accept cons pair for tcp connections. #34

New issue

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

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

Already on GitHub? # to your account

Merged
merged 1 commit into from
Oct 16, 2016

Conversation

mahinshaw
Copy link
Contributor

Coming takes a cons pair for connecting to tcp connections. This allows inf-clojure (or setting inf-clojure-program) to accept a cons pair for connecting to a clojure socket repl server. I tested this with planck via planck -n 5555.

@@ -322,7 +322,9 @@ of `inf-clojure-program'). Runs the hooks from
(if (not (comint-check-proc "*inf-clojure*"))
;; run the new process in the project's root when in a project folder
(let ((default-directory (inf-clojure-project-root))
(cmdlist (split-string cmd)))
(cmdlist (if (consp cmd)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm guessing the related defcustom should be updated as well.

@mahinshaw mahinshaw force-pushed the socket-connection branch from 184f0a6 to 1d2bf1d Compare June 20, 2016 13:06
@mahinshaw
Copy link
Contributor Author

Sorry about that. I updated the defcustom.

@bbatsov
Copy link
Member

bbatsov commented Oct 15, 2016

Btw, what exactly is the reasoning behind this change? I'm guessing that split-string won't work properly in such scenarios?

I'm not 100% certain what are you trying to achieve here.

@mahinshaw
Copy link
Contributor Author

I was trying to connect to a clojure socket repl. And since a cons pair for tcp connections is part of the comint API, it made sense to leverage that option.

@bbatsov
Copy link
Member

bbatsov commented Oct 15, 2016

Got it. Well, I guess you should make this a bit more explicit in the defcustom and maybe mention it in the README. I didn't know about this comint functionality.

@mahinshaw
Copy link
Contributor Author

@bbatsov I updated the defcustom doc string, and the readme. Let me know if that description is what you are looking for.

@bbatsov bbatsov merged commit eec15c4 into clojure-emacs:master Oct 16, 2016
@bbatsov
Copy link
Member

bbatsov commented Oct 16, 2016

👍

@bbatsov
Copy link
Member

bbatsov commented Mar 2, 2017

Btw, see #44 - I think we can extend this to something that would offer a much better planck support out-of-the-box

@bbatsov bbatsov mentioned this pull request Mar 2, 2017
6 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants