Go-to-definition (via LSP) fails in merlin sourcetree when symbols are from copied sources

[ManasJayanth]

I used Emacs with Eglot with opam exec -- ocamllsp as the language server command.

In mpipeline.ml when I try to run go-to-definition on Pparse.apply_pp - it fails. If it run it on just Pparse it takes me to the compiler-libs instead of taking me to vendored/patched source files in src/ocaml/driver/pparse.ml

merlin/src/kernel/mpipeline.ml

Lines 240 to 242 in c76379c

	Pparse.apply_pp ~workdir
	~filename:Mconfig.(config.query.filename)
	~source ~pp:workval

additional information

$ opam exec -- ocamllsp --version
1.19.0

$ opam exec -- ocaml --version
The OCaml toplevel, version 5.2.0

[voodoos]

Thanks for your report, I was able to reproduce it.

[voodoos]

So it is due to the fact the the pparse module is copied via a Dune rules and Merlin doesn't use the added line directive to resolve the correct path. Copy directives support is very lacking right now.