Re: RFR: JDK-8152666: The new Hotspot Build System

2016-04-07 Thread Daniel D. Daugherty
> I'm not sure if I'm formally allowed to be a reviewer, since I've > wrote the absolute majority of the code myself. The way I've done this in the past is a "Contributed-by:" line listing all of the folks that contributed and a "Reviewed-by:" line listing all the reviewers. Magnus, you reviewed

Re: Provide zipped javadoc archive from make

2016-04-07 Thread Jiri Vanek
On 04/07/2016 03:01 PM, Erik Joelsson wrote: On 2016-04-07 14:48, Jiri Vanek wrote: Hello, For the JDK 9 change, beware that we are going to be making new bundle targets for all kinds of bundles. I'm hoping to start that work soon. It might mean a reimplementation of this patch, not sure ye

Re: Provide zipped javadoc archive from make

2016-04-07 Thread Erik Joelsson
On 2016-04-07 14:48, Jiri Vanek wrote: Hello, For the JDK 9 change, beware that we are going to be making new bundle targets for all kinds of bundles. I'm hoping to start that work soon. It might mean a reimplementation of this patch, not sure yet. Your docs bundle is quite different from th

Re: Provide zipped javadoc archive from make

2016-04-07 Thread Jiri Vanek
Hello, For the JDK 9 change, beware that we are going to be making new bundle targets for all kinds of bundles. I'm hoping to start that work soon. It might mean a reimplementation of this patch, not sure yet. Your docs bundle is quite different from the bundle we need so we likely need to pro

Re: Provide zipped javadoc archive from make

2016-04-07 Thread Jiri Vanek
Hello, For the JDK 9 change, beware that we are going to be making new bundle targets for all kinds of bundles. I'm hoping to start that work soon. It might mean a reimplementation of this patch, not sure yet. Your docs bundle is quite different from the bundle we need so we likely need to pro

Re: Provide zipped javadoc archive from make

2016-04-07 Thread Erik Joelsson
Hello, For the JDK 9 change, beware that we are going to be making new bundle targets for all kinds of bundles. I'm hoping to start that work soon. It might mean a reimplementation of this patch, not sure yet. Your docs bundle is quite different from the bundle we need so we likely need to pr

Re: RFR: JDK-8152666: The new Hotspot Build System

2016-04-07 Thread Maurizio Cimadamore
On 06/04/16 10:14, Erik Joelsson wrote: Hello, I assume the mx projects are for Java code or do they also generate projects for native? The new top level target is only meant to replace the old Visual Studio project generator, at least for now. +1 for having the build system a more central r

Re: Provide zipped javadoc archive from make

2016-04-07 Thread Jiri Vanek
Hello! As I sad I did: I used your patch (also with remarks and suggestions from the last email) http://pkgs.fedoraproject.org/cgit/rpms/java-1.8.0-openjdk.git/tree/jdk8-archivedJavadoc.patch created subpackage http://pkgs.fedoraproject.org/cgit/rpms/java-1.8.0-openjdk.git/commit/?id=db2f51d

Re: RFR: JDK-8153660: jwdp.so/dll missing from JRE image

2016-04-07 Thread Alan Bateman
On 07/04/2016 09:55, Erik Joelsson wrote: Hello, This small patch adds the module jdk.jdwp.agent to the JRE image. Looks good, this was accidentally dropped when we integrated JEP 220. -Alan

Re: RFR(xs): 8153727: AIX jdk build broken after 8145174

2016-04-07 Thread Thomas Stüfe
Thanks all! On Thu, Apr 7, 2016 at 12:48 PM, Erik Joelsson wrote: > Looks good to me. > > /Erik > > > On 2016-04-07 12:40, Thomas Stüfe wrote: > >> Hi Volker, >> >> >> >> On Thu, Apr 7, 2016 at 12:27 PM, Volker Simonis > > >> wrote: >> >> Hi Thomas, >>> >>> thanks for doing this fix. It looks go

Re: RFR [9] 8153286: Move sun.misc.GC to java.rmi ( sun.rmi.transport )

2016-04-07 Thread Magnus Ihse Bursie
On 2016-04-04 13:34, Alan Bateman wrote: On 04/04/2016 12:04, Erik Joelsson wrote: Makefile looks good. If you move Java_sun_rmi_transport_GC_maxObjectInspectionAge out of libjava, should you also remove it from the mapfile for libjava? Yes, libjava/mapfile-vers will need to be updated t

Re: RFR: JDK-8151973: Add Jib and JPRT configuration for linux-arm64 open only

2016-04-07 Thread Magnus Ihse Bursie
On 2016-04-07 11:55, Erik Joelsson wrote: Any chance I could get a reivew for this? I think the profile should be named "linux-aarch64", since that's how the open platform port refers to itself, and also to avoid collision with the closed "arm64" port. Otherwise, the patch looks good. /M

Re: RFR(xs): 8153727: AIX jdk build broken after 8145174

2016-04-07 Thread Erik Joelsson
Looks good to me. /Erik On 2016-04-07 12:40, Thomas Stüfe wrote: Hi Volker, On Thu, Apr 7, 2016 at 12:27 PM, Volker Simonis wrote: Hi Thomas, thanks for doing this fix. It looks good. I've also forwarded your request to build-dev as this is a build change. Just one question: do we pass

Re: RFR: JDK-8152666: The new Hotspot Build System

2016-04-07 Thread Erik Joelsson
On 2016-04-07 11:57, Magnus Ihse Bursie wrote: On 2016-04-06 11:10, Erik Joelsson wrote: Hello Dan and thank you for the review! I know it's a lot to chew through. I have incorporated your changes and published a new webrev: http://cr.openjdk.java.net/~erikj/8152666/webrev.02/ I'm not sure

Re: RFR(xs): 8153727: AIX jdk build broken after 8145174

2016-04-07 Thread Thomas Stüfe
Hi Volker, On Thu, Apr 7, 2016 at 12:27 PM, Volker Simonis wrote: > Hi Thomas, > > thanks for doing this fix. It looks good. > > I've also forwarded your request to build-dev as this is a build change. > > Just one question: do we pass the new test > > test/java/awt/SplashScreen/MultiResolutio

Re: RFR(xs): 8153727: AIX jdk build broken after 8145174

2016-04-07 Thread Volker Simonis
Hi Thomas, thanks for doing this fix. It looks good. I've also forwarded your request to build-dev as this is a build change. Just one question: do we pass the new test test/java/awt/SplashScreen/MultiResolutionSplash/unix/UnixMultiResolutionSplashTest.java which came in with 8145174 on AIX or d

Re: RFR: JDK-8152666: The new Hotspot Build System

2016-04-07 Thread Magnus Ihse Bursie
On 2016-04-06 11:10, Erik Joelsson wrote: Hello Dan and thank you for the review! I know it's a lot to chew through. I have incorporated your changes and published a new webrev: http://cr.openjdk.java.net/~erikj/8152666/webrev.02/ I'm not sure if I'm formally allowed to be a reviewer, since I

Re: RFR: JDK-8151973: Add Jib and JPRT configuration for linux-arm64 open only

2016-04-07 Thread Erik Joelsson
Any chance I could get a reivew for this? /Erik On 2016-03-30 12:10, Erik Joelsson wrote: Updated patch: http://cr.openjdk.java.net/~erikj/8151973/webrev.top.01/ Now that JDK-8152545 has been resolved, there is nothing stopping the open linux-aarch64 port from being cross compiled. With these

RFR: JDK-8153660: jwdp.so/dll missing from JRE image

2016-04-07 Thread Erik Joelsson
Hello, This small patch adds the module jdk.jdwp.agent to the JRE image. Bug: https://bugs.openjdk.java.net/browse/JDK-8153660 Patch: diff -r 4f4adcdff6b4 make/common/Modules.gmk --- a/make/common/Modules.gmk +++ b/make/common/Modules.gmk @@ -115,6 +115,7 @@ # JRE_TOOL_MODULES += \ +j