ocaml / tuareg

Emacs OCaml mode
GNU General Public License v3.0
360 stars 79 forks source link

ocamldebug.el: support setting breakpoints in dune "wrapped" libraries #246

Closed SkySkimmer closed 11 months ago

SkySkimmer commented 3 years ago

The module name needs to be changed from "foo" to "wrapper__Foo".

Only works when there is a single "library" stanza in the dune file as I didn't want to have to figure out which file belongs to which library.

bbatsov commented 2 years ago

Might be good to document the proposed behavior somewhere (and it's limitations), as I doubt that everyone will be expecting it. The code itself looks reasonable to me.

monnier commented 2 years ago

I'm not sure if seq.el was available in Emacs 24.4 (which tuareg currently targets). Perhaps it should be added an explicit package dep?

seq was added to Emacs in Emacs-25, indeed.