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.
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
unison: fix build with ocamlPackages_4_09 #82619
Conversation
Fixes NixOS#61867 and NixOS#61505, bumps the ocaml version unison is built against to 4.09. The patches included here appear in the trunk version of unison, but were not backported to 2.51.2.
Why choosing OCaml 4.09 rather than the default version of OCaml in |
Local and remote instances need to be built with the same (or very close) Ocaml versions to communicate -- it could be good to pin it to a fixed version, but I don't know whether 4.09 is ideal. Debian/Ubuntu are shipping Unison 2.48 w/ Ocaml 4.08, Archlinux is using 4.09. The current default version in We could maybe add an configuration option to specify the Ocaml to use, if there is an easy way to make it select the correct patches. |
Generally speaking, packages in
The patches that you add in this PR fix the build of unison with all versions of OCaml between 4.06 and 4.09 (included). Therefore, I don’t see a good reason to choose a different default version of OCaml than in the rest of |
Co-Authored-By: Vincent Laporte <vbgl@users.noreply.github.com>
Sounds good to me. |
@GrahamcOfBorg build unison usync |
Fixes NixOS#61867 and NixOS#61505, bumps the ocaml version unison is built against to 4.08. The patches included here appear in the trunk version of unison, but were not backported to 2.51.2. (cherry picked from commit 3355e8d)
Merged into 20.03 as 27eca64 Thanks! |
Motivation for this change
Fixes #61867 and #61505, bumps the ocaml version unison is built
against to 4.09. The patches included here appear in the trunk version
of unison, but were not backported to 2.51.2.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)