Re: [8u] Request for approval for 8172053: (ppc64) Downport of 8170153 breaks build on linux/ppc64 (big endian)

2016-12-28 Thread Erik Joelsson
Change looks ok. /Erik On 2016-12-27 23:30, David Holmes wrote: Hi Volker, As this is a build change I have added build-dev. The change looks fine to me. Aside: More generally it would be nice if there was a simple way to record minimum compiler versions necessary for a given flag/option.

Re: [8u] Request for approval for 8172053: (ppc64) Downport of 8170153 breaks build on linux/ppc64 (big endian)

2016-12-28 Thread Volker Simonis
On Tue, Dec 27, 2016 at 11:30 PM, David Holmes wrote: > Hi Volker, > > As this is a build change I have added build-dev. > > The change looks fine to me. > Thanks for reviewing! > Aside: More generally it would be nice if there was a simple way to record > minimum compiler versions necessary for

Re: RFR(s): 8171958: Several tests from java/time/test/java/time/format requiring jdk.localedata for execution

2016-12-28 Thread Sergei Kovalev
Hi Rachna, Thank you for the comment. I've reviewed usage of module declaration in TEST.properties file and find that it could be removed without any impact. In both cases (with original and modified TEST.properties file) I get same result. Test results: passed: 142; failed: 27 The reason o

RFR(XS): 8067237: [TESTBUG] javax/xml/ws/xsanymixed/Test.java failed on compilation

2016-12-28 Thread Aleks Efimov
Hello, Please, help to review javax/xml/ws/xsanymixed/Test.java test fix that addresses test failure [1] with JDK9 JRE runtime due to missing wsimport tool. The fix is to use the tool from compile JDK: http://cr.openjdk.java.net/~aefimov/8067237/9/00/ The modification was tested with JTRE

Re: RFR(XS): 8067237: [TESTBUG] javax/xml/ws/xsanymixed/Test.java failed on compilation

2016-12-28 Thread Lance Andersen
+1 > On Dec 28, 2016, at 10:03 AM, Aleks Efimov wrote: > > Hello, > > Please, help to review javax/xml/ws/xsanymixed/Test.java test fix that > addresses test failure [1] with JDK9 JRE runtime due to missing wsimport > tool. The fix is to use the tool from compile JDK: >http://cr.openjdk.ja

RE: Does jvisualvm work in JDK 9?

2016-12-28 Thread Frank Yuan
Hi Dmitry Thank you very much for your help! Unluckily visualvm doesn't work for 147 and 150 as well, I have to give up using this tool in current stage... Frank > -Original Message- > From: Dmitry Buzdin [mailto:buz...@gmail.com] > Sent: Wednesday, December 28, 2016 4:57 PM > To: Fr