Skip to content

Commit

Permalink
Merge pull request #5054 from Yuehan-Lin/autoBool
Browse files Browse the repository at this point in the history
Changed AUTO_DETECT variable from on/off into true/false
  • Loading branch information
llxia authored Mar 12, 2019
2 parents 1b67870 + 105ed11 commit 15c5880
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 10 deletions.
8 changes: 4 additions & 4 deletions test/TestConfig/envSettings.mk
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@

HOTSPOT_IMPLS=$(or hotspot,sap)

ifneq ($(AUTO_DETECT), off)
ifneq ($(AUTO_DETECT), false)
ifndef SPEC
export SPEC:=$(DETECTED_SPEC)
else
Expand Down Expand Up @@ -66,13 +66,13 @@ ifneq ($(AUTO_DETECT), off)
endif
else
ifndef SPEC
$(error Please export AUTO_DETECT=on or export SPEC value manually (i.e., export SPEC=linux_x86-64_cmprssptrs))
$(error Please export AUTO_DETECT=true or export SPEC value manually (i.e., export SPEC=linux_x86-64_cmprssptrs))
endif
ifndef JDK_VERSION
$(error Please export AUTO_DETECT=on or export JDK_VERSION value manually (i.e., export JDK_VERSION=8))
$(error Please export AUTO_DETECT=true or export JDK_VERSION value manually (i.e., export JDK_VERSION=8))
endif
ifndef JDK_IMPL
$(error Please export AUTO_DETECT=on or export JDK_IMPL value manually (i.e., export JDK_IMPL=openj9))
$(error Please export AUTO_DETECT=true or export JDK_IMPL value manually (i.e., export JDK_IMPL=openj9))
endif
endif

6 changes: 3 additions & 3 deletions test/TestConfig/run_configure.mk
Original file line number Diff line number Diff line change
Expand Up @@ -33,10 +33,10 @@ ifneq (,$(findstring jre,$(JAVA_BIN)))
endif

testconfig:
ifneq ($(AUTO_DETECT), off)
@echo "AUTO_DETECT is turned on"
ifneq ($(AUTO_DETECT), false)
@echo "AUTO_DETECT is set to true"
ant -f ./src/build_envInfo.xml -DJAVA_BIN=$(JAVA_BIN) -DJAVAC=$(JAVAC)
else
@echo "AUTO_DETECT is turned off"
@echo "AUTO_DETECT is set to false"
endif
$(MAKE) -f makeGen.mk AUTO_DETECT=$(AUTO_DETECT)
6 changes: 3 additions & 3 deletions test/docs/OpenJ9TestUserGuide.md
Original file line number Diff line number Diff line change
Expand Up @@ -60,13 +60,13 @@ tools should be installed on your test machine to run tests.
* auto detection:
By default, AUTO_DETECT is turned on. SPEC, JDK_VERSION, and JDK_IMPL do not need to be exported.
If you wish to turn off AUTO_DETECT and export SPEC, JDK_VERSION, and JDK_IMPL manually, you can turn off AUTO_DETECT.
If you do not wish to use AUTO_DETECT and export SPEC, JDK_VERSION, and JDK_IMPL manually, you can export AUTO_DETECT to false.
```
export AUTO_DETECT=off
export AUTO_DETECT=false
```
e.g.,
```
export AUTO_DETECT=off
export AUTO_DETECT=false
export SPEC=linux_x86-64_cmprssptrs
export JDK_VERSION=8
export JDK_IMPL=openj9
Expand Down

0 comments on commit 15c5880

Please sign in to comment.