Skip to content

Commit

Permalink
Fix target for ppx.exe
Browse files Browse the repository at this point in the history
Signed-off-by: Romain Beauxis <[email protected]>
  • Loading branch information
toots committed Aug 29, 2020
1 parent 7a6fa37 commit 86a69da
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
3 changes: 3 additions & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@ Unreleased
- Fix `dune-package` installation when META templates are present (#3743, fixes
#3746, @rgrinberg)

- Fix `ppx.exe` being compiled for the wrong target when cross-compiling
(#3698, @toots)

2.7.0 (13/08/2020)
------------------

Expand Down
3 changes: 3 additions & 0 deletions src/dune_rules/preprocessing.ml
Original file line number Diff line number Diff line change
Expand Up @@ -395,6 +395,9 @@ let gen_rules sctx components =

let ppx_driver_exe sctx libs =
let key = Digest.to_string (Key.Decoded.of_libs libs |> Key.encode) in
(* Make sure to compile ppx.exe for the compiling host. See: #2252, #2286 and
#3698 *)
let sctx = SC.host sctx in
ppx_exe sctx ~key

let get_cookies ~loc ~expander ~lib_name libs =
Expand Down

0 comments on commit 86a69da

Please sign in to comment.