Re: 7140918: Remove dependency on apt and com.sun.mirror API

2012-01-31 Thread Alan Bateman
On 31/01/2012 01:17, David Holmes wrote: : Okay. So as part of this will the new zip file be installed before the change gets pushed? Yes, that's right. -Alan

Patch for: 7132779: build-infra merge: Enable ccache to work for most developer builds.

2012-01-31 Thread Fredrik Öhrström
Hello hotspot-runtime-dev and build-dev! This is a patch to enable ccache to work for all builds, not only builds where the build number has been explicitly set. Webrev is here: http://cr.openjdk.java.net/~ohrstrom/7132779/webrev.00/ Exported hg patch is here:

Re: How to handle a merge problem

2012-01-31 Thread John Coomes
David Holmes (david.hol...@oracle.com) wrote: On 31/01/2012 2:50 PM, Pete Brunet wrote: ... Or maybe I should start from scratch, apply my patch and fix the issues that way. I seem to remember that the one time I did have a problem applying a patch to fresh clone I was able to fix them.

Re: Patch for: 7132779: build-infra merge: Enable ccache to work for most developer builds.

2012-01-31 Thread Staffan Larsen
Looks good, but where is the new jre_release_version() method used? /Staffan On 31 jan 2012, at 13:31, Fredrik Öhrström wrote: Hello hotspot-runtime-dev and build-dev! This is a patch to enable ccache to work for all builds, not only builds where the build number has been explicitly set.

Re: Patch for: 7132779: build-infra merge: Enable ccache to work for most developer builds.

2012-01-31 Thread David Holmes
On 1/02/2012 5:53 AM, Staffan Larsen wrote: Looks good, but where is the new jre_release_version() method used? It's currently only used in some closed code. The new method avoids the need to make the JRE_RELEASE_VERSION define available to that closed code. I don't understand the comment

...\jdk\make\docs\Makefile needs a fix

2012-01-31 Thread Pete Brunet
Back in November there was a change to ...\jdk\make\docs\Makefile to fix an out of memory problem by raising the memory size for a 32 bit build (line 74) from 512 to 612. That didn't work on my machine: 64 bit Windows 7 with 8 mb. I raised it up to 1024 to bypass the problem. I don't know if