Skip to content

Commit

Permalink
Pin dune to work around compiler options split
Browse files Browse the repository at this point in the history
ocaml/ocaml@51e5cf2 split the compiler
flags for native and bytecode compiler to separate flags. It also
changes the flag names which breaks builds of several packages. This
commit is a workaround for that change.
  • Loading branch information
punchagan committed Jun 5, 2024
1 parent 5f17123 commit 77be892
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -109,8 +109,13 @@ ifeq (1, $(USE_SYS_DUNE_HACK))
opam repo add upstream "git+https://github.com/ocaml/opam-repository.git" --on-switch=$(CONFIG_SWITCH_NAME) --rank 2
opam install --switch=$(CONFIG_SWITCH_NAME) --yes ocamlfind
opam install --switch=$(CONFIG_SWITCH_NAME) --yes "dune.$(SANDMARK_DUNE_VERSION)" "dune-configurator.$(SANDMARK_DUNE_VERSION)"
# Pin the version so it doesn't change when installing packages
opam pin add --switch=$(CONFIG_SWITCH_NAME) --yes -n dune "$(SANDMARK_DUNE_VERSION)"
@{ case "$*" in \
5.3.*) \
echo "Pinning dune to fixed version to work around binary and native compiler options split"; \
opam pin add --yes --switch $* dune."$(SANDMARK_DUNE_VERSION)" https://github.com/punchagan/dune.git#fix-compiler-opts;; \
*) \
opam pin add --switch=$(CONFIG_SWITCH_NAME) --yes -n dune "$(SANDMARK_DUNE_VERSION)" ;; \
esac };
endif

ocamls=$(wildcard ocaml-versions/*.json)
Expand Down

0 comments on commit 77be892

Please sign in to comment.