Skip to content

Commit

Permalink
Hackish workaround for ocurrent/obuilder#77
Browse files Browse the repository at this point in the history
  • Loading branch information
kit-ty-kate committed Jan 27, 2023
1 parent 8292a87 commit f56091f
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion lib/opam_build.ml
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,16 @@ let opam_install ~variant ~opam_version ~pin ~lower_bounds ~with_tests ~pkg =
[]
) @ [
run ~network "opam %s || true" (match opam_version with `V2_1 | `Dev -> "update --depexts" | `V2_0 -> "depext -u");
] @
(if with_tests then [
(* TODO: Remove this hack when https://github.com/ocurrent/obuilder/issues/77 is fixed *)
(* NOTE: This hack will fail for packages that have src: "git+https://..." *)
run ~cache ~network "((%sopam reinstall --with-test %s) > /dev/null 2>&1 ; echo \"exit code: $?\") || true"
(match opam_version with `V2_1 | `Dev -> "" | `V2_0 -> fmt "opam depext%s %s && " with_tests_opt pkg) pkg
] else []) @ [
(* TODO: Replace by two calls to opam install + opam install -t using the OPAMDROPINSTALLEDPACKAGES feature *)
run ~cache ~network
(* NOTE: See above for the ~network:(if with_tests ...) hack *)
run ~cache ~network:(if with_tests then [] else network)
{|%sopam reinstall%s %s;
res=$?;
test "$res" != 31 && exit "$res";
Expand Down

0 comments on commit f56091f

Please sign in to comment.