From 9cb3cd10f5159a163ea494f765c583a389324c2d Mon Sep 17 00:00:00 2001 From: Adam Joseph Date: Wed, 13 Dec 2023 05:02:01 -0800 Subject: fix(tvix/eval): propagate catchables through builtins.intersectAttrs Change-Id: I4ada8cf10611e98519cb1b1da11bfd06815f2932 Reviewed-on: https://cl.tvl.fyi/c/depot/+/10358 Autosubmit: Adam Joseph Reviewed-by: sterni Tested-by: BuildkiteCI --- tvix/eval/src/builtins/mod.rs | 6 ++++++ .../src/tests/tvix_tests/eval-okay-catchable-in-intersectattrs.exp | 1 + .../src/tests/tvix_tests/eval-okay-catchable-in-intersectattrs.nix | 1 + 3 files changed, 8 insertions(+) create mode 100644 tvix/eval/src/tests/tvix_tests/eval-okay-catchable-in-intersectattrs.exp create mode 100644 tvix/eval/src/tests/tvix_tests/eval-okay-catchable-in-intersectattrs.nix (limited to 'tvix/eval') diff --git a/tvix/eval/src/builtins/mod.rs b/tvix/eval/src/builtins/mod.rs index 6ecbb50284..d75249e94a 100644 --- a/tvix/eval/src/builtins/mod.rs +++ b/tvix/eval/src/builtins/mod.rs @@ -576,6 +576,12 @@ mod pure_builtins { #[builtin("intersectAttrs")] async fn builtin_intersect_attrs(co: GenCo, x: Value, y: Value) -> Result { + if x.is_catchable() { + return Ok(x); + } + if y.is_catchable() { + return Ok(y); + } let left_set = x.to_attrs()?; if left_set.is_empty() { return Ok(Value::attrs(NixAttrs::empty())); diff --git a/tvix/eval/src/tests/tvix_tests/eval-okay-catchable-in-intersectattrs.exp b/tvix/eval/src/tests/tvix_tests/eval-okay-catchable-in-intersectattrs.exp new file mode 100644 index 0000000000..c508d5366f --- /dev/null +++ b/tvix/eval/src/tests/tvix_tests/eval-okay-catchable-in-intersectattrs.exp @@ -0,0 +1 @@ +false diff --git a/tvix/eval/src/tests/tvix_tests/eval-okay-catchable-in-intersectattrs.nix b/tvix/eval/src/tests/tvix_tests/eval-okay-catchable-in-intersectattrs.nix new file mode 100644 index 0000000000..cd4d62d05f --- /dev/null +++ b/tvix/eval/src/tests/tvix_tests/eval-okay-catchable-in-intersectattrs.nix @@ -0,0 +1 @@ +(builtins.tryEval (builtins.intersectAttrs (throw "fred") {})).success -- cgit 1.4.1