Re: printf errors compiling with new gcc

2009-10-23 Thread John Rose
We talked about this on the bsd-port mailing list. It may be due to new error checks in GCC 4.2. On Mac, even after those compile-time errors are fixed, GCC 4.2 does not build a usable hotspot. After upgrading to Snow Leopard (10.6), I had to back the gcc to 4.0 for building hotspot. HT

Re: 6856630: Restructure jaxp/jaxws repository

2009-10-23 Thread Andrew John Hughes
2009/10/23 Kelly O'Hair : > > Sigh... if I'm changing jaxp, might as well change jaxws too > keep them in sync. > > I effectively included the drop.dir change you wanted with these > webrevs: > > http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-tl-jaxp/webrev/ > http://cr.openjdk.java.net/~ohair

Re: hotspot build failure on ubuntu x86_64

2009-10-23 Thread Christian Thalinger
On Fri, 2009-10-23 at 14:32 +0800, Weijun Wang wrote: > Hi Martin > > Since the error line is only a printf message, I just remove the line. > And now everything builds fine! > > I'll see if the PTR_FORMAT macro is used in other places and why there's > no problem there. This line is part of the

Re: 6856630: Restructure jaxp/jaxws repository

2009-10-23 Thread Kelly O'Hair
Jonathan Gibbons wrote: Kelly O'Hair wrote: Andrew, Sorry I'm so slow on this, ... I'm looking at updating the jaxp bundle. The webrev is: http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-tl-jaxp-6861589/webrev/ Updated jaxp source bundle, and complete removal of anything that would use

Re: 6856630: Restructure jaxp/jaxws repository

2009-10-23 Thread Jonathan Gibbons
Kelly O'Hair wrote: Jonathan Gibbons wrote: Kelly O'Hair wrote: Andrew, Sorry I'm so slow on this, ... I'm looking at updating the jaxp bundle. The webrev is: http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-tl-jaxp-6861589/webrev/ Updated jaxp source bundle, and complete removal of an

Re: 6856630: Restructure jaxp/jaxws repository

2009-10-23 Thread Kelly O'Hair
Andrew John Hughes wrote: 2009/10/23 Kelly O'Hair : Sigh... if I'm changing jaxp, might as well change jaxws too keep them in sync. I effectively included the drop.dir change you wanted with these webrevs: http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-tl-jaxp/webrev/ http://cr.openjdk.

Re: 6856630: Restructure jaxp/jaxws repository

2009-10-23 Thread Kelly O'Hair
Jonathan Gibbons wrote: Kelly O'Hair wrote: Jonathan Gibbons wrote: Kelly O'Hair wrote: Andrew, Sorry I'm so slow on this, ... I'm looking at updating the jaxp bundle. The webrev is: http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-tl-jaxp-6861589/webrev/ Updated jaxp source bundle,