Skip to content

LSP support #252

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

LSP support #252

wants to merge 1 commit into from

Conversation

davidgranstrom
Copy link
Owner

@davidgranstrom davidgranstrom commented Sep 19, 2024 •
edited
Loading

This PR consists of adjustments to use the LanguageServer.quark LSP for sclang.

At the moment this is tested with scztt/LanguageServer.quark#29

@davidgranstrom davidgranstrom mentioned this pull request Sep 19, 2024
Open
@salkin-mada
Copy link
Contributor

This is very exciting!

@davidgranstrom
Copy link
Owner Author

Not sure why yet, but for some reason Server.program is set to nil when using the language server at the moment. An easy fix is to set it manually and then boot the server for the time being.

Server.program = "scsynth";
s.boot;

@bsssssss
Copy link

Hey so happy to see this !
I noticed something interesting about the Server.program issue. If I run sclang from command line, the issue still appears, but if I run it with sudo it works. I also noticed that my startup.scd file in Platform.systemAppSupportDir was not loaded without sudo. (I also suspect this issue was related to this)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@davidgranstrom @salkin-mada @bsssssss