Skip to content
This repository was archived by the owner on Sep 8, 2022. It is now read-only.

SI-7786 Partest finds javac #62

Merged
merged 2 commits into from
Jun 8, 2016
Merged

Conversation

som-snytt
Copy link
Contributor

Since partest is probably running with a JDK,
use JDK_HOME/bin/javac for compilation, if it
exists; otherwise, fall back to whatever is on
the command line.

Javac can still be specified explicitly.

Examines .java file header for "tool args" comments of the form
javac: -deprecation.

som-snytt added 2 commits June 3, 2016 12:06
Since partest is probably running with a JDK,
use JDK_HOME/bin/javac for compilation, if it
exists; otherwise, fall back to whatever is on
the command line.

Javac can still be specified explicitly.
Examines .java file header for "tool args" comments of the form
`javac: -deprecation`.
@lrytz
Copy link
Member

lrytz commented Jun 8, 2016

LGTM

@lrytz lrytz merged commit 8be7e3b into scala:master Jun 8, 2016
@som-snytt som-snytt deleted the issue/7786-pick-javac branch April 6, 2017 23:53
lrytz added a commit to lrytz/scala-partest that referenced this pull request May 9, 2018
lrytz added a commit to lrytz/scala-partest that referenced this pull request May 9, 2018
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 this pull request may close these issues.

2 participants