absurdfarce commented on code in PR #1923:
URL:
https://github.com/apache/cassandra-java-driver/pull/1923#discussion_r1544088933
##########
Jenkinsfile:
##########
@@ -137,8 +136,8 @@ def executeTests() {
printenv | sort
mvn -B -V ${INTEGRATION_TESTS_FILTER_ARGUMENT} -T 1 verify \
- -Ptest-jdk-${TEST_JAVA_VERSION} \
- -DtestJavaHome=${TEST_JAVA_HOME} \
+ -Ptest-jdk-'''+testJavaVersion+''' \
+ -DtestJavaHome='''+testJavaHome+''' \
Review Comment:
We can't use string interpolation here since the ${} vals are intended to be
passed through literally to the shell (so that the shell can do variable
subsitution based on env vars). So instead we just add whatever we got for the
two vars above directly into the multi-line string here and then continue on
with a new multi-line string.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org
