Fix issue eclipse-che#218.

It should not add GAE related files into `eclipse/php` image. Use `eclipse/php:gae` instead. Signed-off-by: Masaki Muranaka <[email protected]>
PizzaFactory · Nov 15, 2018 · f28679e · f28679e
1 parent 6e76a4a
commit f28679e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 14 deletions.
diff --git a/recipes/php/Dockerfile b/recipes/php/Dockerfile
@@ -87,18 +87,10 @@ RUN mkdir -p ${HOME}/che/ls-php/php-language-server && \
     rm -rf vendor && \
     sudo chgrp -R 0 ${HOME}/che && \
     sudo chmod -R g+rwX ${HOME}/che
-ENV GAE_VERSION="1.9.70"
 RUN sudo apt-get update && \
     sudo apt-get install --no-install-recommends -y -q build-essential python2.7 python2.7-dev python-pip php-bcmath && \
     sudo pip install -U pip && \
     sudo pip install virtualenv
-RUN cd /home/user/ && wget -q https://storage.googleapis.com/appengine-sdks/featured/google_appengine_${GAE_VERSION}.zip && \
-    unzip -q google_appengine_${GAE_VERSION}.zip && \
-    rm google_appengine_${GAE_VERSION}.zip && \
-    for f in "/home/user/google_appengine"; do \
-      sudo chgrp -R 0 ${f} && \
-      sudo chmod -R g+rwX ${f}; \
-    done
 EXPOSE 8080 8000
 
 # label is used in Servers tab to display mapped port for Apache process on 80 port in the container

diff --git a/recipes/php/gae/Dockerfile b/recipes/php/gae/Dockerfile
@@ -10,16 +10,17 @@
 
 FROM eclipse/php
 ENV GAE /home/user/google_appengine
+ENV GAE_VERSION="1.9.70"
 
 RUN sudo apt-get update && \
     sudo apt-get install --no-install-recommends -y -q build-essential python2.7 python2.7-dev python-pip php-bcmath && \
     sudo pip install -U pip && \
     sudo pip install virtualenv
-RUN cd /home/user/ && wget -q https://storage.googleapis.com/appengine-sdks/featured/google_appengine_1.9.40.zip && \
-    unzip -q google_appengine_1.9.40.zip && \
-    rm google_appengine_1.9.40.zip && \
-    for f in "/home/user"; do \
-    sudo chgrp -R 0 ${f} && \
-    sudo chmod -R g+rwX ${f}; \
+RUN cd /home/user/ && wget -q https://storage.googleapis.com/appengine-sdks/featured/google_appengine_${GAE_VERSION}.zip && \
+    unzip -q google_appengine_${GAE_VERSION}.zip && \
+    rm google_appengine_${GAE_VERSION}.zip && \
+    for f in "/home/user/google_appengine"; do \
+      sudo chgrp -R 0 ${f} && \
+      sudo chmod -R g+rwX ${f}; \
     done
 EXPOSE 8080 8000