maurice/snix
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

docs(glue/snix_build): document why /nix/store is scratch

Browse source 
Even without nix/store in here, all output paths need to be write-able.

Change-Id: Ibeeba503844dee78de11fd2aa79b3ad207795059
Reviewed-on: https://cl.snix.dev/c/snix/+/30542
Tested-by: besadii
Autosubmit: Florian Klink <flokli@flokli.de>
Reviewed-by: Vova Kryachko <v.kryachko@gmail.com>
This commit is contained in:
Florian Klink 2025-05-28 17:25:32 +03:00 • committed by clbot
parent 688973ad78
commit 80f5b5c44e
1 changed files with 9 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

10
snix/glue/src/snix_build.rs
View file

@ -189,7 +189,15 @@ pub(crate) fn derivation_to_build_request(
inputs_dir: nix_compat::store_path::STORE_DIR[1..].into(), inputs_dir: nix_compat::store_path::STORE_DIR[1..].into(),
constraints, constraints,
working_dir: "build".into(), working_dir: "build".into(),
scratch_paths: vec!["build".into(), "nix/store".into()], scratch_paths: vec![
"build".into(),
// This is in here because Nix allows you to do
// `pkgs.runCommand "foo" {} "mkdir -p $out;touch /nix/store/aaaa"`
// (throwing away the /nix/store/aaaa post-build),
// not because it's a sane thing to do.
// FUTUREWORK: check if nothing exploits this.
"nix/store".into(),
],
additional_files: additional_files additional_files: additional_files
.into_iter() .into_iter()
.map(|(path, contents)| AdditionalFile { .map(|(path, contents)| AdditionalFile {