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

Starting the LSP server locally inside the JVM (second take) #7

Merged
merged 9 commits into from
Apr 8, 2018

Conversation

ddekany
Copy link
Contributor

@ddekany ddekany commented Apr 7, 2018

So this time it uses server/freemarker-languageserver.jar. See the commit message.

Note that this PR is after the other pending PR. If you merge that PR first, then this PR is only a single commit.

Also, please see my continued suffering with the development environment setup in the README.md... would be great if you describe how you do that. Certainly it's not easy for any other newcomer either.

ddekany added 6 commits March 27, 2018 22:15
…ally amateurish, but that's how I managed with it so far.
…e process. It still uses server/freemarker-languageserver-all.jar, proving that later this will be able to link to a runtime chosen freemarker.jar as well (that's why it creates its own ClassLoader and uses reflection to invoke the launcher). Note that the committed freemarker-languageserver-all.jar has the new launcher class, but the other classes are from 1492c645aa79ec9b9c610ab619657edccae0fe4a, as their current version crashes with NPE.
@ddekany ddekany changed the title Starting the LSP locally inside the JVM (second take) Starting the LSP server locally inside the JVM (second take) Apr 7, 2018
* http://www.eclipse.org/legal/epl-v10.html
*
* Contributors:
* Angelo Zerr <[email protected]> - initial API and implementation
Copy link
Owner

Choose a reason for hiding this comment

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

please add your name

* http://www.eclipse.org/legal/epl-v10.html
*
* Contributors:
* Angelo Zerr <[email protected]> - initial API and implementation
Copy link
Owner

Choose a reason for hiding this comment

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

please add your name

@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
<?pde version="3.8"?><target name="bluesky" sequenceNumber="7">
<?pde version="3.8"?><target name="lsp4-freemarker" sequenceNumber="7">
Copy link
Owner

Choose a reason for hiding this comment

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

rename to lsp4e-freemarker

@angelozerr
Copy link
Owner

Note that this PR is after the other pending PR. If you merge that PR first, then this PR is only a single commit.

Just few comments to change and I will merge it. Please too regenerate the freemarker-server-all-.jar whith last version of source code (NPE should be fixed)

Also, please see my continued suffering with the development environment setup in the README.md... would be great if you describe how you do that. Certainly it's not easy for any other newcomer either.

Ok I will see that

@angelozerr
Copy link
Owner

@ddekany please update server/freemarker-languageserver.jar too. It have disabled the uses of tolerant parser which must support position update (otherwise you will have NPE when outline will be refreshed).

@angelozerr angelozerr merged commit 79b27a2 into angelozerr:master Apr 8, 2018
@angelozerr
Copy link
Owner

Thanks @ddekany !

@angelozerr
Copy link
Owner

angelozerr commented Apr 8, 2018

@ddekany I think your class LocalStreamConnectionProvider should be hosted in lsp4e project. Is there any chance that you create a gerrit patch for lsp4e eclipse project? If you have not time, are you OK I create this gerrit patch?

@ddekany
Copy link
Contributor Author

ddekany commented Apr 8, 2018

@angelozerr Honestly, I don't feel confident to offer a POC like this to lsp4e itself... I did nearly 0 research. Anyway, since you already know your way around there, feel free to do the gerrit patch. At least we get some feedback on what did I miss...

@angelozerr
Copy link
Owner

angelozerr commented Apr 14, 2018

Ok thanks for your answer. I have created a bugzilla for lsp4e at https://bugs.eclipse.org/bugs/show_bug.cgi?id=533584 and if lsp4e is interested, I will create a gerrit patch with your work (if you are OK)..

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