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

[java] The Java LS does not report any diagnostics on Windows #1332

Closed
kittaakos opened this issue Feb 21, 2018 · 7 comments
Closed

[java] The Java LS does not report any diagnostics on Windows #1332

kittaakos opened this issue Feb 21, 2018 · 7 comments
Assignees
Labels
bug bugs found in the application java issues related to the java language

Comments

@kittaakos
Copy link
Contributor

But with JS/TS it works. So there should be something with the Java LS on Windows.

@kittaakos kittaakos added bug bugs found in the application OS/Windows issues related to the Windows OS java issues related to the java language labels Feb 21, 2018
@svenefftinge
Copy link
Contributor

Do you have java installed? Anything in the logs?

@kittaakos
Copy link
Contributor Author

kittaakos commented Feb 23, 2018

Do you have java installed?

Yes.

Anything in the logs?

I have modified the js files inside the node_modules, added a few additional logging, to see if there are any errors there. Nothing.

Please note, the CA works, for instance. And here is the interesting part: this does not report an error although I do not have the type Strssaing and the method prinssat does not exist on the PrintStream:

class azxxzssasadaA {
    public static void main(Strssaing[] args) {
        System.out.prinssat("boo");
    }
}

And this reports an error. See the d after the closing curly brace:

class azxxzssasadaA {
    public static void main(Strssaing[] args) {
        System.out.prinssat("boo");
    }d
}

@kittaakos
Copy link
Contributor Author

I think it is not a Theia issue. If you agree, please close.
screen shot 2018-02-23 at 10 51 37

@kittaakos
Copy link
Contributor Author

Same on OS X.

@kittaakos kittaakos removed the OS/Windows issues related to the Windows OS label Feb 23, 2018
@kittaakos kittaakos changed the title [Windows][java] The Java LS does not report any diagnostics on Windows [java] The Java LS does not report any diagnostics on Windows Feb 23, 2018
@gorkem
Copy link
Contributor

gorkem commented Mar 25, 2018

Some part of the missing diagnostics is due the the missing configuration support on Theia. jdt.ls depends on didChangeConfiguration notification to correctly initialize. I think jdt.ls needs to move the initialization logic as well as defined on eclipse-jdtls/eclipse.jdt.ls#601

@gorkem
Copy link
Contributor

gorkem commented Mar 30, 2018

This is also affected from the fix made on TypeFox/vscode-languageserver-node@e0c1f19

@gorkem gorkem self-assigned this Apr 2, 2018
@gorkem
Copy link
Contributor

gorkem commented May 1, 2018

With the latest updates I no longer can reproduce this issue. Closing. Please reopen if it still exists

@gorkem gorkem closed this as completed May 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application java issues related to the java language
Projects
None yet
Development

No branches or pull requests

3 participants