RFR JDK-8010267 & JDK-8010268 : Makefile maintenance for test targets

2013-03-18 Thread Mike Duigou
A two small changes to review: If approved I will commit to TL (or someone else can commit to build for me) Mike JDK-8010267 : Add test-clean for cleaning of testoutput directory from output directory. diff --git a/common/makefiles/Main.gmk b/common/makefiles/Main.gmk --- a/common/makefiles/M

Re: RFR 8009517: Disable fatal compiler warning in the old build

2013-03-18 Thread Brad Wetmore
Sorry for the delay in response, I've been pulled in yet another direction, and this has come back up in priority. On 3/9/2013 12:11 AM, Chris Hegarty wrote: I agree about warning creeping problems. This is a temporary solution, we should soon be fixing the underlying hashcode/equals problems.

Fix x11 header check (was: Re: New official README-builds.html)

2013-03-18 Thread Omair Majid
On 03/15/2013 03:55 PM, Omair Majid wrote: > In file included from > /builddir/build/BUILD/java-1.8.0-openjdk/jdk8/build/jdk8.build/jdk/gensrc_x11wrappers/sizer.64.c:11:0: > /builddir/build/BUILD/java-1.8.0-openjdk/jdk8/jdk/src/solaris/native/sun/awt/awt_p.h:51:36: > fatal error: X11/extensions/Xre

Re: New official README-builds.html

2013-03-18 Thread Andrew Hughes
- Original Message - > 2013/3/18 Andrew Hughes : > > > > > Depends how many distros you intend to support. > > Well, I guess that if somebody doesn't use one between > rhel/fedora/suse/ubuntu/mint/debian than most likely will figure out > the exact commands anyway, I think those are still

RFR: 8009426 "profiles" target fails due to nashorn if "images" is not built first

2013-03-18 Thread David Holmes
Very simple fix/cleanup. webrev: http://cr.openjdk.java.net/~dholmes/8009426/webrev/ nashorn.jar was being added to the set of JARS targets regardless of what was being built, but nashorn is not a dependency for profiles target. JARS is not used the way it used to be now we have profiles and

Re: Empty jdk8/build/nashorn repository

2013-03-18 Thread David Katleman
On 3/18/2013 10:54 AM, Andrew Hughes wrote: - Original Message - Hi, The repository at http://hg.openjdk.java.net/jdk8/build/nashorn is empty with no changesets. Now the build fails (on 'make all') with an error: ## Starting nashorn /bin/sh: line 0: cd: /home/omajid/devel/jdk8-build/n

Re: Empty jdk8/build/nashorn repository

2013-03-18 Thread Andrew Hughes
- Original Message - > Hi, > > The repository at http://hg.openjdk.java.net/jdk8/build/nashorn is > empty > with no changesets. Now the build fails (on 'make all') with an > error: > > ## Starting nashorn > /bin/sh: line 0: cd: /home/omajid/devel/jdk8-build/nashorn/makefiles: > No > such

Re: New official README-builds.html

2013-03-18 Thread Mario Torre
2013/3/18 Andrew Hughes : > > Depends how many distros you intend to support. Well, I guess that if somebody doesn't use one between rhel/fedora/suse/ubuntu/mint/debian than most likely will figure out the exact commands anyway, I think those are still good suggestions to keep around. Cheers, Ma

Re: New official README-builds.html

2013-03-18 Thread Andrew Hughes
- Original Message - > 2013/3/15 Andrew Hughes : > >> Might want to update configure to check for these libs on linux. > >> > > > > Well that won't work everywhere: > > > > $ apt-get install libx11-dev libxext-dev libxt-dev libxrender-dev > > bash: apt-get: command not found > > > > so th

Re: New official README-builds.html

2013-03-18 Thread Andrew Hughes
- Original Message - > On 03/15/2013 02:05 PM, Andrew Hughes wrote: > >> Might want to update configure to check for these libs on linux. > >> > > > > Well that won't work everywhere: > > > > $ apt-get install libx11-dev libxext-dev libxt-dev libxrender-dev > > bash: apt-get: command no

Empty jdk8/build/nashorn repository

2013-03-18 Thread Omair Majid
Hi, The repository at http://hg.openjdk.java.net/jdk8/build/nashorn is empty with no changesets. Now the build fails (on 'make all') with an error: ## Starting nashorn /bin/sh: line 0: cd: /home/omajid/devel/jdk8-build/nashorn/makefiles: No such file or directory make: *** [nashorn-only] Error 1

hg: jdk8/build/jdk: 8010030: Allow configure to detect if EC implementation is present

2013-03-18 Thread omajid
Changeset: 624bcb480006 Author:omajid Date: 2013-03-18 10:46 -0400 URL: http://hg.openjdk.java.net/jdk8/build/jdk/rev/624bcb480006 8010030: Allow configure to detect if EC implementation is present Reviewed-by: andrew, dholmes ! makefiles/CompileNativeLibraries.gmk

Re: Allow configure to detect if EC implementation is present

2013-03-18 Thread Omair Majid
Hi, On 03/15/2013 07:15 AM, David Holmes wrote: > On 15/03/2013 7:04 PM, Erik Joelsson wrote: >> And as David pointed out, please notify me and Tim Bell when you push > > If I am added to that list we will have global timezone coverage :) > Though we should all see the push notifications anyway,

hg: jdk8/build: 8010030: Allow configure to detect if EC implementation is present

2013-03-18 Thread omajid
Changeset: e2057191f6da Author:omajid Date: 2013-03-18 10:47 -0400 URL: http://hg.openjdk.java.net/jdk8/build/rev/e2057191f6da 8010030: Allow configure to detect if EC implementation is present Reviewed-by: andrew, dholmes ! common/autoconf/configure.ac ! common/autoconf/generated-

Re: Code review request: 8010192: Enable native JGSS provider on Mac

2013-03-18 Thread Erik Joelsson
The build changes look ok. /Erik On 2013-03-18 12:00, Weijun Wang wrote: Please take a look at http://cr.openjdk.java.net/~weijun/8010192/webrev.00/ The Mac native JGSS provider is back. Besides reverting changes made in http://hg.openjdk.java.net/jdk8/tl/jdk/rev/5a2ab2c3f106, there are:

Code review request: 8010192: Enable native JGSS provider on Mac

2013-03-18 Thread Weijun Wang
Please take a look at http://cr.openjdk.java.net/~weijun/8010192/webrev.00/ The Mac native JGSS provider is back. Besides reverting changes made in http://hg.openjdk.java.net/jdk8/tl/jdk/rev/5a2ab2c3f106, there are: 1. GSSManagerImpl.java: enable the provider 2. SunNativeProvider.java: lib

Re: Building on Mac

2013-03-18 Thread Daniel Fuchs
Hi Peter, Did you 'install the command line tools' in Xcode? The fact that you have to refer to /Applications/Xcode.app/... let me think that perhaps you didn't. Start Xcode, go to preferences, and explore the different panes: there's one that will let you install the command line tools. If

Re: Building on Mac (David DeHaven)

2013-03-18 Thread Erik Joelsson
A few comments inline. On 2013-03-16 21:38, Mani Sarkar wrote: Hi, The OpenJDK adopters in the community have been testing out the steps lately to build OpenJDK on the MacOS and so far things are fine, we didn't need to install MacPort or have major issues with the Mac. You can have a look at