diff options
author | Vincent Ambo <mail@tazj.in> | 2022-10-17T14·51+0300 |
---|---|---|
committer | clbot <clbot@tvl.fyi> | 2022-10-18T09·15+0000 |
commit | 13a5e7dd5ba6a5e448390e5ceb7f41825e7593c2 (patch) | |
tree | 9afe684d68521fc71d9273d3508de7f0eb869d15 /tvix/eval/src/tests/tvix_tests/eval-okay-lazy-with.nix | |
parent | 0063e7e913c199538fe67d55e714dd34c09cece3 (diff) |
fix(tvix/eval): wrap dynamic resolution in an extra thunk r/5156
Without this change it was possible to cause situations (see the new test) in which a `with`-namespace was forced prematurely. Change-Id: I879ea7763b43edc693feace2c73c890d426fafd3 Reviewed-on: https://cl.tvl.fyi/c/depot/+/7031 Autosubmit: tazjin <tazjin@tvl.su> Tested-by: BuildkiteCI Reviewed-by: Adam Joseph <adam@westernsemico.com>
Diffstat (limited to 'tvix/eval/src/tests/tvix_tests/eval-okay-lazy-with.nix')
-rw-r--r-- | tvix/eval/src/tests/tvix_tests/eval-okay-lazy-with.nix | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tvix/eval/src/tests/tvix_tests/eval-okay-lazy-with.nix b/tvix/eval/src/tests/tvix_tests/eval-okay-lazy-with.nix index e4377df97eeb..8b1a0191dcc0 100644 --- a/tvix/eval/src/tests/tvix_tests/eval-okay-lazy-with.nix +++ b/tvix/eval/src/tests/tvix_tests/eval-okay-lazy-with.nix @@ -1,6 +1,6 @@ # The 'namespace' of a with should only be evaluated if an identifier # from it is actually accessed. -with (builtins.throw "should not occur"); +with (abort "should not be evaluated"); 42 |