Skip to content

Commit

Permalink
Install rsync to the internal Cygwin installation
Browse files Browse the repository at this point in the history
The VCS backends all require rsync
  • Loading branch information
dra27 committed Jan 28, 2024
1 parent 73c671f commit 1b1172e
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions master_changes.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ users)
## Plugins

## Init
* Add rsync package to internal Cygwin packages list (enables local pinning and is used by the VCS backends [#5808 @dra27]

## Config report

Expand Down
1 change: 1 addition & 0 deletions src/client/opamInitDefaults.ml
Original file line number Diff line number Diff line change
Expand Up @@ -154,6 +154,7 @@ let required_packages_for_cygwin =
"patch";
"tar";
"unzip";
"rsync";
] |> List.map OpamSysPkg.of_string

let init_scripts () = [
Expand Down
2 changes: 1 addition & 1 deletion src/state/opamSysInteract.ml
Original file line number Diff line number Diff line change
Expand Up @@ -932,7 +932,7 @@ let install_packages_commands_t ?(env=OpamVariable.Map.empty) config sys_package
|> OpamStd.String.Set.elements);
`AsUser "rpm", "-q"::"--whatprovides"::packages], None
| Cygwin ->
(* We use setp_x86_64 to install package instead of `cygcheck` that is
(* We use setup_x86_64 to install package instead of `cygcheck` that is
stored in `sys-pkg-manager-cmd` field *)
[`AsUser (OpamFilename.to_string (Cygwin.cygsetup ())),
[ "--root"; (OpamFilename.Dir.to_string (Cygwin.cygroot config));
Expand Down

0 comments on commit 1b1172e

Please sign in to comment.