[Nix-dev] Re: [Nix-commits] SVN commit: nix - 13641 - eelco - nix/trunk/src/libstore

[Ludovic Courtès]

Hello!

Eelco Dolstra <e.dolstra at tudelft.nl> writes:

> Author: eelco
> Date: 2008-12-15 23:55:11 +0000 (Mon, 15 Dec 2008)
> New Revision: 13641
>
> You can view the changes in this commit at:
>    https://svn.nixos.org/viewvc/nix?rev=13641&view=rev
>
> Modified:
>    nix/trunk/src/libstore/build.cc
>
> Log:
> * Delete the chroot if it already exists (e.g. left over from an
>   interrupted build).
>
>
> Changes:
>
> Modified: nix/trunk/src/libstore/build.cc
> ===================================================================
> --- nix/trunk/src/libstore/build.cc	2008-12-15 23:54:10 UTC (rev 13640)
> +++ nix/trunk/src/libstore/build.cc	2008-12-15 23:55:11 UTC (rev 13641)
> @@ -1678,6 +1678,7 @@
>             to ensure that we can create hard-links to non-directory
>             inputs in the fake Nix store in the chroot (see below). */
>          chrootRootDir = drvPath + ".chroot";
> +        if (pathExists(chrootRootDir)) deletePath(chrootRootDir);

Does `deletePath ()' follow bind mounts?  Is there any risk like that?

Thanks,
Ludo'.