Browse Source

Merge pull request #1016 from stephenmoloney/devel-local

make METEOR_EDGE optional. Fix node_gyp bug in docker build.
Lauri Ojansivu 8 years ago
parent
commit
c4835847c0
2 changed files with 22 additions and 22 deletions
  1. 10 10
      .travis.yml
  2. 12 12
      Dockerfile

+ 10 - 10
.travis.yml

@@ -15,13 +15,13 @@ before_install:
   # ^^ Note - need to come up with some way of checking the output from docker run
   # ^^ Note - need to come up with some way of checking the output from docker run
   # that it was a success... perhaps the nodejs server can output a success message?
   # that it was a success... perhaps the nodejs server can output a success message?
 
 
-language: node_js
-
-node_js:
-  - "6.10.2"
-
-install:
-  - "npm install"
-
-script:
-  - "npm test"
+#language: node_js
+#
+#node_js:
+#  - "4.8.1"
+#
+#install:
+#  - "npm install"
+#
+#script:
+#  - "npm test"

+ 12 - 12
Dockerfile

@@ -5,16 +5,18 @@ MAINTAINER wekan
 ARG NODE_VERSION
 ARG NODE_VERSION
 ARG METEOR_RELEASE
 ARG METEOR_RELEASE
 ARG METEOR_EDGE
 ARG METEOR_EDGE
+ARG USE_EDGE
 ARG NPM_VERSION
 ARG NPM_VERSION
 ARG ARCHITECTURE
 ARG ARCHITECTURE
 ARG SRC_PATH
 ARG SRC_PATH
 
 
 # Set the environment variables (defaults where required)
 # Set the environment variables (defaults where required)
-ENV BUILD_DEPS="wget curl bzip2 build-essential python git ca-certificates"
+ENV BUILD_DEPS="wget curl bzip2 build-essential python git ca-certificates gcc-4.9"
 ENV GOSU_VERSION=1.10
 ENV GOSU_VERSION=1.10
 ENV NODE_VERSION ${NODE_VERSION:-v4.8.1}
 ENV NODE_VERSION ${NODE_VERSION:-v4.8.1}
-ENV METEOR_RELEASE ${METEOR_RELEASE:-1.4.5}
-ENV METEOR_EDGE ${METEOR_EDGE:-1.4.4.1}
+ENV METEOR_RELEASE ${METEOR_RELEASE:-1.4.4.1}
+ENV USE_EDGE ${USE_EDGE:-false}
+ENV METEOR_EDGE ${METEOR_EDGE:-1.4.4-rc.6}
 ENV NPM_VERSION ${NPM_VERSION:-4.5.0}
 ENV NPM_VERSION ${NPM_VERSION:-4.5.0}
 ENV ARCHITECTURE ${ARCHITECTURE:-linux-x64}
 ENV ARCHITECTURE ${ARCHITECTURE:-linux-x64}
 ENV SRC_PATH ${SRC_PATH:-./}
 ENV SRC_PATH ${SRC_PATH:-./}
@@ -76,18 +78,16 @@ RUN \
     sed -i "s|RELEASE=.*|RELEASE=${METEOR_RELEASE}\"\"|g" ./install_meteor.sh && \
     sed -i "s|RELEASE=.*|RELEASE=${METEOR_RELEASE}\"\"|g" ./install_meteor.sh && \
     echo "Starting meteor ${METEOR_RELEASE} installation...   \n" && \
     echo "Starting meteor ${METEOR_RELEASE} installation...   \n" && \
     chown wekan:wekan ./install_meteor.sh && \
     chown wekan:wekan ./install_meteor.sh && \
-    ###########################
-    ###########################
-    # Block for ensuring installation of release candidates - perhaps remove later.
-    gosu wekan:wekan sh ./install_meteor.sh || \
-    ( \
+    \
+    # Check if opting for a release candidate instead of major release
+    if [ "$USE_EDGE" = false ]; then \
+      gosu wekan:wekan sh ./install_meteor.sh; \
+    else \
       gosu wekan:wekan git clone --recursive git://github.com/meteor/meteor.git /home/wekan/.meteor && \
       gosu wekan:wekan git clone --recursive git://github.com/meteor/meteor.git /home/wekan/.meteor && \
       cd /home/wekan/.meteor && \
       cd /home/wekan/.meteor && \
       gosu wekan:wekan git checkout release/METEOR@${METEOR_EDGE} && \
       gosu wekan:wekan git checkout release/METEOR@${METEOR_EDGE} && \
-      gosu wekan /home/wekan/.meteor/meteor -- help \
-    ) && \
-    ###########################
-    ###########################
+      gosu wekan /home/wekan/.meteor/meteor -- help; \
+    fi && \
     \
     \
     # Build app
     # Build app
     cd /home/wekan/app && \
     cd /home/wekan/app && \