ocaml / opam

opam is a source-based package manager. It supports multiple simultaneous compiler installations, flexible package constraints, and a Git-friendly development workflow.
https://opam.ocaml.org
Other
1.21k stars 348 forks source link

Tweak incorrect case in `--cygwin-extra-packages` #5964

Closed dra27 closed 1 month ago

dra27 commented 1 month ago

An incorrectly translated case in #5930 means opam init with no arguments at the moment unconditionally installs an internal Cygwin! Easy to review based on the diff of the original commit in https://github.com/ocaml/opam/pull/5930/commits/8bc1214c171e588cd3e574c86c798e6a193ca073 - the | `none, None -> None deleted case at L419 has no equivalent in the new code (indeed, no case yields None).

kit-ty-kate commented 1 month ago

Thanks!