Re: Build of jdk-7udev: Error occurred during initialization of VM

2012-11-14 Thread David Holmes
Whoops! Sorry Kelly - Missed that line involving the java.lang.invoke. David On 15/11/2012 3:52 PM, Tim Bell wrote: See below for the 'flag day' message related to this, sent to the hotspot and 7u dev lists. On 11/14/12 21:24, David Holmes wrote: As general advice yes you need matching jdk an

Re: Review Request: 8001941: build-infra: --disable-precompiled-headers does not seem to work

2012-11-14 Thread Fredrik Öhrström
Looks good. //Fredrik 14 nov 2012 kl. 14:21 skrev Erik Joelsson: > This is a fix for configure parameter --disable-precompiled-headers. It has > now been verified to work. > > http://cr.openjdk.java.net/~erikj/8001941/webrev.root.01/ >

Re: Build of jdk-7udev: Error occurred during initialization of VM

2012-11-14 Thread Tim Bell
See below for the 'flag day' message related to this, sent to the hotspot and 7u dev lists. On 11/14/12 21:24, David Holmes wrote: As general advice yes you need matching jdk and hotspot builds in 7u now (and 8), but this error in itself seems unrelated to invokeDynamic. [snip...] Pete Brun

Re: Build of jdk-7udev: Error occurred during initialization of VM

2012-11-14 Thread David Holmes
As general advice yes you need matching jdk and hotspot builds in 7u now (and 8), but this error in itself seems unrelated to invokeDynamic. David On 15/11/2012 9:27 AM, Kelly O'Hair wrote: It is my understanding that the latest invoke dynamic hotspot/jdk changes has created a situation wher

jdk8 build failure

2012-11-14 Thread Pete Brunet
I did my first clone/build of jdk8 today and the build failed with... make[3]: Entering directory `/cygdrive/c/Users/Pete/JDK8/swing/jdk/make/java/redist/sajdi' ASSEMBLY_IMPORT: ../../../../build/windows-i586/lib/sa-jdi.jar /usr/bin/mkdir -p ../../../../build/windows-i586/lib rm -f ../../../../bu

hg: jdk8/build/jdk: 2 new changesets

2012-11-14 Thread tim . bell
Changeset: ccff3b663797 Author:tbell Date: 2012-11-14 10:21 -0800 URL: http://hg.openjdk.java.net/jdk8/build/jdk/rev/ccff3b663797 8001906: build-infra: warning: [path] bad path element on Solaris Summary: Remove unnecesary -cp parameter from compile line Reviewed-by: ohair, tbell Co

hg: jdk8/build: 6 new changesets

2012-11-14 Thread tim . bell
Changeset: a2df4ee40ecb Author:tbell Date: 2012-11-14 10:05 -0800 URL: http://hg.openjdk.java.net/jdk8/build/rev/a2df4ee40ecb 8002026: build-infra: deploy repository building Summary: Change the compare script to handle deploy build artifacts. Reviewed-by: ohair, tbell Contributed-b

Re: Build of jdk-7udev: Error occurred during initialization of VM

2012-11-14 Thread Kelly O'Hair
It is my understanding that the latest invoke dynamic hotspot/jdk changes has created a situation where you need to build the entire jdk forest or have an import jdk7 that has these new apis or whatever the dependency is. I don't build jdk7 updates much anymore, but somebody should be providing

Build of jdk-7udev: Error occurred during initialization of VM

2012-11-14 Thread Pete Brunet
I recloned hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk today and the build failed as follows: make[3]: Entering directory `/cygdrive/c/Users/Pete/JDK7u/jdk7u-dev/jdk/make/com/sun/jmx' /usr/bin/mkdir -p ../../../../build/windows-i586/classes/javax/management/remote/rmi rm -f ../../../../build/windows-i

hg: jdk8/build/langtools: 16 new changesets

2012-11-14 Thread david . katleman
Changeset: 056d828ac1e1 Author:katleman Date: 2012-11-08 11:53 -0800 URL: http://hg.openjdk.java.net/jdk8/build/langtools/rev/056d828ac1e1 Added tag jdk8-b64 for changeset e6ee43b3e247 ! .hgtags Changeset: 2443d24d096a Author:vromero Date: 2012-11-01 13:06 + URL:

hg: jdk8/build/jdk: 25 new changesets

2012-11-14 Thread david . katleman
Changeset: ad5c1d6b1e16 Author:katleman Date: 2012-11-08 11:52 -0800 URL: http://hg.openjdk.java.net/jdk8/build/jdk/rev/ad5c1d6b1e16 Added tag jdk8-b64 for changeset 26dbd73fb766 ! .hgtags Changeset: bc09a1591629 Author:alanb Date: 2012-11-04 14:07 + URL: http:/

hg: jdk8/build/jaxws: Added tag jdk8-b64 for changeset 5ded18a14bcc

2012-11-14 Thread david . katleman
Changeset: fbe54291c9d3 Author:katleman Date: 2012-11-08 11:51 -0800 URL: http://hg.openjdk.java.net/jdk8/build/jaxws/rev/fbe54291c9d3 Added tag jdk8-b64 for changeset 5ded18a14bcc ! .hgtags

hg: jdk8/build/jaxp: Added tag jdk8-b64 for changeset 27ab79568c34

2012-11-14 Thread david . katleman
Changeset: 5cf3c69a93d6 Author:katleman Date: 2012-11-08 11:51 -0800 URL: http://hg.openjdk.java.net/jdk8/build/jaxp/rev/5cf3c69a93d6 Added tag jdk8-b64 for changeset 27ab79568c34 ! .hgtags

hg: jdk8/build/hotspot: 18 new changesets

2012-11-14 Thread david . katleman
Changeset: 49bc14aaadcc Author:katleman Date: 2012-11-08 11:51 -0800 URL: http://hg.openjdk.java.net/jdk8/build/hotspot/rev/49bc14aaadcc Added tag jdk8-b64 for changeset 5920f72e799c ! .hgtags Changeset: ca8168203393 Author:amurillo Date: 2012-11-02 07:44 -0700 URL:

hg: jdk8/build: 2 new changesets

2012-11-14 Thread david . katleman
Changeset: 838a64965131 Author:katleman Date: 2012-11-08 11:50 -0800 URL: http://hg.openjdk.java.net/jdk8/build/rev/838a64965131 Added tag jdk8-b64 for changeset 1c8370a55b30 ! .hgtags Changeset: b772de306dc2 Author:katleman Date: 2012-11-14 12:28 -0800 URL: http://

hg: jdk8/build/corba: Added tag jdk8-b64 for changeset 54d599a5b4aa

2012-11-14 Thread david . katleman
Changeset: 5132f7900a8f Author:katleman Date: 2012-11-08 11:50 -0800 URL: http://hg.openjdk.java.net/jdk8/build/corba/rev/5132f7900a8f Added tag jdk8-b64 for changeset 54d599a5b4aa ! .hgtags

Re: Review Request: 8003274: build-infra: Makefile changes needed for sjavac

2012-11-14 Thread Kelly O'Hair
On Nov 14, 2012, at 2:06 AM, Fredrik Öhrström wrote: > 12 nov 2012 kl. 19:04 skrev Kelly O'Hair: >> Or maybe >> if [ "$(words $(shell cat file))" -ne 0 ] ; then >> > > Well, this one is better. I'll switch to this one, eventually. No hurry on my account. -kto > > //Fredrik >

Re: Review Request: 8001906: build-infra: warning: [path] bad path element on Solaris

2012-11-14 Thread Kelly O'Hair
Looks ok. -kto On Nov 14, 2012, at 7:03 AM, Erik Joelsson wrote: > Simple fix for compiler warning on solaris: > > http://cr.openjdk.java.net/~erikj/8001906/webrev.jdk.01/ > > /Erik

Re: Review Request: 8003327: build-infra: "/bin/sh: : cannot execute" on solaris

2012-11-14 Thread Kelly O'Hair
Looks good. -kto On Nov 14, 2012, at 6:25 AM, Erik Joelsson wrote: > This minimal patch fixes a problem with error messages from /bin/sh on > solaris. > > http://cr.openjdk.java.net/~erikj/8003327/webrev.root.01/ > > /Erik

Re: Review Request: 8003317: build-infra: Configure fails when current dir is part of a symlink

2012-11-14 Thread Kelly O'Hair
Looks ok. I don't think this fixes my /net/`hostname`[.domainname]/ prefix problems on Solaris 11. But your changes are needed, and the /net/ prefix fix can wait. -kto On Nov 14, 2012, at 6:04 AM, Erik Joelsson wrote: > This is a fix for the logic that prevents configure from creating > confi

Re: Review Request: 8001941: build-infra: --disable-precompiled-headers does not seem to work

2012-11-14 Thread Kelly O'Hair
Looks ok to me. -kto On Nov 14, 2012, at 5:21 AM, Erik Joelsson wrote: > This is a fix for configure parameter --disable-precompiled-headers. It has > now been verified to work. > > http://cr.openjdk.java.net/~erikj/8001941/webrev.root.01/ >

Re: Review Request: 8001875: build-infra: We must be able to force static linking of stdc++

2012-11-14 Thread Kelly O'Hair
Looks ok to me... although I confess I'm going to need to brush up on my m4. ;^) -kto On Nov 14, 2012, at 3:38 AM, Erik Joelsson wrote: > http://cr.openjdk.java.net/~erikj/8001875/webrev.root.01/ > > > This patch changes the conf

Re: invalid bug IDs aborting my pull

2012-11-14 Thread Tim Bell
In the incoming changesets those are the only two with IDs above 80. Off the top of my head (I'm on a concall at the moment)- Do you have the latest version of jcheck? http://hg.openjdk.java.net/code-tools/jcheck Specifically, this changeset to "Allow bugids of the form 8xx"? http:

Re: invalid bug IDs aborting my pull

2012-11-14 Thread mark . reinhold
2012/11/14 8:17 -0800, peter.bru...@oracle.com: > I looked in jcheck.py and see: > > if not (bs[0] in ['1','2','4','5','6','7']): > ch.error(ctx, "Invalid bugid: %s" % bs) > > so either I don't have the latest or '8' needs to be added. I'll try that. You have an antique copy of jcheck.

Re: invalid bug IDs aborting my pull

2012-11-14 Thread Pete Brunet
I looked in jcheck.py and see: if not (bs[0] in ['1','2','4','5','6','7']): ch.error(ctx, "Invalid bugid: %s" % bs) so either I don't have the latest or '8' needs to be added. I'll try that. Pete On 11/14/12 10:12 AM, Chris Hegarty wrote: > On 11/14/2012 04:07 PM, Pete Brunet wrote: >

Re: invalid bug IDs aborting my pull

2012-11-14 Thread Chris Hegarty
On 11/14/2012 04:07 PM, Pete Brunet wrote: In the incoming changesets those are the only two with IDs above 80. Is it possible that you have an old version of jcheck on your client? -Chris. Also I forgot to indicate that the repo is hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk Pete On 11/1

Re: invalid bug IDs aborting my pull

2012-11-14 Thread Pete Brunet
In the incoming changesets those are the only two with IDs above 80. Also I forgot to indicate that the repo is hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk Pete On 11/14/12 9:53 AM, Pete Brunet wrote: > Hi I pulled this morning and jcheck aborted my pull -u because of > invalid bug IDs for > >

Re: invalid bug IDs aborting my pull

2012-11-14 Thread Dmitry Samersoff
Peter, I guess you have to clone fresh copy rather than pull - someone recreate server workspace. -Dmitry On 2012-11-14 19:53, Pete Brunet wrote: > Hi I pulled this morning and jcheck aborted my pull -u because of > invalid bug IDs for > > changeset 5413: bugid 8002040, author dholmes > changes

invalid bug IDs aborting my pull

2012-11-14 Thread Pete Brunet
Hi I pulled this morning and jcheck aborted my pull -u because of invalid bug IDs for changeset 5413: bugid 8002040, author dholmes changeset 5490: bugid 8000989, author twisti I was able to pull -u over the last week or two. Am I doing something wrong? Pete

Re: Review Request: 8002026: build-infra: deploy repository building

2012-11-14 Thread Tim Bell
On 11/14/12 01:22, Erik Joelsson wrote: Yes, it's intended. What I really would like to express is (in pseudo code): if (OS=windows AND BIN_FILE in ACCEPTED_SIZE_DIFF AND (DIFF_SIZE_NUM = 512 OR DIFF_SIZE_NUM = -512)) I wasn't sure I could write this in bash, but maybe I'm wrong. I figured th

Review Request: 8001906: build-infra: warning: [path] bad path element on Solaris

2012-11-14 Thread Erik Joelsson
Simple fix for compiler warning on solaris: http://cr.openjdk.java.net/~erikj/8001906/webrev.jdk.01/ /Erik

Review Request: 8003327: build-infra: "/bin/sh: : cannot execute" on solaris

2012-11-14 Thread Erik Joelsson
This minimal patch fixes a problem with error messages from /bin/sh on solaris. http://cr.openjdk.java.net/~erikj/8003327/webrev.root.01/ /Erik

Review Request: 8003317: build-infra: Configure fails when current dir is part of a symlink

2012-11-14 Thread Erik Joelsson
This is a fix for the logic that prevents configure from creating configurations in undesired directories. If the path to the source root contains a symlink, the check got fooled and refused to create a configuration anywhere. This is currently happening in REs nightly builds of build-infra for

Review Request: 8001941: build-infra: --disable-precompiled-headers does not seem to work

2012-11-14 Thread Erik Joelsson
This is a fix for configure parameter --disable-precompiled-headers. It has now been verified to work. http://cr.openjdk.java.net/~erikj/8001941/webrev.root.01/ Remember to regenerate configure after applying the patch and to also

Review Request: 8001875: build-infra: We must be able to force static linking of stdc++

2012-11-14 Thread Erik Joelsson
http://cr.openjdk.java.net/~erikj/8001875/webrev.root.01/ This patch changes the configure paramater --disable-static-link-stdc++ to the following: --with-stdc++lib=,, * default: try static if it works but falling back on dynamic *

Re: Review Request: 8003274: build-infra: Makefile changes needed for sjavac

2012-11-14 Thread Fredrik Öhrström
12 nov 2012 kl. 19:04 skrev Kelly O'Hair: > Or maybe >if [ "$(words $(shell cat file))" -ne 0 ] ; then > Well, this one is better. I'll switch to this one, eventually. //Fredrik

Re: Review Request: 8002026: build-infra: deploy repository building

2012-11-14 Thread Erik Joelsson
On 2012-11-13 23:06, Tim Bell wrote: Hi Erik: This is the open part of this change. It only changes the compare script to handle the deploy build artifacts. http://cr.openjdk.java.net/~erikj/8002026/webrev.root.01/ I was getti

hg: jdk8/build/jdk: 8001965: build-infra: Large compare diffs between new and old on mac

2012-11-14 Thread tim . bell
Changeset: 84f0439ccaab Author:tbell Date: 2012-11-13 13:46 -0800 URL: http://hg.openjdk.java.net/jdk8/build/jdk/rev/84f0439ccaab 8001965: build-infra: Large compare diffs between new and old on mac Summary: The wrong icon source file was used when building closed Reviewed-by: ohair

hg: jdk8/build: 8003274: build-infra: Makefile changes needed for sjavac

2012-11-14 Thread tim . bell
Changeset: f2ac4d0edaae Author:tbell Date: 2012-11-13 15:54 -0800 URL: http://hg.openjdk.java.net/jdk8/build/rev/f2ac4d0edaae 8003274: build-infra: Makefile changes needed for sjavac Summary: changes left in build-infra that are related to sjavac Reviewed-by: ohair, tbell Contribute