From 2adec06e0cb5542a72e1f9634fc1185514d60166 Mon Sep 17 00:00:00 2001 From: dcubed Date: Tue, 3 Apr 2012 10:17:06 -0700 Subject: 7157296: FDS: ENABLE_FULL_DEBUG_SYMBOLS flag should only affect OPT builds Summary: Build option FULL_DEBUG_SYMBOLS=0 only affects OPT builds. Finish enabling ENABLE_FULL_DEBUG_SYMBOLS flag on Windows. Reviewed-by: ohair, jmelvin, sspitsyn --- make/Defs-internal.gmk | 11 ++++++++++- make/hotspot-rules.gmk | 14 +++++++++++++- make/jdk-rules.gmk | 12 +++++++++++- 3 files changed, 34 insertions(+), 3 deletions(-) (limited to 'make') diff --git a/make/Defs-internal.gmk b/make/Defs-internal.gmk index bc3f32b..6afea70 100644 --- a/make/Defs-internal.gmk +++ b/make/Defs-internal.gmk @@ -1,5 +1,5 @@ # -# Copyright (c) 1995, 2011, Oracle and/or its affiliates. All rights reserved. +# Copyright (c) 1995, 2012, Oracle and/or its affiliates. All rights reserved. # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. # # This code is free software; you can redistribute it and/or modify it @@ -354,3 +354,12 @@ ifdef ANT_HOME COMMON_BUILD_ARGUMENTS += ANT_HOME="$(ANT_HOME)" endif +# When all repos support FULL_DEBUG_SYMBOLS and ZIP_DEBUGINFO_FILES, +# then these can be set here: +#ifdef FULL_DEBUG_SYMBOLS +# COMMON_BUILD_ARGUMENTS += FULL_DEBUG_SYMBOLS=$(FULL_DEBUG_SYMBOLS) +#endif +# +#ifdef ZIP_DEBUGINFO_FILES +# COMMON_BUILD_ARGUMENTS += ZIP_DEBUGINFO_FILES="$(ZIP_DEBUGINFO_FILES)" +#endif diff --git a/make/hotspot-rules.gmk b/make/hotspot-rules.gmk index 437f05d..4271210 100644 --- a/make/hotspot-rules.gmk +++ b/make/hotspot-rules.gmk @@ -1,5 +1,5 @@ # -# Copyright (c) 2001, 2011, Oracle and/or its affiliates. All rights reserved. +# Copyright (c) 2001, 2012, Oracle and/or its affiliates. All rights reserved. # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. # # This code is free software; you can redistribute it and/or modify it @@ -71,6 +71,7 @@ endif ifeq ($(DEBUG_NAME), fastdebug) HOTSPOT_TARGET = all_fastdebug endif +BUILD_FLAVOR=$(HOTSPOT_TARGET:all_%=%) ifeq ($(ZERO_BUILD), true) ifeq ($(SHARK_BUILD), true) @@ -83,6 +84,7 @@ endif HOTSPOT_BUILD_ARGUMENTS += $(COMMON_BUILD_ARGUMENTS) HOTSPOT_BUILD_ARGUMENTS += ALT_OUTPUTDIR=$(HOTSPOT_OUTPUTDIR) HOTSPOT_BUILD_ARGUMENTS += ALT_EXPORT_PATH=$(HOTSPOT_EXPORT_PATH) +HOTSPOT_BUILD_ARGUMENTS += BUILD_FLAVOR=$(BUILD_FLAVOR) # Why do these need to be passed in? Because of windows nmake? and MAKEFLAGS=? # Or is there something wrong with hotspot/make/Makefile? @@ -93,6 +95,16 @@ ifeq ($(BUILD_LANGTOOLS), true) HOTSPOT_BUILD_ARGUMENTS += ALT_LANGTOOLS_DIST=$(ABS_LANGTOOLS_DIST) endif +# Move to COMMON_BUILD_ARGUMENTS when all repos support +# FULL_DEBUG_SYMBOLS and ZIP_DEBUGINFO_FILES: +ifdef FULL_DEBUG_SYMBOLS + HOTSPOT_BUILD_ARGUMENTS += FULL_DEBUG_SYMBOLS=$(FULL_DEBUG_SYMBOLS) +endif + +ifdef ZIP_DEBUGINFO_FILES + HOTSPOT_BUILD_ARGUMENTS += ZIP_DEBUGINFO_FILES="$(ZIP_DEBUGINFO_FILES)" +endif + hotspot-build:: $(MKDIR) -p $(HOTSPOT_OUTPUTDIR) $(MKDIR) -p $(HOTSPOT_EXPORT_PATH) diff --git a/make/jdk-rules.gmk b/make/jdk-rules.gmk index 35d7890..42ba6ad 100644 --- a/make/jdk-rules.gmk +++ b/make/jdk-rules.gmk @@ -1,5 +1,5 @@ # -# Copyright (c) 2001, 2011, Oracle and/or its affiliates. All rights reserved. +# Copyright (c) 2001, 2012, Oracle and/or its affiliates. All rights reserved. # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. # # This code is free software; you can redistribute it and/or modify it @@ -70,6 +70,16 @@ endif JDK_BUILD_ARGUMENTS += \ BUILD_HOTSPOT=$(BUILD_HOTSPOT) +# Move to COMMON_BUILD_ARGUMENTS when all repos support +# FULL_DEBUG_SYMBOLS and ZIP_DEBUGINFO_FILES: +ifdef FULL_DEBUG_SYMBOLS + JDK_BUILD_ARGUMENTS += FULL_DEBUG_SYMBOLS=$(FULL_DEBUG_SYMBOLS) +endif + +ifdef ZIP_DEBUGINFO_FILES + JDK_BUILD_ARGUMENTS += ZIP_DEBUGINFO_FILES="$(ZIP_DEBUGINFO_FILES)" +endif + $(JDK_JAVA_EXE):: jdk-build -- cgit v1.2.3