Test ReleaseOptionCurrent.java writes ReleaseOption.class in the source directory

[B. Blaser]

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

             boolean result = compiler.getTask(null, fm, null,
options, null, input).call();
             if (!result) {