Skip to content

Commit 5b60a16

Browse files
authored
Merge pull request ibmruntimes#387 from keithc-ca/path
Remove redundant export of PATH on Windows
2 parents 4a31179 + cd5a597 commit 5b60a16

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

closed/OpenJ9.gmk

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -104,15 +104,13 @@ ifeq (false,$(WARNINGS_AS_ERRORS_OPENJ9))
104104
export UMA_SUPPRESS_WARNINGS_AS_ERRORS := 1
105105
endif
106106

107-
ifeq (windows,$(OPENJDK_TARGET_OS))
107+
ifeq ($(call isTargetOs, windows), true)
108108
# convert windows path to unix path
109109
UnixPath = $(shell $(PATHTOOL) -u $1)
110-
# set Visual Studio environment
111-
# wrap PATH in quotes as it contains spaces (unix path)
112-
EXPORT_COMPILER_ENV_VARS := PATH="$(PATH)"
113-
else ifeq (zos,$(OPENJDK_TARGET_OS))
110+
EXPORT_COMPILER_ENV_VARS :=
111+
else ifeq ($(call isTargetOs, zos), true)
114112
UnixPath = $1
115-
# Allow options to follow the input file name
113+
# allow options to follow the input file name
116114
EXPORT_COMPILER_ENV_VARS := _CC_CCMODE=1 _C89_CCMODE=1 _CXX_CCMODE=1
117115
else
118116
UnixPath = $1

0 commit comments

Comments
 (0)