Skip to content

Commit

Permalink
Update conf-git for Cygwin
Browse files Browse the repository at this point in the history
Native Windows opam already ensures that Git for Windows is installed,
so the depext here is for os = "cygwin" only; on native Windows, git has
already been made a required tool for opam.
  • Loading branch information
dra27 committed Jun 22, 2024
1 parent c51be28 commit 6ca6253
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
4 changes: 4 additions & 0 deletions packages/conf-git/conf-git.1.0/opam
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,10 @@ depexts: [
["git"] {os-distribution = "arch"}
["git"] {os = "macos" & os-distribution = "homebrew"}
["system:git"] {os = "win32" & os-distribution = "cygwinports"}
# This is intentionally os = "cygwin" and *not* os-distribution = "cygwin"
# For native Windows, opam 2.2 recommends Git for Windows, so for native
# Windows there's no need for depext support.
["git"] {os = "cygwin"}
]
synopsis: "Virtual package relying on git"
description:
Expand Down
4 changes: 4 additions & 0 deletions packages/conf-git/conf-git.1.1/opam
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,10 @@ depexts: [
["git"] {os-family = "arch"}
["git"] {os-family = "alpine"}
["system:git"] {os = "win32" & os-distribution = "cygwinports"}
# This is intentionally os = "cygwin" and *not* os-distribution = "cygwin"
# For native Windows, opam 2.2 recommends Git for Windows, so for native
# Windows there's no need for depext support.
["git"] {os = "cygwin"}
["git"] {os = "freebsd"}
]
synopsis: "Virtual package relying on git"
Expand Down

0 comments on commit 6ca6253

Please sign in to comment.