Test ReleaseOptionCurrent.java writes ReleaseOption.class in the source directory (original) (raw)

B. Blaser bsrbnd at gmail.com
Thu May 17 16:28:07 UTC 2018

Hi Jon & David,

Thanks for your feedback.

On 17 May 2018 at 16:27, Jonathan Gibbons <jonathan.gibbons at oracle.com> wrote:

Bernard,

Since you are using classes in the current directory, you do not need to clean the directory (jtreg runs tests in an empty directory) nor do you need to create the directory (javac now does that). -- Jon

The fix is intended to put 'ReleaseOption.class' in the scratch folder, the sub-directory 'classes' being probably unnecessary. If I understand well Jon's explanation, simply '-d .' should make it (as next), is that right?

Thanks, Bernard

diff -r 8e4fcfb4cfe4 test/langtools/tools/javac/options/release/ReleaseOptionCurrent.java --- a/test/langtools/tools/javac/options/release/ReleaseOptionCurrent.java Thu May 17 10:32:26 2018 +0200 +++ b/test/langtools/tools/javac/options/release/ReleaseOptionCurrent.java Thu May 17 18:03:14 2018 +0200 @@ -49,7 +49,7 @@ try (StandardJavaFileManager fm = compiler.getStandardFileManager(null, null, null)) { Iterable<? extends JavaFileObject> input =

fm.getJavaFileObjects(System.getProperty("test.src") + "/ReleaseOption.java");

Source.DEFAULT.name);

"--release", Source.DEFAULT.name);

         boolean result = compiler.getTask(null, fm, null,

options, null, input).call(); if (!result) {

More information about the compiler-dev mailing list