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 instructions on starting server to README #154

Merged
merged 1 commit into from Feb 16, 2017
Merged

Add instructions on starting server to README #154

merged 1 commit into from Feb 16, 2017

Conversation

svarlamov
Copy link
Contributor

Interested to see the feedback on this. I think I might be missing some deeper features/configs that would be worth highlighting

@fbricon
Copy link
Contributor

fbricon commented Feb 15, 2017

@svarlamov thanks for your contribution. However, before we can merge it, you need to do 2 things:


5. To start the server in the active terminal, run:
```bash
java -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=1044 -Declipse.application=org.eclipse.jdt.ls.core.id1 -Dosgi.bundles.defaultStartLevel=4 -Declipse.product=org.eclipse.jdt.ls.core.product -Dlog.protocol=true -Dlog.level=ALL -noverify -Xmx1G -jar ./plugins/org.eclipse.equinox.launcher_1.4.0.v20161219-1356.jar -configuration ./config_linux -data /path/to/data
Copy link
Contributor

Choose a reason for hiding this comment

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

The version of launcher (org.eclipse.equinox.launcher_1.4.0.v20161219-1356.jar) may change as the eclipse versions changes perhaps we should have a note of it.

Copy link
Contributor

@gorkem gorkem left a comment

Choose a reason for hiding this comment

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

Looks OK to me. Let me know if you want to act on the comment about launcher version otherwise I can merge it.

@gorkem
Copy link
Contributor

gorkem commented Feb 15, 2017

@svarlamov do you have an ECA signed? I can not find a record for your email? Can you possibly sign the CLA?

Signed-off-by: Sasha Varlamov <sasha@sashavarlamov.com>

Fix MD formatting

Signed-off-by: Sasha Varlamov <sasha@sashavarlamov.com>

Add instructions for starting the server in the README. Ref: #153

Signed-off-by: Sasha Varlamov <sasha@sashavarlamov.com>

Add instructions for starting the server in the README. Ref: #153

Signed-off-by: Sasha Varlamov <sasha@sashavarlamov.com>
@svarlamov
Copy link
Contributor Author

Signed the ECA and squashed the commits. Also just added a note about the jar file version -- should be good to go

@gorkem gorkem merged commit 3430194 into eclipse-jdtls:master Feb 16, 2017
@gorkem
Copy link
Contributor

gorkem commented Feb 16, 2017

@svarlamov Thanks for the PR

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.

None yet

3 participants