Skip to content

Commit

Permalink
Fix target for ppx.exe
Browse files Browse the repository at this point in the history
  • Loading branch information
toots committed Nov 12, 2020
1 parent 5e03523 commit 4e5a6a1
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 @@ -72,6 +72,9 @@ Unreleased
- instrumentations backends can now receive arguments via `(instrumentation
(backend <name> <args>))`. (#3906, #3932, @nojb)

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

2.7.1 (2/09/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 @@ -384,6 +384,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 4e5a6a1

Please sign in to comment.