Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Disentangle OpamProcess.resolve_command and OpamSystem.resolve_command #5991

Merged
merged 2 commits into from
Jun 7, 2024
Merged

Disentangle OpamProcess.resolve_command and OpamSystem.resolve_command #5991

merged 2 commits into from
Jun 7, 2024

Commits on Jun 7, 2024

  1. Move OpamSystem.t_resolve_command to OpamStd.Sys 

    Allow OpamStd.Sys to be have access to PATH-resolving machinery from
OpamSystem. This also allows the dependency knot between OpamSystem and
OpamProcess to be solved more simply.
    @dra27 @kit-ty-kate
    dra27 authored and kit-ty-kate committed Jun 7, 2024
    Configuration menu
    Copy the full SHA
    c6b6f80 View commit details
    Browse the repository at this point in the history

  2. Expose OpamStd.Sys.resolve_in_path 

    Exposes the portion of OpamStd.Sys.resolve_command which searches the
PATH environment for a given basename.
    @dra27 @kit-ty-kate
    dra27 authored and kit-ty-kate committed Jun 7, 2024
    Configuration menu
    Copy the full SHA
    38859d6 View commit details
    Browse the repository at this point in the history