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

Add editing support to the REPL, using the rustyline crate. #30

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jimblandy
Copy link
Contributor

It's a little weird to have the lisp crate depend on rustyline, instead of just bin/lisp.rs, but I don't know how to arrange that.

@fitzgen
Copy link
Collaborator

fitzgen commented Jul 31, 2017

There are no binary only dependencies, and the issue requesting it was closed :(

@fitzgen
Copy link
Collaborator

fitzgen commented Jul 31, 2017

Seems like the next closest thing is an optional-but-on-by-default feature.

@jimblandy jimblandy force-pushed the rustyline branch 2 times, most recently from 9bb5493 to 0a67163 Compare August 2, 2017 01:17
This as an optional default feature, "repl-edit".
@jimblandy
Copy link
Contributor Author

@fitzgen Okay, I've made it an optional feature. Haven't done this before, so if you have style suggestions, I'm all ears.

@fitzgen
Copy link
Collaborator

fitzgen commented Aug 2, 2017

Just saw this comment, which seems like the best way to do this: rust-lang/cargo#1982 (comment)

Sorry for moving goal posts...

@jimblandy
Copy link
Contributor Author

Oh, that makes total sense!

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

Successfully merging this pull request may close these issues.

2 participants