about summary refs log tree commit diff
path: root/tvix/eval/src/tests/tvix_tests/eval-okay-eq-nested-list.nix (follow)
AgeCommit message (Collapse)AuthorFilesLines
2022-10-31 r/5226 fix(tvix/eval): nix_eq() must recurseAdam Joseph1-0/+1
The current implementation of nix_eq will force one level of thunks and then switch to the (non-forcing) rust Eq::eq() method. This gives incorrect results for lists-of-thunks. This commit changes nix_eq() to be recursive. A regression test (which fails prior to this commit) is included. This fix also causes nix_tests/eval-okay-fromjson.nix to pass, so it is moved out of notyetpassing. Change-Id: I655fd7a5294208a7b39df8e2c3c12a8b9768292f Signed-off-by: Adam Joseph <adam@westernsemico.com> Reviewed-on: https://cl.tvl.fyi/c/depot/+/7142 Tested-by: BuildkiteCI Reviewed-by: tazjin <tazjin@tvl.su>