From f7f5f21e4664a423eeb5b04dee90555a261490a6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Gr=C3=A9goire=20Henry?= <gregoire.henry@ocamlpro.com>
Date: Thu, 8 Dec 2016 16:27:28 +0100
Subject: [PATCH] Fix `make build-deps`

The amended script is now able to handle new dependency in the opam file.
It also abloe to handle `test` attributes.
---
 scripts/install_build_deps.sh | 11 ++++++-----
 src/tezos-deps.opam           |  4 ++--
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/scripts/install_build_deps.sh b/scripts/install_build_deps.sh
index c9bd2ef3e..846306d67 100755
--- a/scripts/install_build_deps.sh
+++ b/scripts/install_build_deps.sh
@@ -1,5 +1,6 @@
 #! /bin/sh
 
+
 OCAML_VERSION=4.03.0
 if [ "$(ocaml -vnum)" != "${OCAML_VERSION}" ]; then
   echo ;
@@ -42,18 +43,18 @@ if [ ! -z "$pin" ] ; then
     opam pin --yes add --no-action --dev-repo ocp-ocamlres
     opam pin --yes add --no-action --dev-repo ocplib-json-typed
     opam pin --yes add --no-action --dev-repo ocplib-resto
+    ## Force opam to take account of the new `tezos-deps.opam`
+    opam pin --yes remove tezos-deps
     opam pin --yes add --no-action tezos-deps src
 fi
 
 if [ ! -z "$depext" ] ; then
+    ## In our CI, this rule is executed as user 'root'
+    ## The other rules are executed as user 'opam'.
     opam list --installed depext || opam install depext
     opam depext ${DEPEXTOPT} tezos-deps
 fi
 
 if [ ! -z "$install" ] ; then
-    if opam list --installed tezos-deps ; then
-	opam upgrade $(opam list -s --required-by tezos-deps | grep -ve '^ocaml *$')
-    else
-	opam install tezos-deps
-    fi
+    opam install --build-test tezos-deps
 fi
diff --git a/src/tezos-deps.opam b/src/tezos-deps.opam
index 102b625b3..cd232cf8b 100644
--- a/src/tezos-deps.opam
+++ b/src/tezos-deps.opam
@@ -31,6 +31,6 @@ depends: [
   "tyxml"
   "js_of_ocaml"
   "sodium" {>= "0.3.0"}
-  "kaputt"     (* only for testing *)
-  "bisect_ppx" (* only for testing *)
+  "kaputt" { test }
+  "bisect_ppx" { test }
 ]