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

[Mike Duigou]

I have updated the webrev to backout the changes to how JT_HOME is set. This will be addressed in the next set of changes (8020779 https://bugs.openjdk.java.net/browse/JDK-8020779 and  8009683 https://bugs.openjdk.java.net/browse/JDK-8009683) which will revisit how JT_HOME is set and how the jtreg executable is found.

http://cr.openjdk.java.net/~mduigou/JDK-8015068/4/webrev/

Mike


On Nov 5 2013, at 10:13 , Mike Duigou <mike.duigou at oracle.com> wrote:

> One final set of changes was required based upon cross platform testing.
> 
> The revised webrev which passes cross platform testing on all platforms is here:
> 
> http://cr.openjdk.java.net/~mduigou/JDK-8015068/3/webrev/
> 
> The changes are:
> 
> - I had to switch to using uname -s because Solaris doesn't support "-o" option.
> 
> - Some internal tools which scrape the results from Stats.txt were confused by the "FIXME CODETOOLS-7900176" text. I have changed it to not include a space.
> 
> The second issue does point out one important difference between the previous -exclude handling and the handling used by this changeset. The set of excludes or summary counts is not part of the current logs. I would like to see the excluded totals added into the runlist.txt file but currently jtreg doesn't include either excluded or ignored in this list.
> 
> Are we ready yet? Finally?
> 
> Mike
> 
> 
> On Oct 17 2013, at 05:23 , Alan Bateman <Alan.Bateman at oracle.com> wrote:
> 
>> On 16/10/2013 21:22, Mike Duigou wrote:
>>> Hello all;
>>> 
>>> With the imminent promotion of JTReg 4.1b07 it's possible to finally consider completing this changeset!
>>> 
>>> http://cr.openjdk.java.net/~mduigou/JDK-8015068/2/webrev/
>>> 
>>> This updated webrev includes handling of the shared library permissions.
>>> 
>>> Thanks,
>>> 
>>> Mike
>> It would good to get this in when you can. I looked at the updated webrev and it looks mostly the same as the previous rounds. The update to set the permissions on DLLs looks fine, I assume that cygwin chmod must be add an ACE that grants execute permission.
>> 
>> -Alan.
>> 
>