You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@joshua.apache.org by to...@apache.org on 2019/09/29 06:52:24 UTC
[joshua] branch master updated: Add OpenJDK dependency required for
"jni.h"
This is an automated email from the ASF dual-hosted git repository.
tommaso pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/joshua.git
The following commit(s) were added to refs/heads/master by this push:
new 899690e Add OpenJDK dependency required for "jni.h"
new 2274de2 Merge pull request #89 from magali-joine/patch-1
899690e is described below
commit 899690e1e628379238a1ff5fafd58b8c5653da7e
Author: ᎹᎦᎵ ᏦᎢᏁ <52...@users.noreply.github.com>
AuthorDate: Wed Sep 25 16:58:15 2019 -0400
Add OpenJDK dependency required for "jni.h"
Fixes docker build: Solves
```
100%] Linking CXX executable ../tests/model_test
[100%] Built target model_test
/code/jni/kenlm_wrap.cc:28:17: fatal error: jni.h: No such file or directory
#include <jni.h>
^
compilation terminated.
Removing intermediate container 0c2cb0c17a12
```
---
distribution/docker/kenlm/Dockerfile | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/distribution/docker/kenlm/Dockerfile b/distribution/docker/kenlm/Dockerfile
index fd82090..5342275 100644
--- a/distribution/docker/kenlm/Dockerfile
+++ b/distribution/docker/kenlm/Dockerfile
@@ -28,7 +28,8 @@ RUN apt-get update && \
liblzma-dev \
libz-dev \
make \
- curl
+ curl \
+ openjdk-8-jdk
# set environment variables
ENV JAVA_HOME=/usr/lib/jvm/java-8-openjdk-amd64/