From 33ba7ba5ecc7baf3f4b58f47fb2570d23106ddb7 Mon Sep 17 00:00:00 2001 From: Jonah Beckford <9566106-jonahbeckford@users.noreply.gitlab.com> Date: Sat, 14 Sep 2024 13:06:44 -0700 Subject: [PATCH] Remove ppx_expect since unavailable on 32-bit archs (3/n) - Remove --deps-only which is buggy. That, at least in opam 2.2.1, uses the dependencies from the repository rather than the local package when doing opam install . --deps-only --- .github/workflows/pages.yml | 2 +- DEVELOPMENT.md | 2 +- ci/build-test.sh | 3 ++- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/.github/workflows/pages.yml b/.github/workflows/pages.yml index cbb003a..c642ae5 100644 --- a/.github/workflows/pages.yml +++ b/.github/workflows/pages.yml @@ -23,7 +23,7 @@ jobs: opam-pin: true opam-depext: false # will complain that diskuvbox is missing - name: Install Opam dependencies - run: opam install . --deps-only --with-doc + run: opam install . --with-doc - name: Cache conda uses: actions/cache@v3 diff --git a/DEVELOPMENT.md b/DEVELOPMENT.md index 6e80714..de11293 100644 --- a/DEVELOPMENT.md +++ b/DEVELOPMENT.md @@ -6,7 +6,7 @@ This is the Windows prerequisite for any development listed on this page: ```powershell dkml init -opam install . --deps-only --with-test --yes +opam install . --with-test --yes ``` On Unix: diff --git a/ci/build-test.sh b/ci/build-test.sh index 876e8a1..94ccc5e 100755 --- a/ci/build-test.sh +++ b/ci/build-test.sh @@ -30,5 +30,6 @@ opamrun exec -- ocamlc -config opamrun update # Make your own build logic! It may look like ... -opamrun install . --deps-only --with-test --yes +# note: --deps-only is buggy. That, at least in opam 2.2.1, uses the dependencies from the repository rather than the local package when doing opam install . --deps-only +opamrun install . --with-test --yes opamrun exec -- dune runtest