hg: jigsaw/jake/langtools: 3 new changesets

2016-11-03 Thread jonathan . gibbons
Changeset: 3286b60836a5 Author:jjg Date: 2016-11-03 13:35 -0700 URL: http://hg.openjdk.java.net/jigsaw/jake/langtools/rev/3286b60836a5 temporary fix to merge multiple provides for a service ! src/jdk.compiler/share/classes/com/sun/tools/javac/jvm/ClassWriter.java Changeset:

Re: [9] RFR: 8167240: Write new tests to cover functionality of existing 'jimage' options

2016-11-03 Thread Alexandre (Shura) Iline
Denis, I can see that you have switched to the top level test library with this change. With that you are getting more module dependencies than just java.base. First of all, it would probably make sense to build only the classes you needed (which would be jdk.test.lib.process.ProcessTools, I

Re: [9] RFR: 8167240: Write new tests to cover functionality of existing 'jimage' options

2016-11-03 Thread Jim Laskey (Oracle)
Nice work. +1 > On Nov 3, 2016, at 10:29 AM, Denis Kononenko > wrote: > > Hi, > > I've done some rework accordingly to Alan's and Shura's comments: > > 1) removed overlapped tests from JImageToolTest.java; > > 2) added new tests JImageVerifyTest.java for jimage

[9] RFR: 8167240: Write new tests to cover functionality of existing 'jimage' options

2016-11-03 Thread Denis Kononenko
Hi, I've done some rework accordingly to Alan's and Shura's comments: 1) removed overlapped tests from JImageToolTest.java; 2) added new tests JImageVerifyTest.java for jimage verify; 3) reorganized jtreg's tags; The new WEBREV can be found here:

hg: jigsaw/jake/jdk: Remove CORBA tests from exclude list, get more tests running

2016-11-03 Thread alan . bateman
Changeset: d370ea60c627 Author:alanb Date: 2016-11-03 06:57 + URL: http://hg.openjdk.java.net/jigsaw/jake/jdk/rev/d370ea60c627 Remove CORBA tests from exclude list, get more tests running ! test/ProblemList.txt ! test/java/security/testlibrary/Proc.java !