-
Notifications
You must be signed in to change notification settings - Fork 123
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
jhome introduced #2608
jhome introduced #2608
Conversation
@volodya-lombrozo pls, review |
@@ -179,7 +180,7 @@ private static String run( | |||
*/ | |||
private static void compileJava(final Path generated, final Path classes) { | |||
new Jaxec( | |||
SnippetTestCase.jdkExecutable("javac"), | |||
new Jhome().path("bin/javac").toString(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yegor256 What do you think if we add javac()
and java()
methods to Jhome
. Then, instead of new Jhome().path("bin/javac")
we will need to write only new Jhome().javac()
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yegor256 Created a separate issue for that: yegor256/jhome#5
@rultor merge |
See #2607
PR-Codex overview
Detailed summary
com.yegor256:jhome:0.0.1
with test scope inpom.xml
.com.yegor256.Jhome
inMainTest.java
andSnippetTestCase.java
to locate executable files insideJAVA_HOME
directory.