Fix: missed 'set' to assign environment variable and log it (#11869)

andsel · web-flow · commit fb8108e63bc7 · 2020-05-07T09:12:27.000+02:00
In batch shell to assign a variable we need set, else the shell interpreter log an error and continue Close #11869
diff --git a/ci/unit_tests.bat b/ci/unit_tests.bat
@@ -41,7 +41,8 @@ echo Using drive !use_drive! for %WORKSPACE%
 
 echo Running core tests..
 if defined BUILD_JAVA_HOME (
-  GRADLE_OPTS="%GRADLE_OPTS% -Dorg.gradle.java.home=%BUILD_JAVA_HOME%"
+  set GRADLE_OPTS="%GRADLE_OPTS% -Dorg.gradle.java.home=%BUILD_JAVA_HOME%"
+  echo GRADLE_OPTS is: %GRADLE_OPTS%, BUILD_JAVA_HOME: %BUILD_JAVA_HOME%
 )
 call .\gradlew.bat test --console=plain --no-daemon --info
 

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,8 @@ echo Using drive !use_drive! for %WORKSPACE%`
`41`	`41`
`42`	`42`	`echo Running core tests..`
`43`	`43`	`if defined BUILD_JAVA_HOME (`
`44`		`- GRADLE_OPTS="%GRADLE_OPTS% -Dorg.gradle.java.home=%BUILD_JAVA_HOME%"`
	`44`	`+ set GRADLE_OPTS="%GRADLE_OPTS% -Dorg.gradle.java.home=%BUILD_JAVA_HOME%"`
	`45`	`+ echo GRADLE_OPTS is: %GRADLE_OPTS%, BUILD_JAVA_HOME: %BUILD_JAVA_HOME%`
`45`	`46`	`)`
`46`	`47`	`call .\gradlew.bat test --console=plain --no-daemon --info`
`47`	`48`