Skip to content

Commit 97fbbac

Browse files
Rewrite absolute external paths into relative paths
Signed-off-by: Marek Kubica <[email protected]>
1 parent a7b65ad commit 97fbbac

File tree

2 files changed

+4
-18
lines changed

2 files changed

+4
-18
lines changed

src/dune_pkg/lock_dir.ml

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1787,18 +1787,9 @@ let loc_in_source_tree loc =
17871787
loc
17881788
|> Loc.map_pos ~f:(fun ({ pos_fname; _ } as pos) ->
17891789
let path = Path.of_string pos_fname in
1790-
match path with
1791-
| External _ | In_source_tree _ -> pos
1792-
| In_build_dir b ->
1793-
(match Path.Build.explode b with
1794-
| ".dev-tools.locks" :: _ ->
1795-
(* we're excluding the hidden dev-tools.locks folders in the build folder
1796-
from rewriting *)
1797-
pos
1798-
| _otherwise ->
1799-
let new_path = in_source_tree path in
1800-
let pos_fname = Path.Source.to_string new_path in
1801-
{ pos with pos_fname }))
1790+
let new_path = in_source_tree path in
1791+
let pos_fname = Path.Source.to_string new_path in
1792+
{ pos with pos_fname })
18021793
;;
18031794

18041795
let check_if_solved_for_platform { solved_for_platforms; _ } ~platform =

test/blackbox-tests/test-cases/pkg/ocamlformat/ocamlformat-dev-tool-fails-to-build.t

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ It fails during the build because of missing OCamlFormat module.
1515
$ DUNE_CONFIG__LOCK_DEV_TOOL=enabled dune fmt
1616
Solution for _build/.dev-tools.locks/ocamlformat:
1717
- ocamlformat.0.26.4
18-
File "$TESTCASE_ROOT/_build/.dev-tools.locks/ocamlformat/ocamlformat.pkg", line 4, characters 6-10:
18+
File "_build/.dev-tools.locks/ocamlformat/ocamlformat.pkg", line 4, characters 6-10:
1919
4 | (run dune build -p %{pkg-self:name} @install))
2020
^^^^
2121
Error: Logs for package ocamlformat
@@ -24,9 +24,4 @@ It fails during the build because of missing OCamlFormat module.
2424
^^^^^^^^^^^
2525
Error: Module "Ocamlformat" doesn't exist.
2626

27-
-> required by
28-
_build/_private/default/.dev-tool/ocamlformat/target/bin/ocamlformat
29-
-> required by _build/default/.formatted/foo.ml
30-
-> required by alias .formatted/fmt
31-
-> required by alias fmt
3227
[1]

0 commit comments

Comments
 (0)