maurice/snix
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

refactor(tvix/store/bin): don't reconstruct StorePath twice

Browse source 
We already have that data in output_path.

Change-Id: Iadf615551f0b44d539098446ff5926994deb606d
Reviewed-on: https://cl.tvl.fyi/c/depot/+/10508
Tested-by: BuildkiteCI
Autosubmit: flokli <flokli@flokli.de>
Reviewed-by: raitobezarius <tvl@lahfa.xyz>
This commit is contained in:
Florian Klink 2023-12-31 18:23:02 +02:00 • committed by clbot
parent fa335aaa68
commit 20c98a51bf
1 changed files with 1 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

9
tvix/store/src/bin/tvix-store.rs
View file

@ -2,7 +2,6 @@ use clap::Subcommand;
use data_encoding::BASE64; use data_encoding::BASE64;
use futures::future::try_join_all; use futures::future::try_join_all;
use nix_compat::store_path; use nix_compat::store_path;
use nix_compat::store_path::StorePath;
use std::path::Path; use std::path::Path;
use std::path::PathBuf; use std::path::PathBuf;
use std::sync::Arc; use std::sync::Arc;
@ -17,7 +16,6 @@ use tvix_castore::proto::directory_service_server::DirectoryServiceServer;
use tvix_castore::proto::node::Node; use tvix_castore::proto::node::Node;
use tvix_castore::proto::GRPCBlobServiceWrapper; use tvix_castore::proto::GRPCBlobServiceWrapper;
use tvix_castore::proto::GRPCDirectoryServiceWrapper; use tvix_castore::proto::GRPCDirectoryServiceWrapper;
use tvix_castore::proto::NamedNode;
use tvix_store::pathinfoservice::PathInfoService; use tvix_store::pathinfoservice::PathInfoService;
use tvix_store::proto::nar_info; use tvix_store::proto::nar_info;
use tvix_store::proto::path_info_service_server::PathInfoServiceServer; use tvix_store::proto::path_info_service_server::PathInfoServiceServer;
@ -321,12 +319,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
log_node(&node, &path); log_node(&node, &path);
println!( println!("{}", output_path.to_absolute_path());
"{}",
StorePath::from_bytes(node.get_name())
.unwrap()
.to_absolute_path()
);
Ok(()) Ok(())
} }