On Wed, 23 Nov 2022 02:16:30 GMT, David Holmes <[email protected]> wrote:
>> @jonathan-gibbons - Thanks for the review! >> >> I could not find an @requires incantation for saying >> do-not-use-slowdebug-bits >> nor one for saying do-not-use-macosx-aarch64. I don't really do a lot with >> @requires so I could be missing something. >> >>> it's too much like brushing the dirt under the carpet. >> >> Please see the parent bugs for >> [JDK-8297367](https://bugs.openjdk.org/browse/JDK-8297367) >> and [JDK-8297369](https://bugs.openjdk.org/browse/JDK-8297369) and you'll >> see that I have clearly documented the failures that I've been seeing. I do >> plan >> to leave those bugs open, but I've gotten tired of accounting for those >> failures >> in my weekly stress testing runs. > >> I could not find an @requires incantation for saying >> do-not-use-slowdebug-bits nor one for saying do-not-use-macosx-aarch64. > > Something like: > > `@requires vm.debug != slowdebug` > `@requires !(os.arch == "aarch64" && os.family == "mac")` @dholmes-ora: > Something like: > > `@requires vm.debug != slowdebug` `@requires !(os.arch == "aarch64" && > os.family == "mac")` A change like this: @@ -25,6 +25,7 @@ * @test * @bug 8190312 * @summary test redirected URLs for -link + * @requires (vm.debug != slowdebug) * @library /tools/lib ../../lib * @modules jdk.compiler/com.sun.tools.javac.api * jdk.compiler/com.sun.tools.javac.main results in a complaint from jtreg like this: test result: Error. Parse Exception: Syntax error in @requires expression: invalid name: slowdebug `(vm.debug == true)` `(vm.debug == false)` both work as does just plain: `vm.debug` so `vm.debug` is a boolean and not a string. ------------- PR: https://git.openjdk.org/jdk/pull/11278
