From 0f8356544c25e5fc17afcdd7598dcbea63d93da9 Mon Sep 17 00:00:00 2001 From: Stefan Frijters Date: Mon, 23 Mar 2026 11:46:06 +0100 Subject: [PATCH] rocqPackages.mkRocqDerivation: move env variables into env --- pkgs/build-support/rocq/default.nix | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/pkgs/build-support/rocq/default.nix b/pkgs/build-support/rocq/default.nix index a6b751a53b23..7a7c5fba7b25 100644 --- a/pkgs/build-support/rocq/default.nix +++ b/pkgs/build-support/rocq/default.nix @@ -98,6 +98,7 @@ let "dropAttrs" "dropDerivationAttrs" "keepAttrs" + "env" ] ++ dropAttrs ) keepAttrs; @@ -162,6 +163,7 @@ let "COQPLUGININSTALL=$(OCAMLFIND_DESTDIR)" ]; docdir-flags = [ "COQDOCINSTALL=$(out)/share/coq/${rocq-core.rocq-version}/user-contrib" ]; + COQUSERCONTRIB = "$out/lib/coq/${rocq-core.rocq-version}/user-contrib"; in stdenv.mkDerivation ( @@ -187,6 +189,15 @@ stdenv.mkDerivation ( args.overrideBuildInputs or ([ rocq-core ] ++ (args.buildInputs or [ ]) ++ extraBuildInputs); inherit enableParallelBuilding; + env = + optionalAttrs setROCQBIN { + ROCQBIN = "${rocq-core}/bin/"; + } + // optionalAttrs (args ? useMelquiondRemake) { + inherit COQUSERCONTRIB; + } + // (args.env or { }); + meta = ( { @@ -208,7 +219,6 @@ stdenv.mkDerivation ( // (args.meta or { }); } - // (optionalAttrs setROCQBIN { ROCQBIN = "${rocq-core}/bin/"; }) // (optionalAttrs (!args ? installPhase && !args ? useMelquiondRemake) { installFlags = rocqlib-flags ++ docdir-flags ++ extraInstallFlags; }) @@ -226,8 +236,7 @@ stdenv.mkDerivation ( runHook postInstall ''; }) - // (optionalAttrs (args ? useMelquiondRemake) rec { - COQUSERCONTRIB = "$out/lib/coq/${rocq-core.rocq-version}/user-contrib"; + // (optionalAttrs (args ? useMelquiondRemake) { preConfigurePhases = [ "autoconf" ]; configureFlags = [ "--libdir=${COQUSERCONTRIB}/${useMelquiondRemake.logpath or ""}" ]; buildPhase = "./remake -j$NIX_BUILD_CORES";