Mercurial check out for a particular JDK8 build
Hello, I have a question regarding mercurial checkout steps. What is the right way to check out source for the master and the nested repos (jaxp, langtools ..etc) for a particular JDK 8 build from http://hg.openjdk.java.net/jdk8/jdk8/ ? I didn't find much luck with hg clone and hg update for particular revision/changeset number (705 in case of JDK8-b89?). Probably I need to pass the right arguments to get_source.sh script but I am not sure what they should be. Thanks for your help. -Shrinivas
Re: RFR: 8010785: JDK 8 build on Linux fails with new build mechanism
Hi Erik,
In CreateJars.gmk I don't understand why you move the update to the JARS
variable inside the BUILD_CRYPTO conditional when the jar file is a
pre-req for a target defined outside of that conditional. What are the
allowed combinations:
BUILD_CRYPTO=yes, OPENJDK=true == OK (normal OpenJDK build)
BUILD_CRYPTO=yes, OPENJDK=false == OK? (builds but doesn't use it?)
BUILD_CRYPTO=no, OPENJDK=true == ILLEGAL? (missing re-req in rule?)
BUILD_CRYPTO=no, OPENJDK=false == OK (normal Oracle JDK build)
This also seems to indicate that the earlier comment block:
469
##
470 # For all security jars, always build the jar, but for closed,
install the prebuilt signed
471 # version instead of the newly built jar. Unsigned jars are
treated as intermediate targets
472 # and explicitly added to the JARS list. For open, signing is not
needed. See SignJars.gmk
473 # for more information.
needs updating to account for this new condition. ("security" covers
these crypto jars).
---
In Setup.gmk, wouldn't this:
38 ifndef OPENJDK
39 # Some licensees do not get the Security Source bundles. We will
40 # fall back on the prebuilt jce.jar so that we can do a best
41 # attempt at building.
42 ifeq ($(wildcard
$(JDK_TOPDIR)/src/share/classes/javax/crypto/Cipher.java),)
43 JCE_PATH :=
$(PATH_SEP)$(JDK_TOPDIR)/make/closed/tools/crypto/jce/jce.jar
44 endif
45 endif
be better handled by a configure check that the sources exist - as is
done for other potentially not-present components? Further I think this
kind of check belongs in a closed build file as it doesn't relate to
building the openjdk sources.
Thanks,
David
On 31/05/2013 8:14 PM, Erik Joelsson wrote:
Finally getting back to this. Updated webrevs:
http://cr.openjdk.java.net/~erikj/8010785/webrev.jdk.02/
http://cr.openjdk.java.net/~erikj/8010785/webrev.root.02/
The javascript part is no longer needed since it has been removed.
/Erik
On 2013-04-11 12:53, Erik Joelsson wrote:
Open part of this review.
The licensee bundles aren't buildable with the new build for several
reasons. I've tried to fix all the issues that I've found and have now
successfully built them on linux, windows and solaris. Here is a list
of the changes that I had to do to OpenJDK:
* Filter out javascript src when the rhino source isn't available.
Also do not copy rhino resource files when not available. This is
controlled by a new variable, INCLUDE_JAVASCRIPT, which we control
from closed configure and shouldn't affect the OpenJDK build. I also
moved the copying of the resources to the correct makefile,
CopyIntoClasses.gmk.
* If javax/crypto isn't available, jce.jar needs to be added to the
bootclasspath of the main java compilation. Also, a number of security
jar files shouldn't be built at all. (Normally these are built just to
exercise the logic, but not used.) The kerberos library is also
excluded by this. Introduced the variable BUILD_CRYPTO, also set by
closed configure to control this. I used the logic ifneq
($(BUILD_CRYPTO),no) to not change the behavior if the variable isn't
set, which it won't be in the open case.
* I removed the logic for setting the closed cacerts file in the open
configure script.
* Also fixing JDK-8005655 by adding logic for unzipping sec-bin (and
friends) if available.
http://cr.openjdk.java.net/~erikj/8010785/webrev.jdk.01/
http://cr.openjdk.java.net/~erikj/8010785/webrev.root.01/
/Erik
Re: Mercurial check out for a particular JDK8 build
On 3/06/2013 6:16 AM, Shrinivas Joshi wrote: Hello, I have a question regarding mercurial checkout steps. What is the right way to check out source for the master and the nested repos (jaxp, langtools ..etc) for a particular JDK 8 build from http://hg.openjdk.java.net/jdk8/jdk8/ ? I didn't find much luck with hg clone and hg update for particular revision/changeset number (705 in case of JDK8-b89?). Probably I need to pass the right arguments to get_source.sh script but I am not sure what they should be. I don't think get_source.sh can aid you here. You need to specify a build tag on the -r option of clone, and using the forest extension to do an fclone is probably simplest eg: > hg fclone -r jdk8-b42 http://hg.openjdk.java.net/jdk8/jdk8 jdk8-b42 David - Thanks for your help. -Shrinivas
Re: RFR: 8015759: hotspot changes needed to compile with Visual Studio 2012
Hi Tim, In compile.make: Please update copyright end year to 2013 (hotspot-runtime process rule). I'm wondering what the impact is of not being able to use _STATIC_CPPLIB? Given it has been deprecated for a while was it actually a no-op? We need to understand what impact this has, if any, on the built bits. --- In sanity.make: Please update copyright end year to 2013 --- In vm.make this addition seems unrelated to a compiler change: !ifdef RELEASE + !ifdef DEVELOP + CXX_FLAGS=$(CXX_FLAGS) /D "DEBUG" + !else CXX_FLAGS=$(CXX_FLAGS) /D "PRODUCT" + !endif !else CXX_FLAGS=$(CXX_FLAGS) /D "ASSERT" !endif What is a RELEASE+DEVELOP build? Further I would expect that any build that sets DEBUG should also set ASSERT ?? --- In unwind_windows_x86.hpp Please update copyright end year to 2013 --- I'd be happy to sponsor the push into hotspot-rt once this is finalized. Just create the changeset and send me a link to it. Thanks, David On 2/06/2013 3:02 AM, Tim Bell wrote: 8015759: hotspot changes needed to compile with Visual Studio 2012 The bug report will be visible in a few days here: http://bugs.sun.com/view_bug.do?bug_id=8015759 Minor Makefile changes and one source file was touched to compile with Visual Studio 2012. These changes also compile under Visual Studio 2010 with no problem so we can upgrade gracefully. Tested by building the forest with both compilers. % hg status M make/windows/makefiles/compile.make M make/windows/makefiles/sanity.make M make/windows/makefiles/vm.make M src/os_cpu/windows_x86/vm/unwind_windows_x86.hpp webrev and patch file are here: http://cr.openjdk.java.net/~tbell/8015759/hotspot/webrev.00/ http://cr.openjdk.java.net/~tbell/8015759/hotspot/webrev.00/hotspot.patch Thanks to Anthony Petrov who provided the initial set of patches for this work. I believe this should go in via the hotspot-rt forest (let me know if that is not correct), in which case I will need a sponsor from that team to push the change. Thanks in advance- Tim
