RFR: 8351138: Running subset of gtests gets error printing result information [v3]

SendaoYan syan at openjdk.org
Wed Mar 5 02:10:34 UTC 2025


On Tue, 4 Mar 2025 17:17:32 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> SendaoYan has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Revert regex change in if expr
>
> make/RunTests.gmk line 543:
> 
>> 541: 	      '/YOU HAVE [0-9]+ DISABLED TEST/ { \
>> 542: 	          if (match($$$$0, /YOU HAVE ([0-9]+) DISABLED TEST/, arr)) { \
>> 543: 	            print arr[1]; \
> 
> I can't see how this change is needed. The block after the `/.../` regex will only be executed for lines matching the regex.

You are right, these two line change has been reverted. Thanks.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23904#discussion_r1980555286


More information about the build-dev mailing list