On Thu, 8 Dec 2022 20:38:59 GMT, Erik Joelsson <er...@openjdk.org> wrote:

> The parsing of the TestID in the TEST parameter currently only handles tests 
> that are .java files. Jtreg tests can also be .html and .sh files. This patch 
> extends the macro to also handle these kinds of files.
> 
> Without the patch, trying to run a shell test directly would fail like this:
> 
> 
> $ make test TEST=open/test/jdk/sun/security/tools/keytool/ListKeychainStore.sh
> Building target 'test' in configuration 'linux-x64'
> Test selection 
> 'open/test/jdk/sun/security/tools/keytool/ListKeychainStore.sh', will run:
> * jtreg:open/test/jdk/sun/security/tools/keytool/ListKeychainStore.sh.sh
> 
> Running test 
> 'jtreg:open/test/jdk/sun/security/tools/keytool/ListKeychainStore.sh.sh'
> Error: Cannot find file: 
> open/test/jdk/sun/security/tools/keytool/ListKeychainStore.sh.sh

Seems okay. Thanks

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

Marked as reviewed by dholmes (Reviewer).

PR: https://git.openjdk.org/jdk20/pull/5

Reply via email to