Skip to content

Commit

Permalink
Merge pull request #35 from danjapapajani/target-os
Browse files Browse the repository at this point in the history
Update target OS conditional for z/OS
  • Loading branch information
keithc-ca authored Sep 5, 2024
2 parents ac89d93 + 450b5a5 commit 980a836
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 6 deletions.
2 changes: 1 addition & 1 deletion closed/OpenJ9.gmk
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ ifeq (windows,$(OPENJDK_TARGET_OS))
# - wrap LIB in single-quotes to hide backslashes from make
# - wrap PATH in double-quotes as it may contain spaces (unix path)
EXPORT_COMPILER_ENV_VARS := LIB='$(OPENJ9_VS_LIB)' PATH="$(PATH)"
else ifeq (zos,$(OPENJDK_TARGET_OS))
else ifeq ($(call isTargetOs, zos), true)
UnixPath = $1
# Allow options to follow the input file name
EXPORT_COMPILER_ENV_VARS := _CC_CCMODE=1 _C89_CCMODE=1 _CXX_CCMODE=1
Expand Down
5 changes: 2 additions & 3 deletions closed/custom/modules/java.base/Copy.gmk
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ $(call openj9_copy_files,, \
$(OPENJ9_VM_BUILD_DIR)/include/jvmti.h \
$(INCLUDE_TARGET_DIR)/jvmti.h)

ifeq (zos,$(OPENJDK_TARGET_OS))
ifeq ($(call isTargetOs, zos), true)
$(call openj9_copy_files,, \
$(OPENJ9_TOPDIR)/runtime/include/jni_convert.h \
$(INCLUDE_TARGET_DIR)/jni_convert.h)
Expand Down Expand Up @@ -99,8 +99,7 @@ $(call openj9_copy_shlibs, \
omrsig \
)

ifeq (zos,$(OPENJDK_TARGET_OS))

ifeq ($(call isTargetOs, zos), true)
$(call openj9_copy_files_and_debuginfos, \
$(addsuffix /$(LIBRARY_PREFIX)j9a2e$(SHARED_LIBRARY_SUFFIX), \
$(OPENJ9_VM_BUILD_DIR) \
Expand Down
4 changes: 2 additions & 2 deletions closed/make/modules/ibm.healthcenter/Copy.gmk
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ $(HEALTHCENTER_EXTRACT) : $(HEALTHCENTER_JAR)
@$(RM) -rf $(@D)
$(call MakeDir, $(@D))
$(UNZIP) -q $< -d $(@D)
ifeq (zos,$(OPENJDK_TARGET_OS))
ifeq ($(call isTargetOs, zos), true)
$(CHTAG) -c ISO8859-1 -t $(HEALTHCENTER_HOME)/healthcenter.properties
endif
@$(TOUCH) $@
Expand Down Expand Up @@ -70,7 +70,7 @@ HEALTHCENTER_LIBRARIES = \
$(wildcard $(HEALTHCENTER_HOME)/plugins/$(LIBRARY_PREFIX)hcmqtt$(SHARED_LIBRARY_SUFFIX))

# User-configurable .properties files should be encoded in EBCDIC on z/OS.
ifeq (zos,$(OPENJDK_TARGET_OS))
ifeq ($(call isTargetOs, zos), true)
FIX_ENCODING := | $(ICONV) -f ISO8859-1 -t IBM-1047
else
FIX_ENCODING :=
Expand Down

0 comments on commit 980a836

Please sign in to comment.