Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make the dist_install test use Java 8 #8609

Merged
merged 1 commit into from May 12, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
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
13 changes: 6 additions & 7 deletions kokoro/linux/dockerfile/test/java_stretch/Dockerfile
@@ -1,6 +1,6 @@
FROM debian:stretch

# Install dependencies. We start with the basic ones require to build protoc
# Install dependencies. We start with the basic ones required to build protoc
# and the C++ build
RUN apt-get update && apt-get install -y \
autoconf \
Expand All @@ -20,12 +20,11 @@ RUN apt-get update && apt-get install -y \
parallel \
time \
wget \
&& apt-get clean

# Java dependencies
RUN apt-get install -y \
# -- For all Java builds -- \
# Java dependencies
maven \
# -- For java_linkage_monitor \
openjdk-8-jdk \
# Python dependencies
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