On Wed, 6 Jul 2022 08:24:21 GMT, Christian Stein <cst...@openjdk.org> wrote:

> Please review the change to update to using jtreg 7.
> 
> The primary change is to the `jib-profiles.js` file, which specifies the 
> version of jtreg to use, for those systems that rely on this file. In 
> addition, the `requiredVersion` has been updated in the various `TEST.ROOT` 
> files.

I think the configure script should verify that we got the minimum jtreg 
needed. This is not really related to the change, but seeing this PR got me 
thinking about it. I also think a change such as this might bite people who 
have local setups with older jtreg installations, and then it would be nice for 
it to fail early in configure.

If you are interested in including such a check, I could contribute the code 
for the configure script.

I just want to check one thing: I see there are multiple `requiredVersion` in 
different TEST.ROOT files. Does this new requirement apply to *all* our jtreg 
tests?

-------------

PR: https://git.openjdk.org/jdk/pull/9393

Reply via email to