From 4f039117093910671be43142826d881fdce8d74d Mon Sep 17 00:00:00 2001 From: Kostas Tsiounis Date: Tue, 27 Aug 2024 11:36:05 -0400 Subject: [PATCH] Avoid rebuilding native code headers for OpenJCEPlus Some OpenJCEPlus headers are already compiled as part of a previous target, so there is no reason to recompile them. Instead an additional variable is passed as part of the make command to indicate that this step can be skipped. Signed-off-by: Kostas Tsiounis --- closed/make/modules/openjceplus/Lib.gmk | 24 +++++++++++++++--------- 1 file changed, 15 insertions(+), 9 deletions(-) diff --git a/closed/make/modules/openjceplus/Lib.gmk b/closed/make/modules/openjceplus/Lib.gmk index 21dedc3bd24..196957b73ec 100644 --- a/closed/make/modules/openjceplus/Lib.gmk +++ b/closed/make/modules/openjceplus/Lib.gmk @@ -23,8 +23,9 @@ include LibCommon.gmk ifeq (true,$(BUILD_OPENJCEPLUS)) # Identify the desired JGSKIT target platform. -OPENJCEPLUS_BOOT_JDK := $(BOOT_JDK) +OPENJCEPLUS_JDK := $(JDK_OUTPUTDIR) OPENJCEPLUS_GSKIT_HOME := $(OPENJCEPLUS_TOPDIR)/ock/jgsk_sdk +OPENJCEPLUS_HEADER_FILES := $(SUPPORT_OUTPUTDIR)/headers/openjceplus OPENJCEPLUS_JCE_CLASSPATH := $(JDK_OUTPUTDIR)/modules/openjceplus:$(JDK_OUTPUTDIR)/modules/java.base OPENJCEPLUS_JGSKIT_MAKE := jgskit.mak OPENJCEPLUS_JGSKIT_MAKE_PATH := $(OPENJCEPLUS_TOPDIR)/src/main/native @@ -42,9 +43,10 @@ else ifeq ($(call isTargetOs, linux), true) endif else ifeq ($(call isTargetOs, windows), true) ifeq ($(call isTargetCpu, x86_64), true) - OPENJCEPLUS_BOOT_JDK := $(call MixedPath,$(OPENJCEPLUS_BOOT_JDK)) + OPENJCEPLUS_JDK := $(call MixedPath,$(OPENJCEPLUS_JDK)) OPENJCEPLUS_GSKIT_HOME := $(call MixedPath,$(OPENJCEPLUS_GSKIT_HOME)) - OPENJCEPLUS_JCE_CLASSPATH := "$(call MixedPath,$(JDK_OUTPUTDIR)/modules/openjceplus)\;$(call MixedPath,$(JDK_OUTPUTDIR)/modules/java.base)" + OPENJCEPLUS_HEADER_FILES := $(call MixedPath,$(OPENJCEPLUS_HEADER_FILES)) + OPENJCEPLUS_JCE_CLASSPATH := "$(call MixedPath,$(JDK_OUTPUTDIR)/modules/openjceplus);$(call MixedPath,$(JDK_OUTPUTDIR)/modules/java.base)" OPENJCEPLUS_JGSKIT_MAKE := jgskit.win64.mak OPENJCEPLUS_JGSKIT_PLATFORM := win64 endif @@ -58,12 +60,16 @@ endif # OPENJCEPLUS_JGSKIT_PLATFORM compile-libs : @$(ECHO) Compiling OpenJCEPlus native code - export \ - GSKIT_HOME=$(OPENJCEPLUS_GSKIT_HOME) \ - JAVA_HOME=$(OPENJCEPLUS_BOOT_JDK) \ - JCE_CLASSPATH=$(OPENJCEPLUS_JCE_CLASSPATH) \ - PLATFORM=$(OPENJCEPLUS_JGSKIT_PLATFORM) \ - && $(MAKE) -j1 -C $(OPENJCEPLUS_JGSKIT_MAKE_PATH) -f $(OPENJCEPLUS_JGSKIT_MAKE) all + $(MAKE) \ + -C $(OPENJCEPLUS_JGSKIT_MAKE_PATH) \ + -f $(OPENJCEPLUS_JGSKIT_MAKE) \ + EXTERNAL_HEADERS=true \ + GSKIT_HOME=$(OPENJCEPLUS_GSKIT_HOME) \ + JAVA_HOME=$(OPENJCEPLUS_JDK) \ + JCE_CLASSPATH=$(OPENJCEPLUS_JCE_CLASSPATH) \ + OPENJCEPLUS_HEADER_FILES=$(OPENJCEPLUS_HEADER_FILES) \ + PLATFORM=$(OPENJCEPLUS_JGSKIT_PLATFORM) \ + all @$(ECHO) OpenJCEplus compile complete TARGETS += compile-libs