Re: RFR: 7133124 Remove redundant packages from JAR command line

2012-01-30 Thread Dmitry Samersoff
John, Actually the goal of my letter is not to promote new integration scheme. Just to remind that we need to put some efforts to internal process review and optimization. But, see answers below (inline): Integration method I mentioned often used in open source projects, because it doesn't

Re: RFR: 7133124 Remove redundant packages from JAR command line

2012-01-30 Thread Robert Ottenhag
Dmitry, I think this discussion diverged somewhat from the original topic, but I do agree with you that we must also attack the problem on a process level. With the model you propose (and also the existing model) I would also like to stress the need for continuous and automatic builds

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

2012-01-30 Thread Alan Bateman
The following webrev is a patch from Miran Kos and Martin Grebac (cc'ed) to remove the JAX-WS dependency on apt and the com.sun.mirror API: http://cr.openjdk.java.net/~alanb/7140918/webrev/ This is needed before Joe Darcy can wield his machete. Please cc'ed Miran and Martin on my review

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

2012-01-30 Thread Joe Darcy
On 01/30/2012 07:08 AM, Alan Bateman wrote: The following webrev is a patch from Miran Kos and Martin Grebac (cc'ed) to remove the JAX-WS dependency on apt and the com.sun.mirror API: http://cr.openjdk.java.net/~alanb/7140918/webrev/ This is needed before Joe Darcy can wield his machete.

Re: RFR: 7133124 Remove redundant packages from JAR command line

2012-01-30 Thread Kelly O'Hair
Diverged and in transit to another planet. :^( A push to a shared repo without verifying it builds on all supported platforms is risky behavior, and one that can consume needless resources finding out it doesn't build, and more importantly waste your co-worker's time undoing it. We have the

RE: RFR: 7133124 Remove redundant packages from JAR command line

2012-01-30 Thread Robert Ottenhag
Inline, -Original Message- From: Kelly O'Hair Sent: Monday, January 30, 2012 6:27 PM To: Robert Ottenhag Cc: Dmitry Samersoff; serviceability-...@openjdk.java.net; John Coomes; build-dev@openjdk.java.net Subject: Re: RFR: 7133124 Remove redundant packages from JAR command line

Re: RFR: 7133124 Remove redundant packages from JAR command line

2012-01-30 Thread Jonathan Gibbons
On 01/30/2012 09:41 AM, Robert Ottenhag wrote: A push to a shared repo without verifying it builds on all supported platforms is risky behavior, and one that can consume needless resources finding out it doesn't build, and more importantly waste your co-worker's time undoing it. We

Re: RFR: 7133124 Remove redundant packages from JAR command line

2012-01-30 Thread Jonathan Gibbons
On 01/30/2012 03:41 PM, Kelly O'Hair wrote: I also think that extremely low or no risk changes need not follow this rule, but the problem is getting people to agree with 'no risk changes' are. I've seen enough 'low risk' changes bring the house down that I'm on the paranoid side. :^( How

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

2012-01-30 Thread David Holmes
On 31/01/2012 1:08 AM, Alan Bateman wrote: The following webrev is a patch from Miran Kos and Martin Grebac (cc'ed) to remove the JAX-WS dependency on apt and the com.sun.mirror API: http://cr.openjdk.java.net/~alanb/7140918/webrev/ This is needed before Joe Darcy can wield his machete.

Re: Patch to fix build breakage with GCC 4.7

2012-01-30 Thread David Holmes
Hi Deepak, The primary change here is a build change so I've cc'ed build-dev. The majority of the changes are to JVMTI demo files hence I've cc'd serviceability-dev. I think JDK8-dev doesn't need to be included now so I've bcc'd it. While gcc compilation on sparc is rare I'm not sure that

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

2012-01-30 Thread Joe Darcy
On 01/30/2012 04:50 PM, David Holmes wrote: On 31/01/2012 1:08 AM, Alan Bateman wrote: The following webrev is a patch from Miran Kos and Martin Grebac (cc'ed) to remove the JAX-WS dependency on apt and the com.sun.mirror API: http://cr.openjdk.java.net/~alanb/7140918/webrev/ This is needed

Re: RFR: 7133124 Remove redundant packages from JAR command line

2012-01-30 Thread Kelly O'Hair
On Jan 30, 2012, at 9:41 AM, Robert Ottenhag wrote: Inline, -Original Message- From: Kelly O'Hair Sent: Monday, January 30, 2012 6:27 PM To: Robert Ottenhag Cc: Dmitry Samersoff; serviceability-...@openjdk.java.net; John Coomes; build-dev@openjdk.java.net Subject: Re: RFR:

Re: Patch to fix build breakage with GCC 4.7

2012-01-30 Thread Kelly O'Hair
This change looks fine to me. It would be ok with me for this change to be pushed into the jdk8/build forest. Once there, I can do a JPRT run to verify it, but I really don't seen any issue here. -kto On Jan 30, 2012, at 4:57 PM, David Holmes wrote: Hi Deepak, The primary change here is a

How to handle a merge problem

2012-01-30 Thread Pete Brunet
I just did the following: sh ./get_source.sh and got this: snip # cd ./jdk hg pull -u pulling from http://hg.openjdk.java.net/jdk8/jdk8//jdk searching for changes adding changesets adding manifests adding file changes added 563 changesets with 3123 changes to 2661 files (+1 heads) not updating:

Re: Patch to fix build breakage with GCC 4.7

2012-01-30 Thread David Holmes
On 31/01/2012 12:36 PM, Deepak Bhole wrote: * David Holmesdavid.hol...@oracle.com [2012-01-30 19:57]: While gcc compilation on sparc is rare I'm not sure that simply deleting the sparc-only option unconditionally is the right thing to do. I thought about that too. But I was unable to find

Re: How to handle a merge problem

2012-01-30 Thread David Holmes
Pete, On 31/01/2012 12:45 PM, Pete Brunet wrote: I just did the following: sh ./get_source.sh and got this: snip # cd ./jdk hg pull -u pulling from http://hg.openjdk.java.net/jdk8/jdk8//jdk searching for changes adding changesets adding manifests adding file changes added 563 changesets with

Re: How to handle a merge problem

2012-01-30 Thread Pete Brunet
Hi David, It might help to know that I had pulled the code back in September or so, made several changes, and then got diverted onto another project for a while. So now I'm trying to get back in sync with the latest code. The first thing I did was update my patch with hg qrefresh and then from

Re: How to handle a merge problem

2012-01-30 Thread David Holmes
On 31/01/2012 2:50 PM, Pete Brunet wrote: Hi David, It might help to know that I had pulled the code back in September or so, made several changes, and then got diverted onto another project for a while. So now I'm trying to get back in sync with the latest code. The first thing I did was

Re: How to handle a merge problem

2012-01-30 Thread Dmitry Samersoff
Pete, 1. Do hg diff in your workspace, save the patch somewhere 2. Do hg fclone http://openjdk_ws_your_need 3. Try to apply your patch PS: Personally, I tried mq but don't find it useful -Dmitry On 2012-01-31 08:50, Pete Brunet wrote: Hi David, It might help to know that I had pulled the