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

Reply via email to