On 7/10/2013 12:37 PM, Tim Bell wrote:
Hi Kelly
How did BUILD_INSTALL ever become true on an OpenJDK build?
There is only one place that the build itself sets BUILD_INSTALL to true
and that's upon detection of install/make/Makefile, in
make/Defs-internal.gmk:
If you have a full forest (or heck, just install/make/Makefile)
BUILD_INSTALL will be set to true
INSTALL_SRC_AVAILABLE := $(call MkExists,$(INSTALL_TOPDIR)/make/Makefile)
ifndef BUILD_INSTALL
ifdef DEV_ONLY
BUILD_INSTALL := false
else
BUILD_INSTALL := $(INSTALL_SRC_AVAILABLE)
endif
endif
Anything else BUILD_INSTALL=true being passed in at the start of the
build, by a script or user.
Is there a log of the build with this problem?
Thanks
Dave
From the 'generic_debug_build' rule in the top level 7update Makefile [1]
generic_debug_build:
$(MAKE) \
ALT_OUTPUTDIR=$(ABS_OUTPUTDIR)/$(REL_JDK_OUTPUTDIR) \
DEBUG_NAME=$(DEBUG_NAME) \
GENERATE_DOCS=false \
BUILD_INSTALL_BUNDLES=true \
CREATE_DEBUGINFO_BUNDLES=false \
$(BOOT_CYCLE_DEBUG_SETTINGS) \
generic_build_repo_series
I will be reopening 8012366 and putting it forward again for approval.
Tim
[1] http://hg.openjdk.java.net/jdk7u/jdk7u/file/tip/Makefile
-kto
On Jul 8, 2013, at 12:03 PM, Tim Bell wrote:
All-
My fix for 8007815 [1] did not handle the case of an OpenJDK build
that does not have the closed source trees. If a user tries to
build that far, the build will fail with "cd: can't cd to
./install/make/installer/binaries/linux"
This is a request for review and approval to fix 8012366 in 7u.
These changes will not throw a build error if the closed directories
do not exist. Makefile changes only; no product code has been
modified:
Link to the bug report:
http://bugs.sun.com/view_bug.do?bug_id=8012366
webrev of the Makefile changes:
http://cr.openjdk.java.net/~tbell/8012366/webrev.00/
Thanks in advance-
Tim
[1] There is no external link for 8007815, sorry. These changes
had to do with building and packaging in the install tree.
