Re: Removal of the old build system, partial review

2013-10-30 Thread David Holmes
On 31/10/2013 1:01 AM, Alan Bateman wrote: I remember Kelly calling for hazard pay for anyone deleting code and triple pay for anyone deleting Makefile code. At a high level the plan looks reasonable. Aside from build tools then what else are you expecting to look at in step 2? So is the plan

Re: Jarreorder and classlists

2013-10-30 Thread Mike Duigou
Hi Erik; It doesn't look like your updated classlists were pushed and I note that 'HashMap$Entry' has been renamed to 'HashMap$Node'. Are there plans to update the classlists before Java 8 is finalized? This does seem like something where "mostly right" could be worse than having no list at all

Re: [8] RfR 8027567: JDK8 build failure: the correct version of GNU make is being rejected

2013-10-30 Thread David Katleman
On 10/30/2013 10:58 AM, Vincent Ryan wrote: Thanks for the review Chris. For the record, here’s the updated webrev: http://cr.openjdk.java.net/~vinnie/8027567/webrev.01/ Change looks fine Vinnie, approved. Thanks Dave On 30 Oct 2013, at 16:57, Chris Hegarty wrote: The sour

Re: [8] RfR 8027567: JDK8 build failure: the correct version of GNU make is being rejected

2013-10-30 Thread Vincent Ryan
Thanks for the review Chris. For the record, here’s the updated webrev: http://cr.openjdk.java.net/~vinnie/8027567/webrev.01/ On 30 Oct 2013, at 16:57, Chris Hegarty wrote: > The source changes look fine to me, I'm happy to be listed as a reviewer for > this. ( generated-configure.sh, I jus

Re: [8] RfR 8027567: JDK8 build failure: the correct version of GNU make is being rejected

2013-10-30 Thread Chris Hegarty
The source changes look fine to me, I'm happy to be listed as a reviewer for this. ( generated-configure.sh, I just assume is ok once common/autoconf/autogen.sh is run ). -Chris. On 30/10/2013 16:24, Vincent Ryan wrote: Sure. I’ll re-run autogen.sh and submit an updated webrev. Thanks. On 3

Re: [8] RfR 8027567: JDK8 build failure: the correct version of GNU make is being rejected

2013-10-30 Thread Vincent Ryan
Sure. I’ll re-run autogen.sh and submit an updated webrev. Thanks. On 30 Oct 2013, at 16:15, Erik Joelsson wrote: > I'm leaving for the day pretty soon, so feel free to push this to tl if you > have time. > > If you do, don't forget to run common/autoconf/autogen.sh and to also submit > the

Re: [8] RfR 8027567: JDK8 build failure: the correct version of GNU make is being rejected

2013-10-30 Thread Erik Joelsson
I'm leaving for the day pretty soon, so feel free to push this to tl if you have time. If you do, don't forget to run common/autoconf/autogen.sh and to also submit the closed generated-configure. /Erik On 2013-10-30 17:07, Vincent Ryan wrote: Thanks Erik. We’re seeing the issue in TL so it’

Re: [8] RfR 8027567: JDK8 build failure: the correct version of GNU make is being rejected

2013-10-30 Thread Vincent Ryan
Thanks Erik. We’re seeing the issue in TL so it’d be great to get this fixed today. Would you like to take ownership of this issue or shall I push your fix to TL? On 30 Oct 2013, at 16:00, Erik Joelsson wrote: > I found a solution that's also more readable. Posting inline since it's so > small

Re: [8] RfR 8027567: JDK8 build failure: the correct version of GNU make is being rejected

2013-10-30 Thread Erik Joelsson
I found a solution that's also more readable. Posting inline since it's so small a change: diff -r 4f2011496393 common/autoconf/basics.m4 --- a/common/autoconf/basics.m4 +++ b/common/autoconf/basics.m4 @@ -514,7 +514,7 @@ if test "x$IS_GNU_MAKE" = x; then AC_MSG_NOTICE([Found potenti

Re: [8] RfR 8027567: JDK8 build failure: the correct version of GNU make is being rejected

2013-10-30 Thread Erik Joelsson
It seems the problem is with the grep tool used to parse the version string. /usr/xpg4/bin/grep doesn't handle '\(3\.8[12]\)\|\(4\.\)' the same as gnu grep. In jprt it finds /usr/sfw/bin/ggrep which works better. I will see if I can figure out something that works with both. /Erik On 2013-10-

Re: Removal of the old build system, partial review

2013-10-30 Thread Jonathan Gibbons
Removal of langtools files OK. -- Jon On 10/30/2013 07:22 AM, Magnus Ihse Bursie wrote: I have started working on removing the old build system. See https://bugs.openjdk.java.net/browse/JDK-8027566. I plan do to this in several steps. I have not decided wether to commit these steps as separa

Re: [8] RfR 8027567: JDK8 build failure: the correct version of GNU make is being rejected

2013-10-30 Thread Vincent Ryan
Thanks Chris. So reverting to the previous version of builds.m4 will fix this issue on Solaris but will undo the fix for 8026528 on Windows. Maybe Eric can advise. On 30 Oct 2013, at 15:05, Chris Hegarty wrote: > Hi Vinnie, > > I have seen this issue myself, kind of funny ;-) > > ... > co

Re: [8] RfR 8027567: JDK8 build failure: the correct version of GNU make is being rejected

2013-10-30 Thread Chris Hegarty
Hi Vinnie, I have seen this issue myself, kind of funny ;-) ... configure: Found GNU make at /java/devtools/i386/bin/make, however this is not version 3.81 or later. (it is: GNU Make 3.81). Ignoring. configure: error: Cannot find GNU make 3.81 or newer! Please put it in the path, or add e.g. M

Re: Removal of the old build system, partial review

2013-10-30 Thread Alan Bateman
I remember Kelly calling for hazard pay for anyone deleting code and triple pay for anyone deleting Makefile code. At a high level the plan looks reasonable. Aside from build tools then what else are you expecting to look at in step 2? So is the plan to do all of this in jdk8 before RDP2? I

[8] RfR 8027567: JDK8 build failure: the correct version of GNU make is being rejected

2013-10-30 Thread Vincent Ryan
Please review this fix to correct the JDK8 build Configure script. It reverts a recent change to common/autoconf/basics.m4 that was causing a build failure on Solaris. Bug: https://bugs.openjdk.java.net/browse/JDK-8027567 Webrev: http://cr.openjdk.java.net/~vinnie/8027567/

Removal of the old build system, partial review

2013-10-30 Thread Magnus Ihse Bursie
I have started working on removing the old build system. See https://bugs.openjdk.java.net/browse/JDK-8027566. I plan do to this in several steps. I have not decided wether to commit these steps as separate bugs, or just use this division to facilitate developing and reviewing the fix. (Feedba