On Sat, 13 Mar 2021 14:20:20 GMT, Daniel D. Daugherty
wrote:
>> Igor Ignatyev has refreshed the contents of this pull request, and previous
>> commits have been removed. The incremental views will show differences
>> compared to the previous content of the PR.
>
> I downloaded the patch and
On Sat, 13 Mar 2021 06:44:12 GMT, Igor Ignatyev wrote:
>> Hi all,
>>
>> could you please review this dull patch that replaces `ClassFileInstaller`
>> w/ `jdk.test.lib.helpers.ClassFileInstaller` in all jtreg test descriptions
>> to ensure we won't get split testlibrary, and removes
>>
On Sat, 13 Mar 2021 06:16:37 GMT, Ioi Lam wrote:
> But I don't understand why this error can happen. It seems like jtreg would
> allow two test cases to interfere with each other.
The root cause seems to be
https://bugs.openjdk.java.net/browse/CODETOOLS-7902847
-
PR:
On Sat, 13 Mar 2021 06:16:37 GMT, Ioi Lam wrote:
>> Igor Ignatyev has refreshed the contents of this pull request, and previous
>> commits have been removed. The incremental views will show differences
>> compared to the previous content of the PR. The pull request contains one
>> new commit
> Hi all,
>
> could you please review this dull patch that replaces `ClassFileInstaller` w/
> `jdk.test.lib.helpers.ClassFileInstaller` in all jtreg test descriptions to
> ensure we won't get split testlibrary, and removes
> `jdk/test/lib/ClassFileInstaller.java` (so it won't be accidentally
> Hi all,
>
> could you please review this dull patch that replaces `ClassFileInstaller` w/
> `jdk.test.lib.helpers.ClassFileInstaller` in all jtreg test descriptions to
> ensure we won't get split testlibrary, and removes
> `jdk/test/lib/ClassFileInstaller.java` (so it won't be accidentally
On Sat, 13 Mar 2021 04:31:31 GMT, Igor Ignatyev wrote:
> Hi all,
>
> could you please review this dull patch that replaces `ClassFileInstaller` w/
> `jdk.test.lib.helpers.ClassFileInstaller` in all jtreg test descriptions to
> ensure we won't get split testlibrary, and removes
>
On Sat, 13 Mar 2021 04:31:31 GMT, Igor Ignatyev wrote:
> Hi all,
>
> could you please review this dull patch that replaces `ClassFileInstaller` w/
> `jdk.test.lib.helpers.ClassFileInstaller` in all jtreg test descriptions to
> ensure we won't get split testlibrary, and removes
>
Hi all,
could you please review this dull patch that replaces `ClassFileInstaller` w/
`jdk.test.lib.helpers.ClassFileInstaller` in all jtreg test descriptions to
ensure we won't get split testlibrary, and removes
`jdk/test/lib/ClassFileInstaller.java` (so it won't be accidentally used).
from