Re: RFR: 8212008: Use of TREAT_EXIT_CODE_1_AS_0 hide problems with jtreg Java

2018-10-12 Thread Jonathan Gibbons
Unrelated to the proposed change, the test/hotspot/jtreg/Makefile contains this line: 30 31 USE_JTREG_VERSION := 4.1 That would appear to be somewhat out of date, or (more likely) not used. -- Jon On 10/10/2018 09:01 AM, Erik Joelsson wrote: Looks good. A side effect of this is that if

Re: RFR: JDK-8212028: Use run-test makefile framework for testing in Oracle's Mach5

2018-10-12 Thread Magnus Ihse Bursie
> 12 okt. 2018 kl. 18:16 skrev Erik Joelsson : > > >> On 2018-10-12 01:37, Magnus Ihse Bursie wrote: >> Hi Erik, >> >> Thank you for preserving through this, so we finally can move to 100% >> run-test! >> >>> On 2018-10-12 00:29, Erik Joelsson wrote: >>> >>> Hello, >>> >>> (adding servicea

Re: RFR: 8212110: Build of saproc.dll broken on Windows 32 bit after JDK-8210647

2018-10-12 Thread Severin Gehwolf
Hi Erik, On Fri, 2018-10-12 at 09:09 -0700, Erik Joelsson wrote: > Looks good to me. > > I wonder why only this library has that flag. Perhaps something that > should be applied more generally or not at all? (In a followup of > course) Thanks for the review! I suppose the original intention wa

Re: RFR: JDK-8212028: Use run-test makefile framework for testing in Oracle's Mach5

2018-10-12 Thread Erik Joelsson
On 2018-10-12 01:37, Magnus Ihse Bursie wrote: Hi Erik, Thank you for preserving through this, so we finally can move to 100% run-test! On 2018-10-12 00:29, Erik Joelsson wrote: Hello, (adding serviceability-dev and hotspot-dev for test changes) Bug: https://bugs.openjdk.java.net/browse

Re: RFR: 8212110: Build of saproc.dll broken on Windows 32 bit after JDK-8210647

2018-10-12 Thread Erik Joelsson
Looks good to me. I wonder why only this library has that flag. Perhaps something that should be applied more generally or not at all? (In a followup of course) /Erik On 2018-10-12 02:20, Severin Gehwolf wrote: Hi, Please review this fix for a build failure of saproc.dll on Windows 32 bit.

Re: 8212110: Build of saproc.dll broken on Windows 32 bit after JDK-8210647

2018-10-12 Thread Severin Gehwolf
Hi Martin, On Fri, 2018-10-12 at 10:52 +, Doerr, Martin wrote: > Hi Severin, > > thank you for providing this fix so quickly. It looks good to me. Thanks for testing and the review. > I was able to build jdk11u with this patch. > > Just for information: > I'm using Visual Studio 2013 and c

RE: 8212110: Build of saproc.dll broken on Windows 32 bit after JDK-8210647

2018-10-12 Thread Doerr, Martin
Hi Severin, thank you for providing this fix so quickly. It looks good to me. I was able to build jdk11u with this patch. Just for information: I'm using Visual Studio 2013 and configure with --disable-warnings-as-errors. (Otherwise I got Creating support/modules_cmds/jdk.hotspot.agent/jhsdb.exe

RFR: 8212110: Build of saproc.dll broken on Windows 32 bit after JDK-8210647

2018-10-12 Thread Severin Gehwolf
Hi, Please review this fix for a build failure of saproc.dll on Windows 32 bit. For 32 bit Windows builds flag -RTC1 gets added unconditionally. Yet, as per compiler doc[1] it should only be added for development builds. This causes a build failure post JDK-8210647 as in release/fastdebug variant

Re: RFR: JDK-8212028: Use run-test makefile framework for testing in Oracle's Mach5

2018-10-12 Thread Magnus Ihse Bursie
Hi Erik, Thank you for preserving through this, so we finally can move to 100% run-test! On 2018-10-12 00:29, Erik Joelsson wrote: Hello, (adding serviceability-dev and hotspot-dev for test changes) Bug: https://bugs.openjdk.java.net/browse/JDK-8212028 Webrev: http://cr.openjdk.java.net/~