From 65f32dfa52f51d59581368aafbf18832d276977c Mon Sep 17 00:00:00 2001
From: Gaëtan Gilbert
Date: Mon, 19 Oct 2020 14:53:40 +0200
Subject: Bench: move variables to the script

IMO it makes more sense this way, also it's more convenient if someone
wants to run the script locally.
---
 dev/bench/gitlab-bench.yml | 12 ------------
 dev/bench/gitlab.sh        | 25 +++++++++++++------------
 2 files changed, 13 insertions(+), 24 deletions(-)

(limited to 'dev')

diff --git a/dev/bench/gitlab-bench.yml b/dev/bench/gitlab-bench.yml
index 4275e3d121..25545cf565 100644
--- a/dev/bench/gitlab-bench.yml
+++ b/dev/bench/gitlab-bench.yml
@@ -11,18 +11,6 @@ bench:
     - timing
   variables:
     GIT_DEPTH: ""
-    coq_pr_number: ""
-    coq_pr_comment_id: ""
-    new_ocaml_switch: "ocaml-base-compiler.4.07.1"
-    old_ocaml_switch: "ocaml-base-compiler.4.07.1"
-    new_coq_repository: "https://gitlab.com/coq/coq.git"
-    old_coq_repository: "https://gitlab.com/coq/coq.git"
-    new_coq_opam_archive_git_uri: "https://github.com/coq/opam-coq-archive.git"
-    old_coq_opam_archive_git_uri: "https://github.com/coq/opam-coq-archive.git"
-    new_coq_opam_archive_git_branch: "master"
-    old_coq_opam_archive_git_branch: "master"
-    num_of_iterations: 1
-    coq_opam_packages: "coq-performance-tests-lite coq-engine-bench-lite coq-hott coq-bignums coq-mathcomp-ssreflect coq-mathcomp-fingroup coq-mathcomp-algebra coq-mathcomp-solvable coq-mathcomp-field coq-mathcomp-character coq-mathcomp-odd-order coq-math-classes coq-corn coq-flocq coq-compcert coq-geocoq coq-color coq-coqprime coq-coqutil coq-bedrock2 coq-rewriter coq-fiat-core coq-fiat-parsers coq-fiat-crypto coq-unimath coq-sf-plf coq-coquelicot coq-lambda-rust coq-verdi coq-verdi-raft coq-fourcolor coq-rewriter-perf-SuperFast"
   artifacts:
     name: "$CI_JOB_NAME"
     paths:
diff --git a/dev/bench/gitlab.sh b/dev/bench/gitlab.sh
index 41f204385f..1dea97160c 100755
--- a/dev/bench/gitlab.sh
+++ b/dev/bench/gitlab.sh
@@ -40,18 +40,19 @@ echo $PWD
 #check_variable "JOB_NAME"
 #check_variable "JENKINS_URL"
 check_variable "CI_JOB_URL"
-check_variable "coq_pr_number"
-check_variable "coq_pr_comment_id"
-check_variable "new_ocaml_switch"
-check_variable "new_coq_repository"
-check_variable "new_coq_opam_archive_git_uri"
-check_variable "new_coq_opam_archive_git_branch"
-check_variable "old_ocaml_switch"
-check_variable "old_coq_repository"
-check_variable "old_coq_opam_archive_git_uri"
-check_variable "old_coq_opam_archive_git_branch"
-check_variable "num_of_iterations"
-check_variable "coq_opam_packages"
+
+: "${coq_pr_number:=''}"
+: "${coq_pr_comment_id:=''}"
+: "${new_ocaml_switch:='ocaml-base-compiler.4.07.1'}"
+: "${old_ocaml_switch:='ocaml-base-compiler.4.07.1'}"
+: "${new_coq_repository:='https://gitlab.com/coq/coq.git'}"
+: "${old_coq_repository:='https://gitlab.com/coq/coq.git'}"
+: "${new_coq_opam_archive_git_uri:='https://github.com/coq/opam-coq-archive.git'}"
+: "${old_coq_opam_archive_git_uri:='https://github.com/coq/opam-coq-archive.git'}"
+: "${new_coq_opam_archive_git_branch:='master'}"
+: "${old_coq_opam_archive_git_branch:='master'}"
+: "${num_of_iterations:=1}"
+: "${coq_opam_packages:='coq-performance-tests-lite coq-engine-bench-lite coq-hott coq-bignums coq-mathcomp-ssreflect coq-mathcomp-fingroup coq-mathcomp-algebra coq-mathcomp-solvable coq-mathcomp-field coq-mathcomp-character coq-mathcomp-odd-order coq-math-classes coq-corn coq-flocq coq-compcert coq-geocoq coq-color coq-coqprime coq-coqutil coq-bedrock2 coq-rewriter coq-fiat-core coq-fiat-parsers coq-fiat-crypto coq-unimath coq-sf-plf coq-coquelicot coq-lambda-rust coq-verdi coq-verdi-raft coq-fourcolor coq-rewriter-perf-SuperFast'}"
 
 new_coq_commit=$(git rev-parse HEAD^2)
 old_coq_commit=$(git merge-base HEAD^1 $new_coq_commit)
-- 
cgit v1.2.3