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

[Jonathan Gibbons]

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<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) {