diff options
author | Florian Klink <flokli@flokli.de> | 2024-01-16T19·14+0200 |
---|---|---|
committer | flokli <flokli@flokli.de> | 2024-02-18T08·59+0000 |
commit | c6605992c06bf6adc93a7bb19b0ed34b8c213bf3 (patch) | |
tree | 1cbc0564aca6ac44011a90a77f7bf77e3d729b15 /tvix/Cargo.nix | |
parent | 6b22d4e1faec2b81173fc7029a5107cbe7db1a50 (diff) |
feat(tvix/glue): drive builds on IO r/7551
That's one possible abstraction to drive builds. Whenever IO into a store path is requested, we look up the root node, and in case we don't have it in PathInfoService, but KnownPaths gives us a Derivation for that output path, trigger a build and await the result. This recursively might trigger builds for parent paths if they haven't been built yet. Another option would be to simply expose a PathInfoService interface for a builder too, and loop all building into IO via PathInfoService composition - but let's start with something. Note tvix-cli doesn't have a configurable BuildService yet, it's plugged to the DummyBuildService, so whenever it needs to do a build, it'll fail, but that's how it can be provoked: ``` (builtins.readFile (import <nixpkgs> {}).hello.outPath + "/bin/hello") […] error[E029]: I/O error: /nix/store/cg8a576pz2yfc1wbhxm1zy4x7lrk8pix-hello-2.12.1: builds are not supported with DummyBuildService --> [code]:1:2 | 1 | (builtins.readFile (import <nixpkgs> {}).hello.outPath + "/bin/hello") | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ``` Note how this fails, while pure output path calculation (`(import <nixpkgs> {}).hello.outPath + "/bin/hello")`) still succeeds. Change-Id: Id2075d8a2b18554d0dd608b4b29146a8cd411e7f Reviewed-on: https://cl.tvl.fyi/c/depot/+/10793 Reviewed-by: raitobezarius <tvl@lahfa.xyz> Tested-by: BuildkiteCI
Diffstat (limited to 'tvix/Cargo.nix')
-rw-r--r-- | tvix/Cargo.nix | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/tvix/Cargo.nix b/tvix/Cargo.nix index 28ef774ac71f..a39c8ef38d54 100644 --- a/tvix/Cargo.nix +++ b/tvix/Cargo.nix @@ -10619,6 +10619,10 @@ rec { else ./glue; dependencies = [ { + name = "async-recursion"; + packageId = "async-recursion"; + } + { name = "bstr"; packageId = "bstr"; } @@ -10631,6 +10635,10 @@ rec { packageId = "data-encoding"; } { + name = "futures"; + packageId = "futures"; + } + { name = "nix-compat"; packageId = "nix-compat"; } |