-
Notifications
You must be signed in to change notification settings - Fork 35
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
Consider defining a subtype of Base.REPL.AbstractREPL and running that inside of comint instead of (ab)using ansi-term #26
Comments
I think this is an interesting approach. I agree that The roadmap for exploring this would be roughly the following:
1-4 don't involve any Elisp, can be done mostly in Julia. I will explore this in the medium run, but help is always appreciated. |
(agree about Github vs Markdown, but formatted the Markdown, hope it is OK). |
I looked into this a bit, with the following conclusion:
Generic implementation notes: So in contrast to what I thought above, there is very little need for Julia coding ATM, just getting one's hands dirty with |
Hm. You're right; that's not at all as bad as I thought. Couple of stray
Please let me know me know if you start working on this somewhere else. I'll |
For what it's worth, regarding support for If you'd like, I'd be very happy to spend some time in the next couple of weeks putting together a pull request that updates julia-repl to support a comint repl in addition to the term one. :) |
A PR would be very helpful, or at least a link to your solution. |
Okay, great! I'll get started and will submit a work-in-progress merge request when I have it working with julia-repl. |
I have no idea about the difficultly level here and am definitely not volunteering to do it myself necessarily, but the less hacky way of solving the problems with ESS would be to do this instead of abandoning comint entirely.
Since julia itself already includes two REPLs, it shouldn't be impossible to include only the features that are compatible with comint from LineEditREPL without losing access to the niceties needed for things like using Gallium. Once the
<:AbstractREPL
was written, all that would need to be done would be to swap out the existing REPL for our shiny new one using this function here during startup, and we'd be golden. All the niceties ofjulia-repl
without any of the funkiness.Posting this here as an issue as a sort of long-term wishlist or to get any criticism that exists on the idea. This might even make more sense within ESS than as an alteration of
julia-repl
.The text was updated successfully, but these errors were encountered: