RFR: 7903097: jtreg could implement ToolProvider [v7]

Jonathan Gibbons jjg at openjdk.java.net
Thu Apr 7 15:33:04 UTC 2022


On Thu, 7 Apr 2022 14:52:24 GMT, Christian Stein <cstein at openjdk.org> wrote:

>> Implement `ToolProvider`, now that the main `jtreg` tool requires JDK >= 9.
>
> Christian Stein has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 11 commits:
> 
>  - Merge branch 'master' into 7903097-implement-toolprovider
>  - Fix indentation
>    
>    [skip actions]
>  - Fix java source launcher call
>  - Apply suggested changes
>  - Prevent non-zero exit code by using supported option
>  - Move `ToolProvider`-related test into a dedicated folder
>  - Update more expected numbers
>  - Expect another passing test
>  - Re-use I18N messages by moving `JtregToolProvider` to `tool` package
>  - Add rule to copy service-related resource files
>  - ... and 1 more: https://git.openjdk.java.net/jtreg/compare/cdcd462f...49c11b09

Marked as reviewed by jjg (Lead).

src/share/classes/com/sun/javatest/regtest/tool/JtregToolProvider.java line 43:

> 41: 
> 42:     @Override
> 43:     public int run(PrintWriter out, PrintWriter err, String... args) {

OK for now, but at some point we should merge this with Tool.main

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

PR: https://git.openjdk.java.net/jtreg/pull/54


More information about the jtreg-dev mailing list