Skip to content
This repository has been archived by the owner on Oct 5, 2022. It is now read-only.

[java] javac is not found #192

Closed
nlpguyz opened this issue Jul 1, 2019 · 3 comments · Fixed by #193
Closed

[java] javac is not found #192

nlpguyz opened this issue Jul 1, 2019 · 3 comments · Fixed by #193

Comments

@nlpguyz
Copy link

nlpguyz commented Jul 1, 2019

I launched the docker image like
sudo docker run -it -p 3000:3000 -v "$(pwd):/home/project:cached" theiaide/theia-java:next

In the terminal, "javac" is not found. The path of "java" points to "jre" instead of "jdk"
theia@cccf82b4924d:/home/project$ which java
/opt/ibm/java/jre/bin/java

Can you please check if JDK is pulled in? Thanks.

marcdumais-work added a commit that referenced this issue Jul 2, 2019
Fixes #192

Signed-off-by: Marc Dumais <marc.dumais@ericsson.com>
@marcdumais-work
Copy link
Member

Hi @nlpguyz ,

It looks like javac is part of the image, but its directory is not in the path: /opt/ibm/java/bin/javac.

Another consideration is that the image we start-from, ibmcom/ibmjava:8-sdk is deprecated and should be replaced. But we can try for a quick fix right now for your issue.

@marcdumais-work
Copy link
Member

PR: #193

@nlpguyz
Copy link
Author

nlpguyz commented Jul 3, 2019

Thanks for your prompt response. The patch fixed the issue!

marcdumais-work added a commit that referenced this issue Jul 3, 2019
Fixes #192

Signed-off-by: Marc Dumais <marc.dumais@ericsson.com>
dwjbosman pushed a commit to dwjbosman/theia-apps that referenced this issue Jul 16, 2019
Fixes theia-ide#192

Signed-off-by: Marc Dumais <marc.dumais@ericsson.com>
Signed-off-by: Dinne Bosman <dwjbosman@yahoo.com>
dwjbosman pushed a commit to dwjbosman/theia-apps that referenced this issue Jul 24, 2019
Fixes theia-ide#192

Signed-off-by: Marc Dumais <marc.dumais@ericsson.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants