ocaml-community / utop

Universal toplevel for OCaml
Other
833 stars 110 forks source link

expunge: move system code to dune #439

Closed emillon closed 1 year ago

emillon commented 1 year ago

The logic to create utop from utop-full is as follows:

Previously, this was all done in expunge/expunge.ml, meaning core logic

Besides simplicity and portability, the other advantage is that the intermediate modules.txt can be inspected directly.

emillon commented 1 year ago

this works with sandboxing, and I confirmed (after fixing a typo) that it produces the same binary,