From 0d7bc9c3b390602340b9acd9fa8ce16bdede0824 Mon Sep 17 00:00:00 2001 From: Thierry Martinez Date: Sun, 24 May 2020 14:59:45 +0200 Subject: [PATCH 01/12] Fix #183 (pre-ppxlib only): CI testing for our reverse dependencies This should implement @gasche suggestion: https://github.com/ocaml-ppx/ppx_deriving/issues/183 We should do that for `master` as well (I expect there will be more failures there!) but I begin here. --- .travis.yml | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/.travis.yml b/.travis.yml index 1d304f7e..a6c7608a 100644 --- a/.travis.yml +++ b/.travis.yml @@ -5,16 +5,16 @@ install: wget https://raw.githubusercontent.com/ocaml/ocaml-ci-scripts/master/.t script: bash -ex .travis-opam.sh env: matrix: - - OCAML_VERSION=4.02 # we require >=4.02.2 but this picks 4.02.3 - - OCAML_VERSION=4.03 - - OCAML_VERSION=4.04 - - OCAML_VERSION=4.05 - - OCAML_VERSION=4.06 - - OCAML_VERSION=4.07 - - OCAML_VERSION=4.08 - - OCAML_VERSION=4.09 - - OCAML_VERSION=4.10 - - OCAML_VERSION=4.11.0+trunk OCAML_BETA=enable + - REVDEPS=y OCAML_VERSION=4.02 # we require >=4.02.2 but this picks 4.02.3 + - REVDEPS=y OCAML_VERSION=4.03 + - REVDEPS=y OCAML_VERSION=4.04 + - REVDEPS=y OCAML_VERSION=4.05 + - REVDEPS=y OCAML_VERSION=4.06 + - REVDEPS=y OCAML_VERSION=4.07 + - REVDEPS=y OCAML_VERSION=4.08 + - REVDEPS=y OCAML_VERSION=4.09 + - REVDEPS=y OCAML_VERSION=4.10 + - REVDEPS=y OCAML_VERSION=4.11.0+trunk OCAML_BETA=enable PINS="ppx_tools:https://github.com/kit-ty-kate/ppx_tools.git#411" os: - linux From 55510ebcb8f9b834e953c9e6059a1457000e967e Mon Sep 17 00:00:00 2001 From: Thierry Martinez Date: Sun, 24 May 2020 17:42:34 +0200 Subject: [PATCH 02/12] Hand-picked list of dependencies --- .travis.yml | 112 ++++++++++++++++++++++++++++++++++++++++++++++------ 1 file changed, 101 insertions(+), 11 deletions(-) diff --git a/.travis.yml b/.travis.yml index a6c7608a..caa46668 100644 --- a/.travis.yml +++ b/.travis.yml @@ -5,16 +5,106 @@ install: wget https://raw.githubusercontent.com/ocaml/ocaml-ci-scripts/master/.t script: bash -ex .travis-opam.sh env: matrix: - - REVDEPS=y OCAML_VERSION=4.02 # we require >=4.02.2 but this picks 4.02.3 - - REVDEPS=y OCAML_VERSION=4.03 - - REVDEPS=y OCAML_VERSION=4.04 - - REVDEPS=y OCAML_VERSION=4.05 - - REVDEPS=y OCAML_VERSION=4.06 - - REVDEPS=y OCAML_VERSION=4.07 - - REVDEPS=y OCAML_VERSION=4.08 - - REVDEPS=y OCAML_VERSION=4.09 - - REVDEPS=y OCAML_VERSION=4.10 - - REVDEPS=y OCAML_VERSION=4.11.0+trunk OCAML_BETA=enable - PINS="ppx_tools:https://github.com/kit-ty-kate/ppx_tools.git#411" + - OCAML_VERSION=4.02 # we require >=4.02.2 but this picks 4.02.3 + REVDEPS="ppx_deriving_morphism + visitors" + - OCAML_VERSION=4.03 + REVDEPS="ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" + - OCAML_VERSION=4.04 + REVDEPS="ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" + - OCAML_VERSION=4.05 + REVDEPS="ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" + - OCAML_VERSION=4.06 + REVDEPS="ppx_deriving_crowbar + ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" + - OCAML_VERSION=4.07 + REVDEPS="ppx_deriving_hardcaml + ppx_deriving_crowbar + ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" + - OCAML_VERSION=4.08 + REVDEPS="ppx_deriving_hardcaml + ppx_deriving_crowbar + ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" + - OCAML_VERSION=4.09 + REVDEPS="ppx_deriving_hardcaml + ppx_deriving_crowbar + ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" + - OCAML_VERSION=4.10 + REVDEPS="ppx_deriving_hardcaml + ppx_deriving_crowbar + ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" + - OCAML_VERSION=4.11.0+trunk OCAML_BETA=enable + PINS="ppx_tools:https://github.com/kit-ty-kate/ppx_tools.git#411 + ppx_deriving_yojson:https://github.com/thierry-martinez/ppx_deriving_yojson#411_lightning" + REVDEPS="ppx_deriving_hardcaml + ppx_deriving_crowbar + ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" os: - linux From d874295cdf227616f5485596df45709244c63d18 Mon Sep 17 00:00:00 2001 From: Thierry Martinez Date: Sun, 24 May 2020 18:48:50 +0200 Subject: [PATCH 03/12] Try single-line strings --- .travis.yml | 102 ++++++---------------------------------------------- 1 file changed, 11 insertions(+), 91 deletions(-) diff --git a/.travis.yml b/.travis.yml index caa46668..9d73d599 100644 --- a/.travis.yml +++ b/.travis.yml @@ -6,105 +6,25 @@ script: bash -ex .travis-opam.sh env: matrix: - OCAML_VERSION=4.02 # we require >=4.02.2 but this picks 4.02.3 - REVDEPS="ppx_deriving_morphism - visitors" + REVDEPS="ppx_deriving_morphism visitors" - OCAML_VERSION=4.03 - REVDEPS="ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" + REVDEPS="ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" - OCAML_VERSION=4.04 - REVDEPS="ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" + REVDEPS="ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" - OCAML_VERSION=4.05 - REVDEPS="ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" + REVDEPS="ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" - OCAML_VERSION=4.06 - REVDEPS="ppx_deriving_crowbar - ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" + REVDEPS="ppx_deriving_crowbar ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" - OCAML_VERSION=4.07 - REVDEPS="ppx_deriving_hardcaml - ppx_deriving_crowbar - ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" + REVDEPS="ppx_deriving_hardcaml ppx_deriving_crowbar ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" - OCAML_VERSION=4.08 - REVDEPS="ppx_deriving_hardcaml - ppx_deriving_crowbar - ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" + REVDEPS="ppx_deriving_hardcaml ppx_deriving_crowbar ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" - OCAML_VERSION=4.09 - REVDEPS="ppx_deriving_hardcaml - ppx_deriving_crowbar - ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" + REVDEPS="ppx_deriving_hardcaml ppx_deriving_crowbar ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" - OCAML_VERSION=4.10 - REVDEPS="ppx_deriving_hardcaml - ppx_deriving_crowbar - ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" + REVDEPS="ppx_deriving_hardcaml ppx_deriving_crowbar ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" - OCAML_VERSION=4.11.0+trunk OCAML_BETA=enable - PINS="ppx_tools:https://github.com/kit-ty-kate/ppx_tools.git#411 - ppx_deriving_yojson:https://github.com/thierry-martinez/ppx_deriving_yojson#411_lightning" - REVDEPS="ppx_deriving_hardcaml - ppx_deriving_crowbar - ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" + PINS="ppx_tools:https://github.com/kit-ty-kate/ppx_tools.git#411 ppx_deriving_yojson:https://github.com/thierry-martinez/ppx_deriving_yojson#411_lightning" + REVDEPS="ppx_deriving_hardcaml ppx_deriving_crowbar ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_protobuf ppx_deriving_morphism visitors" os: - linux From b936dd82ab9776fbcc90fdc511caa98312ad0a2f Mon Sep 17 00:00:00 2001 From: Thierry Martinez Date: Sun, 24 May 2020 18:55:50 +0200 Subject: [PATCH 04/12] Revert "Try single-line strings" This reverts commit d874295cdf227616f5485596df45709244c63d18. --- .travis.yml | 102 ++++++++++++++++++++++++++++++++++++++++++++++------ 1 file changed, 91 insertions(+), 11 deletions(-) diff --git a/.travis.yml b/.travis.yml index 9d73d599..caa46668 100644 --- a/.travis.yml +++ b/.travis.yml @@ -6,25 +6,105 @@ script: bash -ex .travis-opam.sh env: matrix: - OCAML_VERSION=4.02 # we require >=4.02.2 but this picks 4.02.3 - REVDEPS="ppx_deriving_morphism visitors" + REVDEPS="ppx_deriving_morphism + visitors" - OCAML_VERSION=4.03 - REVDEPS="ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" + REVDEPS="ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" - OCAML_VERSION=4.04 - REVDEPS="ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" + REVDEPS="ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" - OCAML_VERSION=4.05 - REVDEPS="ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" + REVDEPS="ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" - OCAML_VERSION=4.06 - REVDEPS="ppx_deriving_crowbar ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" + REVDEPS="ppx_deriving_crowbar + ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" - OCAML_VERSION=4.07 - REVDEPS="ppx_deriving_hardcaml ppx_deriving_crowbar ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" + REVDEPS="ppx_deriving_hardcaml + ppx_deriving_crowbar + ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" - OCAML_VERSION=4.08 - REVDEPS="ppx_deriving_hardcaml ppx_deriving_crowbar ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" + REVDEPS="ppx_deriving_hardcaml + ppx_deriving_crowbar + ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" - OCAML_VERSION=4.09 - REVDEPS="ppx_deriving_hardcaml ppx_deriving_crowbar ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" + REVDEPS="ppx_deriving_hardcaml + ppx_deriving_crowbar + ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" - OCAML_VERSION=4.10 - REVDEPS="ppx_deriving_hardcaml ppx_deriving_crowbar ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_cmdliner ppx_deriving_protobuf ppx_deriving_morphism visitors" + REVDEPS="ppx_deriving_hardcaml + ppx_deriving_crowbar + ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_cmdliner + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" - OCAML_VERSION=4.11.0+trunk OCAML_BETA=enable - PINS="ppx_tools:https://github.com/kit-ty-kate/ppx_tools.git#411 ppx_deriving_yojson:https://github.com/thierry-martinez/ppx_deriving_yojson#411_lightning" - REVDEPS="ppx_deriving_hardcaml ppx_deriving_crowbar ppx_deriving_yojson ppx_deriving_madcast ppx_deriving_protocol ppx_deriving_rpc ppx_deriving_argparse ppx_deriving_protobuf ppx_deriving_morphism visitors" + PINS="ppx_tools:https://github.com/kit-ty-kate/ppx_tools.git#411 + ppx_deriving_yojson:https://github.com/thierry-martinez/ppx_deriving_yojson#411_lightning" + REVDEPS="ppx_deriving_hardcaml + ppx_deriving_crowbar + ppx_deriving_yojson + ppx_deriving_madcast + ppx_deriving_protocol + ppx_deriving_rpc + ppx_deriving_argparse + ppx_deriving_protobuf + ppx_deriving_morphism + visitors" os: - linux From 476b1e038dd57fe6bfc21adae69a4bb46a0c2688 Mon Sep 17 00:00:00 2001 From: Thierry Martinez Date: Sun, 24 May 2020 18:56:42 +0200 Subject: [PATCH 05/12] Move comment outside node --- .travis.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index caa46668..5eeea40c 100644 --- a/.travis.yml +++ b/.travis.yml @@ -5,7 +5,8 @@ install: wget https://raw.githubusercontent.com/ocaml/ocaml-ci-scripts/master/.t script: bash -ex .travis-opam.sh env: matrix: - - OCAML_VERSION=4.02 # we require >=4.02.2 but this picks 4.02.3 + # we require >=4.02.2 but this picks 4.02.3 + - OCAML_VERSION=4.02 REVDEPS="ppx_deriving_morphism visitors" - OCAML_VERSION=4.03 From 94bd74aef8e6011f4f371b34309df95d92bd79f5 Mon Sep 17 00:00:00 2001 From: Thierry Martinez Date: Sun, 24 May 2020 19:40:38 +0200 Subject: [PATCH 06/12] Forgot .git in ppx_deriving_yojson URL --- .travis.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index 5eeea40c..eab8de9f 100644 --- a/.travis.yml +++ b/.travis.yml @@ -96,7 +96,7 @@ env: visitors" - OCAML_VERSION=4.11.0+trunk OCAML_BETA=enable PINS="ppx_tools:https://github.com/kit-ty-kate/ppx_tools.git#411 - ppx_deriving_yojson:https://github.com/thierry-martinez/ppx_deriving_yojson#411_lightning" + ppx_deriving_yojson:https://github.com/thierry-martinez/ppx_deriving_yojson.git#411_lightning" REVDEPS="ppx_deriving_hardcaml ppx_deriving_crowbar ppx_deriving_yojson From 6dfeead305d06942e6f28662ed36d4deccfee0e0 Mon Sep 17 00:00:00 2001 From: Thierry Martinez Date: Sun, 24 May 2020 20:36:30 +0200 Subject: [PATCH 07/12] Set TESTS=true Suggested by Gabriel Scherer. --- .travis.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.travis.yml b/.travis.yml index eab8de9f..67dc923b 100644 --- a/.travis.yml +++ b/.travis.yml @@ -4,6 +4,8 @@ dist: xenial install: wget https://raw.githubusercontent.com/ocaml/ocaml-ci-scripts/master/.travis-opam.sh script: bash -ex .travis-opam.sh env: + global: + - TESTS=true matrix: # we require >=4.02.2 but this picks 4.02.3 - OCAML_VERSION=4.02 From 4e974c353d6dc2959cd4a72325bb1869628d453e Mon Sep 17 00:00:00 2001 From: Thierry Martinez Date: Sun, 24 May 2020 20:38:50 +0200 Subject: [PATCH 08/12] REVDEPS as global since there is an installability check --- .travis.yml | 94 +++++------------------------------------------------ 1 file changed, 8 insertions(+), 86 deletions(-) diff --git a/.travis.yml b/.travis.yml index 67dc923b..3632ea00 100644 --- a/.travis.yml +++ b/.travis.yml @@ -6,49 +6,6 @@ script: bash -ex .travis-opam.sh env: global: - TESTS=true - matrix: - # we require >=4.02.2 but this picks 4.02.3 - - OCAML_VERSION=4.02 - REVDEPS="ppx_deriving_morphism - visitors" - - OCAML_VERSION=4.03 - REVDEPS="ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" - - OCAML_VERSION=4.04 - REVDEPS="ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" - - OCAML_VERSION=4.05 - REVDEPS="ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" - - OCAML_VERSION=4.06 - REVDEPS="ppx_deriving_crowbar - ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" - - OCAML_VERSION=4.07 REVDEPS="ppx_deriving_hardcaml ppx_deriving_crowbar ppx_deriving_yojson @@ -60,54 +17,19 @@ env: ppx_deriving_protobuf ppx_deriving_morphism visitors" + matrix: + # we require >=4.02.2 but this picks 4.02.3 + - OCAML_VERSION=4.02 + - OCAML_VERSION=4.03 + - OCAML_VERSION=4.04 + - OCAML_VERSION=4.05 + - OCAML_VERSION=4.06 + - OCAML_VERSION=4.07 - OCAML_VERSION=4.08 - REVDEPS="ppx_deriving_hardcaml - ppx_deriving_crowbar - ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" - OCAML_VERSION=4.09 - REVDEPS="ppx_deriving_hardcaml - ppx_deriving_crowbar - ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" - OCAML_VERSION=4.10 - REVDEPS="ppx_deriving_hardcaml - ppx_deriving_crowbar - ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_cmdliner - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" - OCAML_VERSION=4.11.0+trunk OCAML_BETA=enable PINS="ppx_tools:https://github.com/kit-ty-kate/ppx_tools.git#411 ppx_deriving_yojson:https://github.com/thierry-martinez/ppx_deriving_yojson.git#411_lightning" - REVDEPS="ppx_deriving_hardcaml - ppx_deriving_crowbar - ppx_deriving_yojson - ppx_deriving_madcast - ppx_deriving_protocol - ppx_deriving_rpc - ppx_deriving_argparse - ppx_deriving_protobuf - ppx_deriving_morphism - visitors" os: - linux From 7848565ed2fcb49e2b6b6ebc209867b1ca36cc1f Mon Sep 17 00:00:00 2001 From: Thierry Martinez Date: Sun, 24 May 2020 20:39:59 +0200 Subject: [PATCH 09/12] Add pin for ppx_deriving_protobuf --- .travis.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index 3632ea00..1f4f602a 100644 --- a/.travis.yml +++ b/.travis.yml @@ -30,6 +30,7 @@ env: - OCAML_VERSION=4.10 - OCAML_VERSION=4.11.0+trunk OCAML_BETA=enable PINS="ppx_tools:https://github.com/kit-ty-kate/ppx_tools.git#411 - ppx_deriving_yojson:https://github.com/thierry-martinez/ppx_deriving_yojson.git#411_lightning" + ppx_deriving_yojson:https://github.com/thierry-martinez/ppx_deriving_yojson.git#411_lightning + ppx_deriving_protobuf::https://github.com/thierry-martinez/ppx_deriving_protobuf.git#411" os: - linux From 09dc53b4fb8331bc4983645a75dca63418caa2d7 Mon Sep 17 00:00:00 2001 From: Thierry Martinez Date: Sun, 24 May 2020 22:17:57 +0200 Subject: [PATCH 10/12] Remove spurious colon --- .travis.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index 1f4f602a..77d3245d 100644 --- a/.travis.yml +++ b/.travis.yml @@ -31,6 +31,6 @@ env: - OCAML_VERSION=4.11.0+trunk OCAML_BETA=enable PINS="ppx_tools:https://github.com/kit-ty-kate/ppx_tools.git#411 ppx_deriving_yojson:https://github.com/thierry-martinez/ppx_deriving_yojson.git#411_lightning - ppx_deriving_protobuf::https://github.com/thierry-martinez/ppx_deriving_protobuf.git#411" + ppx_deriving_protobuf:https://github.com/thierry-martinez/ppx_deriving_protobuf.git#411" os: - linux From 0e7e2e7eba1c0ca5384562eed3fa308145343641 Mon Sep 17 00:00:00 2001 From: Thierry Martinez Date: Sun, 24 May 2020 22:19:02 +0200 Subject: [PATCH 11/12] Reverse OCAML_VERSION order I think that CI results on recent versions are often those that are the most interesting, so it is better to compute them first. --- .travis.yml | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/.travis.yml b/.travis.yml index 77d3245d..6403e3f2 100644 --- a/.travis.yml +++ b/.travis.yml @@ -18,19 +18,18 @@ env: ppx_deriving_morphism visitors" matrix: - # we require >=4.02.2 but this picks 4.02.3 - - OCAML_VERSION=4.02 - - OCAML_VERSION=4.03 - - OCAML_VERSION=4.04 - - OCAML_VERSION=4.05 - - OCAML_VERSION=4.06 - - OCAML_VERSION=4.07 - - OCAML_VERSION=4.08 - - OCAML_VERSION=4.09 - - OCAML_VERSION=4.10 - OCAML_VERSION=4.11.0+trunk OCAML_BETA=enable PINS="ppx_tools:https://github.com/kit-ty-kate/ppx_tools.git#411 ppx_deriving_yojson:https://github.com/thierry-martinez/ppx_deriving_yojson.git#411_lightning ppx_deriving_protobuf:https://github.com/thierry-martinez/ppx_deriving_protobuf.git#411" + - OCAML_VERSION=4.10 + - OCAML_VERSION=4.09 + - OCAML_VERSION=4.08 + - OCAML_VERSION=4.07 + - OCAML_VERSION=4.06 + - OCAML_VERSION=4.05 + - OCAML_VERSION=4.04 + - OCAML_VERSION=4.03 + - OCAML_VERSION=4.02 # we require >=4.02.2 but this picks 4.02.3 os: - linux From 7364f03518aa3d75c7a0f2a502561349a5856cc5 Mon Sep 17 00:00:00 2001 From: Thierry Martinez Date: Sun, 24 May 2020 23:18:52 +0200 Subject: [PATCH 12/12] Remove TESTS=true It is the default. Here is a pull-request to allow reverse-dependencies to be tested. https://github.com/ocaml/ocaml-ci-scripts/pull/338 --- .travis.yml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/.travis.yml b/.travis.yml index 6403e3f2..8a0b50c0 100644 --- a/.travis.yml +++ b/.travis.yml @@ -5,8 +5,7 @@ install: wget https://raw.githubusercontent.com/ocaml/ocaml-ci-scripts/master/.t script: bash -ex .travis-opam.sh env: global: - - TESTS=true - REVDEPS="ppx_deriving_hardcaml + - REVDEPS="ppx_deriving_hardcaml ppx_deriving_crowbar ppx_deriving_yojson ppx_deriving_madcast