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
smlnj: 110.91 -> 110.95, add 64-bit support #74745
Conversation
Hm, this seems to break even the latest revision of |
bad480f
to
2ccdb28
Compare
Fixed |
Also drop isDarwin check in `top-level/all-packages.nix` till darwin build is fixed.
Alright, with all changes accepted by upstream this PR is now ready. |
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.
LGTM
smlnj = if stdenv.isDarwin | ||
then callPackage ../development/compilers/smlnj { } | ||
else pkgsi686Linux.callPackage ../development/compilers/smlnj { }; |
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.
💯
@GrahamcOfBorg build smlnj |
Motivation for this change
Update package, use 64-bit support added in 110.94.
Also drop isDarwin check in
top-level/all-packages.nix
till darwin build is fixed.I needed to patch
heap2exec
for it to run after installation; I filed an upstream bug here.With these changes on
x86_64-linux
only 64-bit version is installed , thoughsmlnj
also supports simultaneous 32 and 64 bit installation. Would it be useful to add support for joint installation?Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @thoughtpolice