JakeHillion/nixpkgs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ocamlPackages.zarith: use buildOcaml instead of mkDerivation

Browse Source 
This has two main advantages:

  - By setting hasSharedObjects = true, buildOcaml will automatically
include a setup-hook.sh that sets CAML_LD_LIBRARY_PATH in dependent
expressions. This is needed to pick up dllzarith.so properly which is
shipped as party of the library.

  - We can kill the ugly assert in the expression and instead change it
to use minimumSupportedOcamlVersion.

Signed-off-by: Austin Seipp <aseipp@pobox.com>
This commit is contained in:
Austin Seipp 2019-01-14 21:50:15 -06:00
parent ee3949d720
commit 9d5a7af4e5
1 changed files with 13 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
pkgs/development/ocaml-modules/zarith/default.nix
View File

@ -1,8 +1,9 @@
{ stdenv, fetchurl, ocaml, findlib, pkgconfig, gmp, perl }:
{ stdenv, buildOcaml, fetchurl
, ocaml, findlib, pkgconfig, perl
, gmp
}:
assert stdenv.lib.versionAtLeast ocaml.version "3.12.1";
let param =
let source =
if stdenv.lib.versionAtLeast ocaml.version "4.02"
then {
version = "1.7";
@ -15,18 +16,20 @@ let param =
};
in
stdenv.mkDerivation rec {
name = "zarith-${version}";
inherit (param) version;
buildOcaml rec {
name = "zarith";
inherit (source) version;
src = fetchurl { inherit (source) url sha256; };
src = fetchurl {
inherit (param) url sha256;
};
minimumSupportedOcamlVersion = "3.12.1";
nativeBuildInputs = [ pkgconfig ];
buildInputs = [ ocaml findlib perl ];
propagatedBuildInputs = [ gmp ];
# needed so setup-hook.sh sets CAML_LD_LIBRARY_PATH for dllzarith.so
hasSharedObjects = true;
patchPhase = "patchShebangs ./z_pp.pl";
configurePhase = ''
./configure -installdir $out/lib/ocaml/${ocaml.version}/site-lib