From fd89f97be97d4f698efd208140062a3f701ef76e Mon Sep 17 00:00:00 2001 From: Eelco Dolstra Date: Tue, 24 Mar 2015 11:50:22 +0100 Subject: [PATCH] Add the closure of store paths to the chroot Thus, for example, to get /bin/sh in a chroot, you only need to specify /bin/sh=${pkgs.bash}/bin/sh in build-chroot-dirs. The dependencies of sh will be added automatically. --- release.nix | 7 +++---- src/libstore/build.cc | 8 ++++++++ 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/release.nix b/release.nix index c5f700d8e..47ba1c1c3 100644 --- a/release.nix +++ b/release.nix @@ -95,11 +95,10 @@ let # Provide a default value for the ‘build-chroot-dirs’ setting # that includes /bin/sh pointing to bash. - preHook = lib.optionalString stdenv.isLinux ( - let sh = stdenv.shell; in + preHook = lib.optionalString stdenv.isLinux '' - export DEFAULT_CHROOT_DIRS="/bin/sh=${sh} $(tr '\n' ' ' < ${writeReferencesToFile sh})" - ''); + export DEFAULT_CHROOT_DIRS="/bin/sh=${stdenv.shell}" + ''; enableParallelBuilding = true; diff --git a/src/libstore/build.cc b/src/libstore/build.cc index 88efb9a65..1c510fa7d 100644 --- a/src/libstore/build.cc +++ b/src/libstore/build.cc @@ -1799,6 +1799,14 @@ void DerivationGoal::startBuilder() } dirsInChroot[tmpDir] = tmpDir; + /* Add the closure of store paths to the chroot. */ + PathSet closure; + for (auto & i : dirsInChroot) + if (isInStore(i.second)) + computeFSClosure(worker.store, toStorePath(i.second), closure); + for (auto & i : closure) + dirsInChroot[i] = i; + string allowed = settings.get("allowed-impure-host-deps", string(DEFAULT_ALLOWED_IMPURE_PREFIXES)); PathSet allowedPaths = tokenizeString(allowed);