RFR: 7903659 Verify Problemlist contents [v6]

Ludvig Janiuk lujaniuk at openjdk.org
Thu Mar 28 09:21:56 UTC 2024


On Wed, 27 Mar 2024 18:11:19 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:

>> Ludvig Janiuk has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - oops
>>  - fix strange bash contruct
>
> src/share/classes/com/sun/javatest/regtest/tool/i18n.properties line 262:
> 
>> 260: help.main.nativepath.desc=Path to location of native libraries and programs needed \
>> 261:     by the tests.
>> 262: help.main.ve.desc=Verify contents of exclude files
> 
> "validate" may be a better word than "verify"

done

> test/verifyexclude/VerifyExcludeTest.gmk line 190:
> 
>> 188: 	$(BUILDTESTDIR)/verifyexclude.exist.ok \
>> 189:     $(BUILDTESTDIR)/verifyexclude.match.format.1.ok \
>> 190:     $(BUILDTESTDIR)/verifyexclude.shortform.ok
> 
> Something may have gone wrong with the indentation here

fixed

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

PR Review Comment: https://git.openjdk.org/jtreg/pull/181#discussion_r1542569907
PR Review Comment: https://git.openjdk.org/jtreg/pull/181#discussion_r1542569775


More information about the jtreg-dev mailing list