From e1d8c67bffdc429fb57544241e50c7c5a0644440 Mon Sep 17 00:00:00 2001 From: Erik Joelsson Date: Fri, 27 Jan 2017 10:15:41 +0100 Subject: [PATCH] 8170483: Remove modules_src_jake workaround for JavaFX transition to new module-info syntax Reviewed-by: alanb, mchung --- common/autoconf/generated-configure.sh | 8 +------- common/autoconf/source-dirs.m4 | 6 ------ make/common/Modules.gmk | 5 +---- 3 files changed, 2 insertions(+), 17 deletions(-) diff --git a/common/autoconf/generated-configure.sh b/common/autoconf/generated-configure.sh index 006fff3175c..3c0f4644798 100644 --- a/common/autoconf/generated-configure.sh +++ b/common/autoconf/generated-configure.sh @@ -5180,7 +5180,7 @@ VS_SDK_PLATFORM_NAME_2013= #CUSTOM_AUTOCONF_INCLUDE # Do not change or remove the following line, it is needed for consistency checks: -DATE_WHEN_GENERATED=1484571183 +DATE_WHEN_GENERATED=1485508515 ############################################################################### # @@ -31283,12 +31283,6 @@ $as_echo "$as_me: The path of IMPORT_MODULES_TOPDIR, which resolves as \"$path\" if test -d "$IMPORT_MODULES_TOPDIR/modules_src"; then IMPORT_MODULES_SRC="$IMPORT_MODULES_TOPDIR/modules_src" fi - # Workaround for using different imported module-info.java in Jake due to a - # change in format. Remove once new format is standard in JDK 9 and javafx - # delivers just that. - if test -d "$IMPORT_MODULES_TOPDIR/modules_src_jake"; then - IMPORT_MODULES_SRC="$IMPORT_MODULES_TOPDIR/modules_src_jake $IMPORT_MODULES_SRC" - fi if test -d "$IMPORT_MODULES_TOPDIR/make"; then IMPORT_MODULES_MAKE="$IMPORT_MODULES_TOPDIR/make" fi diff --git a/common/autoconf/source-dirs.m4 b/common/autoconf/source-dirs.m4 index 49db4734596..95053e760dd 100644 --- a/common/autoconf/source-dirs.m4 +++ b/common/autoconf/source-dirs.m4 @@ -114,12 +114,6 @@ AC_DEFUN_ONCE([SRCDIRS_SETUP_IMPORT_MODULES], if test -d "$IMPORT_MODULES_TOPDIR/modules_src"; then IMPORT_MODULES_SRC="$IMPORT_MODULES_TOPDIR/modules_src" fi - # Workaround for using different imported module-info.java in Jake due to a - # change in format. Remove once new format is standard in JDK 9 and javafx - # delivers just that. - if test -d "$IMPORT_MODULES_TOPDIR/modules_src_jake"; then - IMPORT_MODULES_SRC="$IMPORT_MODULES_TOPDIR/modules_src_jake $IMPORT_MODULES_SRC" - fi if test -d "$IMPORT_MODULES_TOPDIR/make"; then IMPORT_MODULES_MAKE="$IMPORT_MODULES_TOPDIR/make" fi diff --git a/make/common/Modules.gmk b/make/common/Modules.gmk index c3c032ce542..49f301d9da8 100644 --- a/make/common/Modules.gmk +++ b/make/common/Modules.gmk @@ -182,15 +182,12 @@ SRC_SUBDIRS += share/classes # Find all module-info.java files for the current build target platform and # configuration. -# TODO: The $(firstword call is part of a workaround for using different -# imported module-info.java in Jake due to a change in format. Remove once -# new format is standard in JDK 9 and javafx delivers just that. # Param 1 - Module to find for, set to * for finding all FindAllModuleInfos = \ $(wildcard \ $(foreach sub, $(SRC_SUBDIRS), \ $(patsubst %,%/$(strip $1)/$(sub)/module-info.java, $(TOP_SRC_DIRS))) \ - $(patsubst %,%/$(strip $1)/module-info.java, $(firstword $(IMPORT_MODULES_SRC)))) + $(patsubst %,%/$(strip $1)/module-info.java, $(IMPORT_MODULES_SRC))) # Find module-info.java files in the specific source dir # Param 1 - Src dir to find module-info.java files in