RFR: 8015068 : (m) Use jtreg -exclude for problemlist.txt processing

Mike Duigou mike.duigou at oracle.com
Thu Aug 8 04:18:35 UTC 2013





On 2013-08-07, at 18:22, Alan Bateman <Alan.Bateman at oracle.com> wrote:

> On 07/08/2013 14:44, Mike Duigou wrote:
>> Hello all;
>> 
>> This changesest simplifies how the the jdk/test/Makefile processes excluded tests. Previously the test exclusions were pre-processed by scripts in the Makefile before being passed to JTReg. JTReg will now the all the processing. The change depends upon improvements in JTReg since the test exclude mechanism was originally defined.
>> 
>> There are some changes in the reporting. Most obvious is that the excludelist.txt output file is no longer produced. The 'excluded' count in the summary report may be removed in a future reporting depending on whether the value can be calculated some other way.
>> 
>> Additional to the exclusion list processing changes some other changes are possible as a result of the changes to exclusion processing. In particular the obsolete (and incorrect) logic for determining the build path included in the test/Makefile is now removed. If no output directory is provided via ALT_OUTPUTDIR then a default location in the current directory is used, "testoutput". At some point in the future this could be improved to get the CONF from the build configuration but that is not currently possible.
>> 
>> Testing of this patch requires using a source build of JTReg as it requires one fix that is not in the promoted builds.
>> 
>> http://cr.openjdk.java.net/~mduigou/JDK-8015068/0/webrev/
>> 
>> Mike
> It's good to see this logic going away. Also defaulting the output directory to TEST_ROOT (= pwd) is an improvement.
> 
> One thing that isn't completely clear to be is the whether the multiple -exclude options work as advertised (because that hasn't worked for me in the past and I always ended cat'ing them as per the old make file logic). I assume you have tested with the Oracle jdk/test/closed repo present to verify this.
> 

Yes. This is the fix that requires the updated jtreg. I won't push this change until jtreg is next promoted (b07)

> -Alan
> 



More information about the build-dev mailing list