Re: Review Request: 8005850: build-infra: Make --enable-openjdk-only really disable custom

2013-01-09 Thread Erik Joelsson
On 2013-01-09 03:44, David Holmes wrote: On 8/01/2013 10:56 PM, Erik Joelsson wrote: The open part of this review. Move handling of --enable-openjdk-only to before the first custom hook call. http://cr.openjdk.java.net/~erikj/8005850/webrev.root.01/ Minor nit: SUPPRESS has two P's :) One

Re: Mac build fun

2013-01-09 Thread Erik Joelsson
Interesting observations. Definitely something to keep in mind at least. /Erik On 2013-01-09 04:30, David Chase wrote: I had a failure building on a Mac (Mountain Lion, 10.8.2), both new build and old build, at this step:

Re: Mac build fun

2013-01-09 Thread Michael McMahon
Are you building on a remote filesystem like NFS? Those .DS_Store files get created by the finder and can cause problems on remote filesystems where they become visible to other software. There is a way to disable creation of .DS_Store on remote filesystems documented at

Re: Review request: 8005096: Move a few source files in swing/beaninfo and in a demo.

2013-01-09 Thread Sergey Malenkov
Hi Fredrik, It seems to be reasonable that these sources should be part of the src.zip and that it was a bug previously that they were not. I don't think so. First, *BeanInfo classes must be packed to dt.jar, not rt.jar. So, it will be strange to provide sources for nonexistent (by default)

Re: Review request: 8005096: Move a few source files in swing/beaninfo and in a demo.

2013-01-09 Thread Fredrik Öhrström
2013/1/9 Sergey Malenkov sergey.malen...@oracle.com: First, *BeanInfo classes must be packed to dt.jar, not rt.jar. So, it will be strange to provide sources for nonexistent (by default) classes. Second, *BeanInfo java files are generated automatically. They are not well-formatted and

Re: Review request: 8005096: Move a few source files in swing/beaninfo and in a demo.

2013-01-09 Thread Sergey Malenkov
Hi Fredrik, Thanks! It looks good to me. Regards, SAM On 09.01.2013 16:07, Fredrik Öhrström wrote: 2013/1/9 Sergey Malenkov sergey.malen...@oracle.com: First, *BeanInfo classes must be packed to dt.jar, not rt.jar. So, it will be strange to provide sources for nonexistent (by default)

hg: jdk8/build/jdk: 8005096: Move a few source files in swing/beaninfo and in a demo.

2013-01-09 Thread fredrik . ohrstrom
Changeset: 1868bde529b8 Author:ohrstrom Date: 2013-01-09 13:33 +0100 URL: http://hg.openjdk.java.net/jdk8/build/jdk/rev/1868bde529b8 8005096: Move a few source files in swing/beaninfo and in a demo. Reviewed-by: ohair, erikj, malenkov ! make/javax/swing/beaninfo/SwingBeans.gmk -

Re: Request for Review: Java SE 8 Compact Profiles

2013-01-09 Thread Erik Joelsson
Looks good to me now. /Erik On 2013-01-08 09:02, David Holmes wrote: I have updated webrevs: http://cr.openjdk.java.net/~dholmes/8004265.v2/webrev.top/ http://cr.openjdk.java.net/~dholmes/8004265.v2/webrev.jdk/ I've addressed the minor suggestions that have been given by Kelly and Erik eg:

Re: Review Request: 8005855: build-infra: Remove -R flag when cross compiling

2013-01-09 Thread Erik Joelsson
8003958 fixes this for sizer generation yes, but there is also one library in CompileNativeLibraries that has X_LIBS on the link line. But I suppose we shouldn't fix it unless there is a problem with it. /Erik On 2013-01-09 03:56, David Holmes wrote: On 9/01/2013 5:56 AM, Kelly O'Hair wrote:

Re: Mac build fun

2013-01-09 Thread David Chase
On 2013-01-09, at 5:07 AM, Michael McMahon michael.x.mcma...@oracle.com wrote: Are you building on a remote filesystem like NFS? Those .DS_Store files get created by the finder and can cause problems on remote filesystems where they become visible to other software. There is a way to

Review Request: 8005903: build-infra: bad symlink: j2sdk-bundle/jdk1.8.0.jdk/Contents/MacOS/libjli.dylib

2013-01-09 Thread Erik Joelsson
Correcting the symlink to libjli in the macosx jdk bundle. http://cr.openjdk.java.net/~erikj/8005903/webrev.jdk.01/ http://cr.openjdk.java.net/%7Eerikj/8005903/webrev.jdk.01/ /Erik

Re: Review Request: 8005903: build-infra: bad symlink: j2sdk-bundle/jdk1.8.0.jdk/Contents/MacOS/libjli.dylib

2013-01-09 Thread Tim Bell
Hi Erik: Correcting the symlink to libjli in the macosx jdk bundle. http://cr.openjdk.java.net/~erikj/8005903/webrev.jdk.01/ http://cr.openjdk.java.net/%7Eerikj/8005903/webrev.jdk.01/ Looks good. Tim

hg: jdk8/build/jdk: 8005903: build-infra: bad symlink: j2sdk-bundle/jdk1.8.0.jdk/Contents/MacOS/libjli.dylib

2013-01-09 Thread erik . joelsson
Changeset: 2cc29d0b9eaf Author:erikj Date: 2013-01-09 16:13 +0100 URL: http://hg.openjdk.java.net/jdk8/build/jdk/rev/2cc29d0b9eaf 8005903: build-infra: bad symlink: j2sdk-bundle/jdk1.8.0.jdk/Contents/MacOS/libjli.dylib Reviewed-by: tbell ! makefiles/Bundles.gmk

Re: Mac build fun

2013-01-09 Thread Kelly O'Hair
Just for the record, I filed a P4 bug JDK-8005946 - Mac build problems with .DS_Store files so this information about .DS_Store files doesn't get completely lost. Not sure what we can do, but figured I should save the issue. -kto On Jan 9, 2013, at 5:33 AM, David Chase wrote: On

Mac problem(?), new build, --with-debug-level=slowdebug

2013-01-09 Thread David Chase
Summary: I wanted to debug, I configured thoroughly with slowdebug, but was not able to debug the result. More detailed: I configured jdk8tl in common/makefiles, sh ../autoconf/configure --with-debug-level=slowdebug cleaned, and then built: make CONF=macosx-x86_64-normal-server-slowdebug

Request for review: 8004834: Add doclint support into javadoc

2013-01-09 Thread Jonathan Gibbons
Build folk, We will soon want to push changes into TL to enable doclint support in javadoc. This will require a minor build change until we can fix up (lots of) errors in our javadoc comments. The problem is that because of the many errors in our doc comments, javadoc will now exit with a

Re: Mac problem(?), new build, --with-debug-level=slowdebug

2013-01-09 Thread Kelly O'Hair
Question 1: What specific forest did you get your sources from? -kto On Jan 9, 2013, at 10:43 AM, David Chase wrote: Summary: I wanted to debug, I configured thoroughly with slowdebug, but was not able to debug the result. More detailed: I configured jdk8tl in common/makefiles, sh

Re: Mac problem(?), new build, --with-debug-level=slowdebug

2013-01-09 Thread David Chase
On 2013-01-09, at 5:26 PM, Kelly O'Hair kelly.oh...@oracle.com wrote: Question 1: What specific forest did you get your sources from? -kto I configured jdk8tl in common/makefiles, default = http://hg.openjdk.java.net/jdk8/tl Then in hotspot-comp, I also configure for slowdebug, and

Re: Request for review: 8004834: Add doclint support into javadoc

2013-01-09 Thread Tim Bell
Jon: We will soon want to push changes into TL to enable doclint support in javadoc. This will require a minor build change until we can fix up (lots of) errors in our javadoc comments. The problem is that because of the many errors in our doc comments, javadoc will now exit with a non-zero

Re: Mac problem(?), new build, --with-debug-level=slowdebug

2013-01-09 Thread Kelly O'Hair
Not sure the configuremake setup works in those repos yet. Just the jdk8/build forest at this time. You could try: hg clone http://hg.openjdk.java.net/jdk8/build jdk8-build cd jdk8-build sh ./get_source.sh sh ./get_source.ch http://closedjdk.us.oracle.com # If you want the full closed