Skip to content

Commit

Permalink
Make the dist_install test use Java 8
Browse files Browse the repository at this point in the history
This is necessary because the Kotlin support requires Java 8 or
higher.

Making this test use Java 8 required switching it to the java_stretch
Docker image and updating that image to include some Python
dependencies.
  • Loading branch information
acozzette committed May 12, 2021
1 parent 0aabf89 commit 95ff3f8
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 3 deletions.
4 changes: 2 additions & 2 deletions kokoro/linux/dist_install/build.sh
Expand Up @@ -7,8 +7,8 @@ set -ex # exit immediately on error
# Change to repo root
cd $(dirname $0)/../../..

# Run tests under release docker image.
export DOCKERFILE_DIR=kokoro/linux/64-bit
export DOCKERHUB_ORGANIZATION=protobuftesting
export DOCKERFILE_DIR=kokoro/linux/dockerfile/test/java_stretch
export DOCKER_RUN_SCRIPT=kokoro/linux/pull_request_in_docker.sh
export OUTPUT_DIR=testoutput
export TEST_SET="dist_install"
Expand Down
7 changes: 7 additions & 0 deletions kokoro/linux/dockerfile/test/java_stretch/Dockerfile
Expand Up @@ -29,3 +29,10 @@ RUN apt-get install -y \
# -- For java_linkage_monitor \
openjdk-8-jdk \
&& apt-get clean

# Python dependencies
RUN apt-get install -y \
python-setuptools \
python-pip \
virtualenv \
&& apt-get clean
2 changes: 1 addition & 1 deletion tests.sh
Expand Up @@ -104,7 +104,7 @@ build_dist_install() {

# Try to install Java
pushd java
use_java jdk7
use_java jdk8
$MVN install
popd

Expand Down

0 comments on commit 95ff3f8

Please sign in to comment.