travis: Test dist path [WIP].

.travis.yml

@@ -2,16 +2,31 @@ sudo: required
 dist: trusty
 
 os:
-- osx
-- linux
+  - osx
+  - linux
+
+osx_image: xcode9.0
 
 addons:
   apt:
     packages:
-    - build-essential
-    - libxext-dev
-    - libxtst-dev
-    - libxkbfile-dev
+      - build-essential
+      - libxext-dev
+      - libxtst-dev
+      - libxkbfile-dev
+  artifacts:
+    paths:
+      - $(ls ./dist/*.AppImage | tr "\n" ":")
+      - $(ls ./dist/*.deb | tr "\n" ":")
+      - $(ls ./dist/*.dmg | tr "\n" ":")
+      - $(ls ./dist/*.zip | tr "\n" ":")
+      - $(ls ./dist/*.dmg.blockmap | tr "\n" ":")
+      - $(ls ./dist/github/*.json | tr "\n" ":")
+      - $(ls ./dist/github/*.yml | tr "\n" ":")
+      - $(ls ./dist/*.yml | tr "\n" ":")
+      - $(ls ./dist/mac/*.yml | tr "\n" ":")
+      - $(ls ./dist/linux/*.yml | tr "\n" ":")
+    debug: true
 
 language: node_js
 node_js:
@@ -24,14 +39,23 @@ before_install:
 
 cache:
   directories:
-  - node_modules
-  - app/node_modules
+    - node_modules
+    - app/node_modules
+    - ~/.cache
 
 script:
-- npm run travis
+  - npm run travis
+  - chmod +x ./scripts/install-release-dependencies.sh
+  - ./scripts/install-release-dependencies.sh
+  - npm run dist
+  - node ./scripts/prepare-artifacts.js
+  # log out /dist files might be useful to know
+  # what files are uploaded
+  - ls dist
+
 notifications:
   webhooks:
     urls:
       - https://zulip.org/zulipbot/travis
     on_success: always
-    on_failure: always
+    on_failure: always

app/package.json

@@ -30,7 +30,7 @@
     "electron-is-dev": "0.3.0",
     "electron-log": "2.2.7",
     "electron-spellchecker": "1.1.2",
-    "electron-updater": "2.18.2",
+    "electron-updater": "2.21.0",
     "electron-window-state": "4.1.1",
     "is-online": "7.0.0",
     "node-json-db": "0.7.3",

package.json

@@ -114,7 +114,7 @@
     "cp-file": "^5.0.0",
     "devtron": "1.4.0",
     "electron": "1.8.2",
-    "electron-builder": "19.53.6",
+    "electron-builder": "20.4.1",
     "electron-connect": "0.6.2",
     "electron-debug": "1.4.0",
     "google-translate-api": "2.3.0",
@@ -164,7 +164,8 @@
     ],
     "ignore": [
       "tests/*.js",
-      "tools/locale-helper/*.js"
+      "tools/locale-helper/*.js",
+      "scripts/*.js"
     ],
     "envs": [
       "node",

scripts/install-release-dependencies.sh

@@ -0,0 +1,9 @@
+#!/usr/bin/env bash
+
+set -e
+if [[ "$TRAVIS_OS_NAME" == "linux" ]]; then
+  sudo apt-get install --no-install-recommends -y icnsutils
+
+  # to build 32 bit from a machine with 64 bit
+  sudo apt-get install --no-install-recommends -y gcc-multilib g++-multilib
+fi

scripts/prepare-artifacts.js

@@ -0,0 +1,28 @@
+#!/usr/bin/env node
+
+if (!process.env.TRAVIS_OS_NAME === 'linux') {
+  process.exit(0);
+}
+
+const fs = require('fs');
+const path = require('path');
+
+// go to dist directory
+process.chdir(path.resolve(__dirname, '../dist'));
+
+const extensions = /\.deb|\.AppImage|\.zip/;
+const files = fs.readdirSync(process.cwd()).filter(file => file.includes);
+
+function changeFileName(file) {
+  file = file.replace(/^z/, 'Z');
+  file = file.replace(/_{2}/, '-');
+  return file;
+}
+
+// Change file name to what we want
+// eg zulip_1.8.2_amd64.deb -> Zulip-1.8.2-amd64.deb
+// and change file name
+files.map(file => {
+  const newFileName = changeFileName(file);
+  fs.renameSync(file, newFileName);
+});