8273797: Stop impersonating "server" VM in all VM variants
Reviewed-by: dholmes, erikj, ihse
This commit is contained in:
parent
240fa6efa2
commit
f242cb5ce0
@ -32,16 +32,10 @@ AC_DEFUN([FLAGS_SETUP_LDFLAGS],
|
|||||||
|
|
||||||
# Setup the target toolchain
|
# Setup the target toolchain
|
||||||
|
|
||||||
# On some platforms (mac) the linker warns about non existing -L dirs.
|
# The target dir matches the name of VM variant
|
||||||
# For any of the variants server, client, minimal or zero, the dir matches the
|
|
||||||
# variant name. The "main" variant should be used for linking. For the
|
|
||||||
# rest, the dir is just server.
|
|
||||||
if HOTSPOT_CHECK_JVM_VARIANT(server) || HOTSPOT_CHECK_JVM_VARIANT(client) \
|
|
||||||
|| HOTSPOT_CHECK_JVM_VARIANT(minimal) || HOTSPOT_CHECK_JVM_VARIANT(zero); then
|
|
||||||
TARGET_JVM_VARIANT_PATH=$JVM_VARIANT_MAIN
|
TARGET_JVM_VARIANT_PATH=$JVM_VARIANT_MAIN
|
||||||
else
|
|
||||||
TARGET_JVM_VARIANT_PATH=server
|
# On some platforms (mac) the linker warns about non existing -L dirs.
|
||||||
fi
|
|
||||||
FLAGS_SETUP_LDFLAGS_CPU_DEP([TARGET])
|
FLAGS_SETUP_LDFLAGS_CPU_DEP([TARGET])
|
||||||
|
|
||||||
# Setup the build toolchain
|
# Setup the build toolchain
|
||||||
|
@ -83,15 +83,6 @@ AC_DEFUN_ONCE([HOTSPOT_SETUP_JVM_VARIANTS],
|
|||||||
AC_MSG_ERROR([Cannot continue])
|
AC_MSG_ERROR([Cannot continue])
|
||||||
fi
|
fi
|
||||||
|
|
||||||
# All "special" variants share the same output directory ("server")
|
|
||||||
VALID_MULTIPLE_JVM_VARIANTS="server client minimal zero"
|
|
||||||
UTIL_GET_NON_MATCHING_VALUES(INVALID_MULTIPLE_VARIANTS, $JVM_VARIANTS, \
|
|
||||||
$VALID_MULTIPLE_JVM_VARIANTS)
|
|
||||||
if test "x$INVALID_MULTIPLE_VARIANTS" != x && \
|
|
||||||
test "x$BUILDING_MULTIPLE_JVM_VARIANTS" = xtrue; then
|
|
||||||
AC_MSG_ERROR([You can only build multiple variants using these variants: '$VALID_MULTIPLE_JVM_VARIANTS'])
|
|
||||||
fi
|
|
||||||
|
|
||||||
# The "main" variant is the one used by other libs to link against during the
|
# The "main" variant is the one used by other libs to link against during the
|
||||||
# build.
|
# build.
|
||||||
if test "x$BUILDING_MULTIPLE_JVM_VARIANTS" = "xtrue"; then
|
if test "x$BUILDING_MULTIPLE_JVM_VARIANTS" = "xtrue"; then
|
||||||
|
@ -34,13 +34,7 @@ JVM_SUPPORT_DIR := $(JVM_VARIANT_OUTPUTDIR)/support
|
|||||||
DTRACE_SUPPORT_DIR := $(JVM_SUPPORT_DIR)/dtrace
|
DTRACE_SUPPORT_DIR := $(JVM_SUPPORT_DIR)/dtrace
|
||||||
|
|
||||||
LIB_OUTPUTDIR := $(call FindLibDirForModule, java.base)
|
LIB_OUTPUTDIR := $(call FindLibDirForModule, java.base)
|
||||||
ifneq ($(filter client minimal zero, $(JVM_VARIANT)), )
|
JVM_LIB_OUTPUTDIR := $(LIB_OUTPUTDIR)/$(JVM_VARIANT)
|
||||||
JVM_VARIANT_SUBDIR := $(JVM_VARIANT)
|
|
||||||
else
|
|
||||||
# Use 'server' as default target directory name for all other variants.
|
|
||||||
JVM_VARIANT_SUBDIR := server
|
|
||||||
endif
|
|
||||||
JVM_LIB_OUTPUTDIR := $(LIB_OUTPUTDIR)/$(JVM_VARIANT_SUBDIR)
|
|
||||||
|
|
||||||
################################################################################
|
################################################################################
|
||||||
|
|
||||||
|
@ -95,16 +95,10 @@ ifeq ($(call And, $(call isTargetOs, windows) $(call isTargetCpu, x86)), true)
|
|||||||
endif
|
endif
|
||||||
DEFAULT_CFG_VARIANT ?= server
|
DEFAULT_CFG_VARIANT ?= server
|
||||||
|
|
||||||
# Any variant other than server, client, minimal, or zero is represented as server in
|
|
||||||
# the cfg file.
|
|
||||||
VALID_CFG_VARIANTS := server client minimal zero
|
|
||||||
CFG_VARIANTS := $(filter $(VALID_CFG_VARIANTS), $(JVM_VARIANTS)) \
|
|
||||||
$(if $(filter-out $(VALID_CFG_VARIANTS), $(JVM_VARIANTS)), server)
|
|
||||||
|
|
||||||
# Change the order to put the default variant first if present.
|
# Change the order to put the default variant first if present.
|
||||||
ORDERED_CFG_VARIANTS := \
|
ORDERED_CFG_VARIANTS := \
|
||||||
$(if $(filter $(DEFAULT_CFG_VARIANT), $(CFG_VARIANTS)), $(DEFAULT_CFG_VARIANT)) \
|
$(if $(filter $(DEFAULT_CFG_VARIANT), $(JVM_VARIANTS)), $(DEFAULT_CFG_VARIANT)) \
|
||||||
$(filter-out $(DEFAULT_CFG_VARIANT), $(CFG_VARIANTS))
|
$(filter-out $(DEFAULT_CFG_VARIANT), $(JVM_VARIANTS))
|
||||||
|
|
||||||
JVMCFG := $(LIB_DST_DIR)/jvm.cfg
|
JVMCFG := $(LIB_DST_DIR)/jvm.cfg
|
||||||
|
|
||||||
|
@ -156,11 +156,8 @@ ifeq ($(call isTargetOsType, unix), true)
|
|||||||
TARGETS += $(LIB_OUTPUTDIR)/$1/$(call SHARED_LIBRARY,jsig)
|
TARGETS += $(LIB_OUTPUTDIR)/$1/$(call SHARED_LIBRARY,jsig)
|
||||||
endef
|
endef
|
||||||
|
|
||||||
# The subdir is the same as the variant for client, minimal or zero, for all
|
# The subdir is the same as the variant
|
||||||
# others it's server.
|
$(foreach v, $(JVM_VARIANTS), $(eval $(call CreateSymlinks,$v)))
|
||||||
VARIANT_SUBDIRS := $(filter client minimal zero, $(JVM_VARIANTS)) \
|
|
||||||
$(if $(filter-out client minimal zero, $(JVM_VARIANTS)), server)
|
|
||||||
$(foreach v, $(VARIANT_SUBDIRS), $(eval $(call CreateSymlinks,$v)))
|
|
||||||
endif
|
endif
|
||||||
############################################################################
|
############################################################################
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user