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/