Re: RFR: 8075617, 8075616 Create tests to check wsgen, schemagen work with multi-version jar

2017-02-01 Thread Alan Bateman
On 01/02/2017 16:46, Andrey Nazarov wrote: Hi Alan, I moved tests to suggested locations. http://cr.openjdk.java.net/~anazarov/8075617/webrev.02/webrev/ Thanks, I'm happy with these locations. -Alan

Re: RFR: 8075617, 8075616 Create tests to check wsgen, schemagen work with multi-version jar

2017-02-01 Thread Andrey Nazarov
Hi Alan, I moved tests to suggested locations. http://cr.openjdk.java.net/~anazarov/8075617/webrev.02/webrev/ —Andrey > On 1 Feb 2017, at 11:25, Alan Bateman wrote: > > On 31/01/2017 21:45, Lance Andersen wrote: > >> Hi Andrey, >> >> The test looks reasonable. >> >> Assuming Alan has blesse

Re: RFR: 8075617, 8075616 Create tests to check wsgen, schemagen work with multi-version jar

2017-02-01 Thread Alan Bateman
On 31/01/2017 21:45, Lance Andersen wrote: Hi Andrey, The test looks reasonable. Assuming Alan has blessed the location and we have run these on all platforms via JPRT, you should be good to go. I was busy with other things and didn't see Andrey's updated webrev proposing to put the tests i

Re: RFR: 8075617, 8075616 Create tests to check wsgen, schemagen work with multi-version jar

2017-01-31 Thread Lance Andersen
Hi Andrey, The test looks reasonable. Assuming Alan has blessed the location and we have run these on all platforms via JPRT, you should be good to go. Best Lance > On Jan 31, 2017, at 8:05 AM, Andrey Nazarov > wrote: > > Hi, > > Anyone please review and approve > > —Andrey >> On 26 Jan 20

Re: RFR: 8075617, 8075616 Create tests to check wsgen, schemagen work with multi-version jar

2017-01-31 Thread Andrey Nazarov
Hi, Anyone please review and approve —Andrey > On 26 Jan 2017, at 21:29, Andrey Nazarov wrote: > > Hi > Move tests to new place. Please review. > http://cr.openjdk.java.net/~anazarov/8075617/webrev.01/webrev/ > > > —Andrey >> On

Re: RFR: 8075617, 8075616 Create tests to check wsgen, schemagen work with multi-version jar

2017-01-26 Thread Andrey Nazarov
Hi Move tests to new place. Please review. http://cr.openjdk.java.net/~anazarov/8075617/webrev.01/webrev/ —Andrey > On 25 Jan 2017, at 19:04, Andrey Nazarov wrote: > > >> On 25 Jan 2017, at 19:00, Alan Bateman wrote: >> >> On 2

Re: RFR: 8075617, 8075616 Create tests to check wsgen, schemagen work with multi-version jar

2017-01-25 Thread Andrey Nazarov
> On 25 Jan 2017, at 19:00, Alan Bateman wrote: > > On 25/01/2017 15:25, Andrey Nazarov wrote: > >> : >> What is the right location? > These tools are maintained in the upstream Metro project, I suspect the tests > for these tools are located there too but I'm not sure. There are handful of >

Re: RFR: 8075617, 8075616 Create tests to check wsgen, schemagen work with multi-version jar

2017-01-25 Thread Alan Bateman
On 25/01/2017 15:25, Andrey Nazarov wrote: : What is the right location? These tools are maintained in the upstream Metro project, I suspect the tests for these tools are located there too but I'm not sure. There are handful of JAXB and JAX-WS tests in the jdk repo but I don't think they dire

Re: RFR: 8075617, 8075616 Create tests to check wsgen, schemagen work with multi-version jar

2017-01-25 Thread Andrey Nazarov
> On 25 Jan 2017, at 18:03, Alan Bateman wrote: > > On 25/01/2017 14:45, Andrey Nazarov wrote: > >> Hi, >> >> I’ve added sanity checks that wsgen and schemagen tools works with >> multi-release jar files. >> >> Review http://cr.openjdk.java.net/~anazarov/8075617/webrev.00/webrev/ >> >> JBS:

Re: RFR: 8075617, 8075616 Create tests to check wsgen, schemagen work with multi-version jar

2017-01-25 Thread Alan Bateman
On 25/01/2017 14:45, Andrey Nazarov wrote: Hi, I’ve added sanity checks that wsgen and schemagen tools works with multi-release jar files. Review http://cr.openjdk.java.net/~anazarov/8075617/webrev.00/webrev/ JBS: https://bugs.openjdk.java.net/browse/JDK-8075616

RFR: 8075617, 8075616 Create tests to check wsgen, schemagen work with multi-version jar

2017-01-25 Thread Andrey Nazarov
Hi, I’ve added sanity checks that wsgen and schemagen tools works with multi-release jar files. Review http://cr.openjdk.java.net/~anazarov/8075617/webrev.00/webrev/ JBS: https://bugs.openjdk.java.net/browse/JDK-8075616 , https://bugs.openjdk.