Re: Review Request: 8005347: build-infra: Verify 'gnumake source' at the top level works ok

2013-01-02 Thread Erik Joelsson
I should have checked myself, but it was late. Here is a new webrev with all copyright years updated. http://cr.openjdk.java.net/~erikj/8005347/webrev.root.04/ /Erik On 2013-01-02 01:56, David Holmes wrote: Sorry Erik I misunderstood the change that had been made. I thought common/autoconf/ve

Review Request: 8005575: build-infra: Three JCK tests fails on Solaris with new RE Autoconf-Based build

2013-01-02 Thread Erik Joelsson
In build-infra, an attempt at unifying the handling of properties files was introduced and used in jaxp jaxws. It changed how properties were cleaned quite drastically for these repos, trying to mimic what was going on with StripProperties in jdk and langtools, but using sed. This worked pretty

hg: jdk8/build: 3 new changesets

2013-01-02 Thread erik . joelsson
Changeset: 2d9bb72b4e34 Author:erikj Date: 2012-12-30 12:15 +0100 URL: http://hg.openjdk.java.net/jdk8/build/rev/2d9bb72b4e34 8004490: build-infra: mac: hotspot is always built in product, regardless of --with-debug-level setting Reviewed-by: tbell ! common/autoconf/generated-conf

Re: Review Request: 8005347: build-infra: Verify 'gnumake source' at the top level works ok

2013-01-02 Thread Erik Joelsson
I accidentally hit hg push in the wrong shell window and pushed this prematurely. What should I do now? /Erik On 2013-01-02 10:50, Erik Joelsson wrote: I should have checked myself, but it was late. Here is a new webrev with all copyright years updated. http://cr.openjdk.java.net/~erikj/8005

Re: Review Request: 8005347: build-infra: Verify 'gnumake source' at the top level works ok

2013-01-02 Thread David Holmes
As it was pushed to build-infra I don't see any major problem. If there are changes needed due to review comments then push a follow up changeset. David On 2/01/2013 8:58 PM, Erik Joelsson wrote: I accidentally hit hg push in the wrong shell window and pushed this prematurely. What should I do

Review Request: 8005635: build-infra: Support building install in jprt

2013-01-02 Thread Erik Joelsson
This is the open part of this review. The changes are mostly concerning compare logic and introducing a couple of new variables in spec for images directories. Contributed by Tim Bell and myself. http://cr.openjdk.java.net/~erikj/8005635/webrev.root.01/ http://cr.openjdk.java.net/~erikj/800563

Re: Review Request: 8005635: build-infra: Support building install in jprt

2013-01-02 Thread Erik Joelsson
Missed the changes to common/makefiles/Jprt.gmk. New webrev: http://cr.openjdk.java.net/~erikj/8005635/webrev.root.02/ /Erik On 2013-01-02 17:14, Erik Joelsson wrote: This is the open part of this review. The changes are mostly concerning compare logic and introducing a couple of new variables

Re: Review Request: 8005575: build-infra: Three JCK tests fails on Solaris with new RE Autoconf-Based build

2013-01-02 Thread Kelly O'Hair
Looks ok to me. -kto On Jan 2, 2013, at 2:46 AM, Erik Joelsson wrote: > In build-infra, an attempt at unifying the handling of properties files was > introduced and used in jaxp jaxws. It changed how properties were cleaned > quite drastically for these repos, trying to mimic what was going on

Re: Review Request: 8005347: build-infra: Verify 'gnumake source' at the top level works ok

2013-01-02 Thread Kelly O'Hair
Full steam ahead...if you ask me... -kto On Jan 2, 2013, at 2:58 AM, Erik Joelsson wrote: > I accidentally hit hg push in the wrong shell window and pushed this > prematurely. What should I do now? > > /Erik > > On 2013-01-02 10:50, Erik Joelsson wrote: >> I should have checked myself, bu

Re: Review Request: 8005635: build-infra: Support building install in jprt

2013-01-02 Thread Kelly O'Hair
Looks ok with me. -kto On Jan 2, 2013, at 10:34 AM, Erik Joelsson wrote: > Missed the changes to common/makefiles/Jprt.gmk. New webrev: > > http://cr.openjdk.java.net/~erikj/8005635/webrev.root.02/ > > /Erik > > On 2013-01-02 17:14, Erik Joelsson wrote: >> This is the open part of this review