Issue 6682 - Automated builds

.gitlab-ci.yml

 # This file is part of Adblock Plus <https://adblockplus.org/>,
 # Copyright (C) 2006-present eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 # GNU General Public License for more details.
 #
 # You should have received a copy of the GNU General Public License
 # along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
 #
 # NOTE:
 # This CI configuration is meant to be run on private runners, which can be
 # provisioned with the Ansible roles from
 # https://gitlab.com/eyeo/devops/ansible-role-adblockplus-builder and
 # https://gitlab.com/eyeo/devops/ansible-role-gitlab-runner
 
 stages:
   - prepare
   - test
+  - build
 
 .dependencies:
   script: &dependencies
     - mkdir -p .git/info
     - pip install --user Jinja2 cryptography
     - python ensure_dependencies.py
     - "[ -d node_modules ] || npm install"
 
-.test_template:
-  stage: test
-  before_script: *dependencies
+.pull_cache:
   cache:
     key: cache_$CI_COMMIT_SHA
     paths:
       - ./
     policy: pull
 
+.test_template:
+  extends: .pull_cache
+  stage: test
+  before_script: *dependencies
+
+.build:
+  extends: .pull_cache
+  before_script: *dependencies
+  stage: build
+  only:
+    - master
+  script:
+    - python build.py build -t $PLATFORM
+  artifacts:
+    paths:
+      - ./*.xpi
+      - ./*.zip
+      - ./*.appx

[Sebastian Noack, 2018/10/08 15:16:55]

Perhaps we should make the filename a little more specific. e.g.
adblockpluschrome-*.zip. Also from the example in the documentation adding ./
doesn't seem to be necessary, or do I miss something here?

[tlucas, 2018/10/08 15:39:13]

IMHO ideally, we would again read the "basename" and the extension for $PLATFORM
here, to match to correct file - unfortunately, (dynamic) variables are not
available in artifact:paths.

-> Done, as per your suggestion.

 
 prepare-dependencies:
   stage: prepare
   script: *dependencies
   cache:
     key: cache_$CI_COMMIT_SHA
     paths:
       - ./
     policy: push
 
 tests:gecko:
   extends: .test_template
   script:
     - npm run test-only -- -g gecko
 
 tests:chrome:
   extends: .test_template
   script:
     - xvfb-run npm run test-only -- -g chrome
 
 lint:
   extends: .test_template
   script:
     - npm run lint
+
+build:chrome:
+  extends: .build
+  variables:
+    PLATFORM: chrome
+
+build:gecko:
+  extends: .build
+  variables:
+    PLATFORM: gecko
+
+build:edge:
+  extends: .build
+  variables:
+    PLATFORM: edge
+  allow_failure: true

[Sebastian Noack, 2018/10/08 15:16:55]

Can we add a comment referring to the issue here?

[tlucas, 2018/10/08 15:39:13]

Done.