You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "Dan Smith (JIRA)" <ji...@apache.org> on 2015/10/15 01:42:06 UTC
[jira] [Comment Edited] (GEODE-60) Geode build environment within
Docker to enable anybody and on any OS and any environment can build Geode
as long as there is Docker
[ https://issues.apache.org/jira/browse/GEODE-60?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14958018#comment-14958018 ]
Dan Smith edited comment on GEODE-60 at 10/14/15 11:42 PM:
-----------------------------------------------------------
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.
{noformat}
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 <de...@geode.incubator.apache.org>
LABEL Vendor="Apache Geode (incubating)"
LABEL version=unstable
-
-RUN yum install -y vim
-
{noformat}
was (Author: upthewaterspout):
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 <de...@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-8.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)