Octachron / codept

Contextual Ocaml DEPendencies Tool: alternative ocaml dependency analyzer
Other
59 stars 10 forks source link

Split directories on platform separator and / #33

Closed jonahbeckford closed 7 months ago

jonahbeckford commented 7 months ago

Fixes the common use of Cygwin/MSYS2 providing paths to codept on Windows.

Fixes https://github.com/Octachron/codept/issues/31.

All call sites that used [split_on_char] with the platform directory separator have switched to the new [split_on_dirs].

Octachron commented 7 months ago

That does looks like the correct fix, thanks! It might be better to have this behavior configurable later on, but let's first go with a simple fix.