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

Jonathan Gibbons jonathan.gibbons at oracle.com
Thu May 17 17:01:02 UTC 2018

Bernard,

-d .  would be reasonable in this case, since there will be no other files in the directory.

-- Jon

On 5/17/18 9:28 AM, B. Blaser wrote:

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"); - List options = Arrays.asList("--release", Source.DEFAULT.name); + List options = Arrays.asList("-d", ".", "--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