martin-g commented on a change in pull request #276: URL: https://github.com/apache/tomcat/pull/276#discussion_r411923652
########## File path: .travis.yml ########## @@ -32,14 +36,7 @@ addons: - libtool-bin install: - - ARCH=`uname -p` - - echo $ARCH - - JDK_X64="https://github.com/AdoptOpenJDK/openjdk8-binaries/releases/download/jdk8u242-b08/OpenJDK8U-jdk_x64_linux_hotspot_8u242b08.tar.gz" - - JDK_ARM64="https://github.com/AdoptOpenJDK/openjdk8-binaries/releases/download/jdk8u232-b09/OpenJDK8U-jdk_aarch64_linux_hotspot_8u232b09.tar.gz" - - if test "X$ARCH" = "Xaarch64"; then JDK_URL=$JDK_ARM64; else JDK_URL=$JDK_X64; fi - - wget -q $JDK_URL && tar xzf OpenJDK*.tar.gz - - mv jdk8* jdk - - export JAVA_HOME=`pwd`/jdk Review comment: The above was here for two reasons: 1) because TravisCI is really inconsistent in what is available in the different types of VMs 1.1) JAVA_HOME is exported for AMD64, but not for ARM64 1.2) `mvn` is available for AMD64, but not for ARM 1.3.) most important - to keep the version of the used JDK the same for all environments. The one provided by Travis for ARM64 was older than the one for AMD64 For the above reasons I'd prefer to keep the control ########## File path: .travis.yml ########## @@ -15,8 +15,12 @@ dist: bionic language: java -jdk: oraclejdk8 -arch: arm64 +jobs: + include: + - arch: s390x + jdk: openjdk11 Review comment: any reason to use JDK 11 for s390x ? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org