[ 
https://issues.apache.org/jira/browse/GEODE-60?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14958018#comment-14958018
 ] 

Dan Smith commented on GEODE-60:
--------------------------------

Hi Jun,

I ran into some problems running those scripts. The geode runtime build was 
failing because git did not exist. The line yum remove -y perl in the base 
image removed git because it depends on perl. I'm attaching another diff that 
fixes some of these issues -GEODE-60-8.diff. Here's the total change from your 
diff. If you are ok with this, I'll commit this last diff.

diff --git a/dev-tools/docker/base/Dockerfile b/dev-tools/docker/base/Dockerfile
index 0bd2982..513811c 100644
--- a/dev-tools/docker/base/Dockerfile
+++ b/dev-tools/docker/base/Dockerfile
@@ -48,7 +48,6 @@ RUN   yum install -y wget which tar git \
                                                
$JAVA_HOME/jre/lib/amd64/libjfx*.so \
                                                jdk-8u51-linux-x64.tar.gz \
                                                /usr/share/locale/* \
-    && yum remove -y perl \
        && yum clean all
 
 ENV PATH $PATH:$JAVA_HOME/bin
diff --git a/dev-tools/docker/compile/Dockerfile 
b/dev-tools/docker/compile/Dockerfile
index d9eb647..bab74d4 100644
--- a/dev-tools/docker/compile/Dockerfile
+++ b/dev-tools/docker/compile/Dockerfile
@@ -19,6 +19,3 @@ MAINTAINER Geode Community <d...@geode.incubator.apache.org>
 
 LABEL Vendor="Apache Geode (incubating)"
 LABEL version=unstable
-
-RUN yum install -y vim
-


> Geode build environment within Docker to enable anybody and on any OS and any 
> environment can build Geode as long as there is Docker
> ------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: GEODE-60
>                 URL: https://issues.apache.org/jira/browse/GEODE-60
>             Project: Geode
>          Issue Type: Sub-task
>          Components: build
>            Reporter: jun aoki
>            Assignee: jun aoki
>         Attachments: GEODE-60-2.patch, GEODE-60-3.patch, GEODE-60-4.patch, 
> GEODE-60-5.patch, GEODE-60-6.patch, GEODE-60-7.patch, GEODE-60.patch
>
>
> geode build is on gradle and seems very tidy but it still requires third 
> party libraries, like Java.
> Let's have a Docker environment so that at least we have an reliable 
> immutable environment and we can use it on CI on builds.apache.org 
> https://builds.apache.org/view/E-G/view/Geode/
> (From my experience, It is a bit of pain to find a right environment on 
> builds.a.o but Docker is a cure so far.) 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to