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

Scheme support #55

Open
dreadwail opened this issue Feb 4, 2017 · 3 comments
Open

Scheme support #55

dreadwail opened this issue Feb 4, 2017 · 3 comments

Comments

@dreadwail
Copy link

I attempted to use this package for MIT scheme but a few things stand in the way (swank server is located at "slime/contrib/swank-mit-scheme.scm" not "slime/start-swank.lisp" for example).

Any thoughts about expanding it to work with any lisp (like scheme) instead of just common-lisp?

@sjlevine
Copy link
Owner

sjlevine commented Feb 6, 2017

Hi Ben,

I haven't used scheme before, but this is an interesting idea.

Have you tried poking around with the swank server start up parameters here? That may be a good place to start.

Unfortunately I don't have the cycles now to ensure support for scheme, but you're welcome to try and modify the code to get things working of course. Since swank seems to have support for scheme already, that's a good sign.

Cheers,
Steve

@dreadwail
Copy link
Author

dreadwail commented Feb 7, 2017

Yea I've experimented with this a bit. The main hitch is this hard-coded script name:

https://github.com/sjlevine/atom-slime/blob/master/lib/swank-starter.coffee#L41

MIT scheme uses a different swank starter script:

https://github.com/slime/slime/blob/master/contrib/swank-mit-scheme.scm

I think all that would be required is the ability to configure atom-slime with the right script to use (using the current value as its default for people who don't need this).

Is this something you might be interested in if I were able to get it working and open a pull-request?

@sjlevine
Copy link
Owner

sjlevine commented Feb 7, 2017

Hi Ben,

Yes, this would be something that'd be nice to have. The current starter script in this package is in need of some re-working -- there have been similar issues in the past, even for different variants of Common Lisp.

If you have time, do feel free to submit a pull request.

Cheers,
Steve

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants