Skip to content

Commit 3fbadfe

Browse files
authored
refactor(utop): reuse traversal build directory (#16309)
Resolve the build directory once while collecting libraries and PPX rewriters for Utop. Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
1 parent 767d608 commit 3fbadfe

1 file changed

Lines changed: 2 additions & 5 deletions

File tree

src/dune_rules/utop.ml

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -109,17 +109,14 @@ let libs_and_ppx_under_dir sctx ~db ~dir =
109109
>>= function
110110
| None -> Memo.return ([], [])
111111
| Some dir ->
112+
let build_dir = Context.build_dir (Super_context.context sctx) in
112113
let+ libs, pps =
113114
Source_tree_map_reduce.map_reduce
114115
dir
115116
~traverse:Source_dir_status.Set.all
116117
~trace_event_name:"Utop rules loading"
117118
~f:(fun dir ->
118-
let dir =
119-
Path.Build.append_source
120-
(Context.build_dir (Super_context.context sctx))
121-
(Source_tree.Dir.path dir)
122-
in
119+
let dir = Path.Build.append_source build_dir (Source_tree.Dir.path dir) in
123120
Dune_load.stanzas_in_dir dir
124121
>>= function
125122
| None -> Memo.return Libs_and_ppxs.empty

0 commit comments

Comments
 (0)