From d07107d68864fe15cf3eef4ca8f299a8fe56e8b5 Mon Sep 17 00:00:00 2001 From: Florian Klink Date: Thu, 20 Mar 2025 12:14:44 +0000 Subject: [PATCH] fix(snix/build): use try_into_name_and_node() The BuildRequest trait currently still uses the proto Build struct as a response type, and that one uses nodes with names, causing .try_into_anonymous_node() to panic. Use try_into_name_and_node(), and discard the name for now. We should update the trait to use a stricter type for this. Change-Id: Ib183a163e885443c58c42808e464d0611eaae324 Reviewed-on: https://cl.snix.dev/c/snix/+/30169 Reviewed-by: Vova Kryachko Tested-by: besadii Autosubmit: Florian Klink --- snix/glue/src/snix_store_io.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/snix/glue/src/snix_store_io.rs b/snix/glue/src/snix_store_io.rs index ecc5e2944..830991f63 100644 --- a/snix/glue/src/snix_store_io.rs +++ b/snix/glue/src/snix_store_io.rs @@ -291,9 +291,9 @@ impl SnixStoreIO { .zip(build_result.outputs_needles.iter()) .zip(drv.outputs.iter()) { - let output_node = output + let (_, output_node) = output .clone() - .try_into_anonymous_node() + .try_into_name_and_node() .expect("invalid node"); let output_needles: Vec<_> = output_needles