about summary refs log tree commit diff
path: root/corp
diff options
context:
space:
mode:
authorAdam Joseph <adam@westernsemico.com>2023-09-18T10·47-0700
committerclbot <clbot@tvl.fyi>2023-12-10T14·16+0000
commit2b4920c35582cddf0a8063e7d82d4de2e250732c (patch)
tree1b6dd93a33ffa102c06992c74c1e5708f43d659f /corp
parentefaff795848a5ab20ded471f63bc02e5d1fb916a (diff)
fix(tvix/cli): panic on root cause of the fetchurl(non-boot) bug r/7142
Currently we produce wrong drvPaths for a large number of packages
that use fetchurl (but not fetchurlBoot, which is what stdenv uses).
A simple reproducer is `pkgs.perl538`.

I debugged this down to the root cause, which is the fact that tvix
doesn't realize that the mapping from FOD-paths to outputHash is
*NOT* a 1:1 mapping.  It is a many-to-one mapping.  You can have
lots of different FODs with the same outputHash or even the same
outPath.  For example, perl538.src and perldevel.src use the same
source tarball but a different `version`.

Anyways, I have found the root cause but have run out of time for a
while, so I've added a panic!() to in the spot where we have a logic
bug in order to call it out.

Change-Id: I9766b39cfe2fe7eafec84945b2ad6cc28f9c4b7d
Reviewed-on: https://cl.tvl.fyi/c/depot/+/9364
Reviewed-by: flokli <flokli@flokli.de>
Tested-by: BuildkiteCI
Autosubmit: Adam Joseph <adam@westernsemico.com>
Diffstat (limited to 'corp')
0 files changed, 0 insertions, 0 deletions