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
haxe: add 4.1 and 4.0 compiler versions #118112
Conversation
Result of 2 packages built successfully:
Result of 2 packages built successfully:
1 suggestion:
|
Sorry for the late response! Tested |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tiny nitpick, otherwise looks good IMO.
Unfortunately there's no way to get Haxe 4.0 and 4.1 to work with extlib 1.7.8 (not even without the minimal install), so we need to package 1.7.7 again, at least until 1.7.9 (?) brings backwards compatibility packages, hopefully.
108cd8e
to
aa7cbd0
Compare
@ofborg build haxe_4_0 haxe_4_1 ocamlPackages.ocaml_extlib ocamlPackages.ocaml_extlib-1-7-7 |
aa7cbd0
to
cfc5998
Compare
Unfortunately there's no way to get Haxe 4.0 and 4.1 to work with extlib 1.7.8 (not even without the minimal install), so we need to package 1.7.7 again, at least until 1.7.9 (?) brings backwards compatibility packages, hopefully.
Motivation for this change
Fixes #95638. cc @OPNA2608
Arguably the extlib 1.7.7 override is not ideal, but I don't really want to be stuck with it in
ocamlPackages
as long as we don't need it for anything else. The compatibility patch from 4.2 unfortunately doesn't even apply on 4.1.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)