hg: jdk7/build/corba: 6733313: corba build warnings: /bin/sh: gcc: not found

2009-05-19 Thread kelly . ohair
Changeset: f1e1cccbd13a Author:ohair Date: 2009-05-19 18:09 -0700 URL: http://hg.openjdk.java.net/jdk7/build/corba/rev/f1e1cccbd13a 6733313: corba build warnings: /bin/sh: gcc: not found Reviewed-by: tbell ! make/common/shared/Compiler-gcc.gmk ! make/common/shared/Compiler-sun.gmk

hg: jdk7/build/langtools: 6843041: Remove duplicate README files in repositories (make/README)

2009-05-19 Thread kelly . ohair
Changeset: 5cdce469ea2a Author:ohair Date: 2009-05-19 17:39 -0700 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/5cdce469ea2a 6843041: Remove duplicate README files in repositories (make/README) Reviewed-by: robilad ! README - make/README

hg: jdk7/build/jaxws: 6843041: Remove duplicate README files in repositories (make/README)

2009-05-19 Thread kelly . ohair
Changeset: 3b054db3e277 Author:ohair Date: 2009-05-19 17:39 -0700 URL: http://hg.openjdk.java.net/jdk7/build/jaxws/rev/3b054db3e277 6843041: Remove duplicate README files in repositories (make/README) Reviewed-by: robilad - make/README

hg: jdk7/build/hotspot: 6843041: Remove duplicate README files in repositories (make/README)

2009-05-19 Thread kelly . ohair
Changeset: a77eddcd510c Author:ohair Date: 2009-05-19 17:40 -0700 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/a77eddcd510c 6843041: Remove duplicate README files in repositories (make/README) Reviewed-by: robilad - make/README

hg: jdk7/build/jaxp: 6843041: Remove duplicate README files in repositories (make/README)

2009-05-19 Thread kelly . ohair
Changeset: 259aef5045a1 Author:ohair Date: 2009-05-19 17:38 -0700 URL: http://hg.openjdk.java.net/jdk7/build/jaxp/rev/259aef5045a1 6843041: Remove duplicate README files in repositories (make/README) Reviewed-by: robilad - make/README

hg: jdk7/build/corba: 6843041: Remove duplicate README files in repositories (make/README)

2009-05-19 Thread kelly . ohair
Changeset: 5dcbe748e580 Author:ohair Date: 2009-05-19 17:38 -0700 URL: http://hg.openjdk.java.net/jdk7/build/corba/rev/5dcbe748e580 6843041: Remove duplicate README files in repositories (make/README) Reviewed-by: robilad - make/README

Re: Need reviewer - removing duplicate README files

2009-05-19 Thread Dalibor Topic
Kelly O'Hair wrote: > > Minor cleanup issue. > > 6843041: Remove duplicate README files in repositories (make/README) > > jdk7: http://cr.openjdk.java.net/~ohair/jdk7/6843041-dup-READMEs/webrev/ > > jdk6: http://cr.openjdk.java.net/~ohair/jdk6/6843041-dup-READMEs/webrev/ Both are fine, thanks

Re: Need reviewer - removing duplicate README files

2009-05-19 Thread Jonathan Gibbons
Whoops yes, I see that -- and I will fix the langtools/README to refer to NetBeans 6. -- Jon On May 19, 2009, at 3:04 PM, Kelly O'Hair wrote: There is a langtools/README and a langtools/make/README, they were almost identical (NetBeans 6.0 vs. NetBeans 5.0 reference). I was deleting the lang

Re: Need reviewer - removing duplicate README files

2009-05-19 Thread Kelly O'Hair
There is a langtools/README and a langtools/make/README, they were almost identical (NetBeans 6.0 vs. NetBeans 5.0 reference). I was deleting the langtools/make/README one. -kto Jonathan Gibbons wrote: The langtools README you're thinking of deleting is not a duplicate of anything and is speci

Re: Need reviewer - removing duplicate README files

2009-05-19 Thread Jonathan Gibbons
The langtools README you're thinking of deleting is not a duplicate of anything and is specific to the langtools repository and contains useful info. -- Jon On May 19, 2009, at 2:55 PM, Kelly O'Hair wrote: Minor cleanup issue. 6843041: Remove duplicate README files in repositories (make/R

Need reviewer - removing duplicate README files

2009-05-19 Thread Kelly O'Hair
Minor cleanup issue. 6843041: Remove duplicate README files in repositories (make/README) jdk7: http://cr.openjdk.java.net/~ohair/jdk7/6843041-dup-READMEs/webrev/ jdk6: http://cr.openjdk.java.net/~ohair/jdk6/6843041-dup-READMEs/webrev/ -kto

Re: Request for approval: Bugs 6837665 and 6829575

2009-05-19 Thread Andrew Haley
Jonathan Gibbons wrote: > That's an exceedingly old file you have there -- langtools > src/share/opensource/* should have disappered a long time ago. Hmm, I checked it out of the repository an hour or so ago. `hg parents' says changeset: 43:9820754525dd tag: tip user:ohair date

Re: Request for approval: Bugs 6837665 and 6829575

2009-05-19 Thread Jonathan Gibbons
That's an exceedingly old file you have there -- langtools src/share/ opensource/* should have disappered a long time ago. -- Jon On May 19, 2009, at 9:58 AM, Andrew Haley wrote: This is for the OpenJDK 6 trees. The patch is very similar to that now committed to OpenJDK 7, but it touches a

Request for approval: Bugs 6837665 and 6829575

2009-05-19 Thread Andrew Haley
This is for the OpenJDK 6 trees. The patch is very similar to that now committed to OpenJDK 7, but it touches a few more places that control debuginfo. http://cr.openjdk.java.net/~aph/6829575-jdk6-webrev/ Andrew.