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.dui...@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.bate...@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. >> >
