From f11c6d610ce002774df714cf2c998989643dccfa Mon Sep 17 00:00:00 2001 From: Florian Klink Date: Sat, 9 Dec 2023 13:14:16 +0200 Subject: chore(tvix/nix_cli): drop crate The only thing this was doing was invoking nix-store --add, which is not gonna help us populate the tvix-store at all (and we now have `tvix-store import`). This is also (rightfully) causing clippy warnings, because of some fields being unused. It's more of a skeleton, and rather than shelling out to Nix for some usecases, we might introduce a "compatible" Nix CLI frontend for a subset of commands. Drop this for now, to decrease the noise and confusion. Change-Id: I2fd399e9320260f08893b685561755af9c7c961c Reviewed-on: https://cl.tvl.fyi/c/depot/+/10226 Autosubmit: flokli Tested-by: BuildkiteCI Reviewed-by: tazjin --- tvix/nix_cli/Cargo.toml | 18 ------- tvix/nix_cli/default.nix | 5 -- tvix/nix_cli/src/bin/nix-store.rs | 106 -------------------------------------- tvix/nix_cli/src/main.rs | 3 -- 4 files changed, 132 deletions(-) delete mode 100644 tvix/nix_cli/Cargo.toml delete mode 100644 tvix/nix_cli/default.nix delete mode 100644 tvix/nix_cli/src/bin/nix-store.rs delete mode 100644 tvix/nix_cli/src/main.rs (limited to 'tvix/nix_cli') diff --git a/tvix/nix_cli/Cargo.toml b/tvix/nix_cli/Cargo.toml deleted file mode 100644 index f9de2b97706b..000000000000 --- a/tvix/nix_cli/Cargo.toml +++ /dev/null @@ -1,18 +0,0 @@ -[package] -name = "nix-cli" -version = "0.1.0" -edition = "2021" - -# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html - -[dependencies.clap] -version = "4.0" - -[dependencies.tempfile] -version = "3.2.0" - -[[bin]] -name = "nix-store" - -[features] -integration_tests = [] diff --git a/tvix/nix_cli/default.nix b/tvix/nix_cli/default.nix deleted file mode 100644 index 4c79e46cabfa..000000000000 --- a/tvix/nix_cli/default.nix +++ /dev/null @@ -1,5 +0,0 @@ -{ depot, ... }: - -depot.tvix.crates.workspaceMembers.nix-cli.build.override { - runTests = true; -} diff --git a/tvix/nix_cli/src/bin/nix-store.rs b/tvix/nix_cli/src/bin/nix-store.rs deleted file mode 100644 index 1cd64e8d1cee..000000000000 --- a/tvix/nix_cli/src/bin/nix-store.rs +++ /dev/null @@ -1,106 +0,0 @@ -fn main() { - main_args(std::env::args().collect()).unwrap_or_else(|e| e.exit()); -} - -pub fn main_args(args: Vec) -> clap::error::Result { - let matches = clap::Command::new("nix-store") - .subcommand(clap::Command::new("--add").arg(clap::Arg::new("FILE").required(true).index(1))) - .try_get_matches_from(args.iter())?; - if let Some(add) = matches.subcommand_matches("--add") { - let file = add.get_one::("FILE").expect("--add needs a file"); - let file_contents = std::fs::read_to_string(file) - .unwrap_or_else(|_| panic!("file {} does not exist", file)); - Ok(NixResult::FileAddedToStore { - content: file_contents, - }) - } else { - panic!("read some arguments that we do not know: {:?}", args) - } -} - -#[derive(Debug, Eq, PartialEq)] -pub enum NixResult { - FileAddedToStore { content: String }, -} - -#[cfg(test)] -mod integration_tests { - use std::collections::VecDeque; - use std::io::Write; - - use super::*; - - #[derive(Debug)] - enum NixOutput { - #[allow(dead_code)] - Err { - status: i32, - stdout: String, - stderr: String, - }, - Ok { - stdout: String, - stderr: String, - }, - } - - fn run_nix_command(cmd: &str, args: Vec) -> NixOutput { - let out = std::process::Command::new(cmd) - .args(args) - .output() - .unwrap_or_else(|_| panic!("could not run {}", cmd)); - match out.status.code().expect("no status code!") { - 0 => NixOutput::Ok { - stdout: String::from_utf8_lossy(&out.stdout).trim_end().to_string(), - stderr: String::from_utf8_lossy(&out.stderr).trim_end().to_string(), - }, - status => NixOutput::Err { - status, - stdout: String::from_utf8_lossy(&out.stdout).trim_end().to_string(), - stderr: String::from_utf8_lossy(&out.stderr).trim_end().to_string(), - }, - } - } - - fn nix_nix_store<'a>(args: Vec) -> NixResult { - match run_nix_command("nix-store", args) { - err @ NixOutput::Err { .. } => panic!("nix-store --add failed: {:#?}", err), - NixOutput::Ok { stdout, .. } => NixResult::FileAddedToStore { - content: std::fs::read_to_string(&stdout) - .unwrap_or_else(|_| panic!("cannot open {} as store file", stdout)), - }, - } - } - - fn tvix_nix_store<'a>(args: Vec) -> NixResult { - eprintln!("running tvix with arguments {:?}", args); - let mut args = VecDeque::from(args); - args.push_front("tvix-store".to_string()); - super::main_args(Vec::from(args)) - .unwrap_or_else(|e| panic!("clap command line parsing failed:\n{}", e)) - } - - #[test] - #[cfg_attr(not(feature = "integration_tests"), ignore)] - fn test_nix_store_add() { - let file_content = "I am a copied file"; - let mut tempfile = tempfile::NamedTempFile::new().expect("cannot create temp file"); - tempfile - .write_all(file_content.as_bytes()) - .expect("could not write to tempfile"); - assert_eq!( - tvix_nix_store(vec![ - "--add".to_string(), - tempfile.path().as_os_str().to_string_lossy().into_owned() - ]), - nix_nix_store(vec![ - "--add".to_string(), - tempfile.path().as_os_str().to_string_lossy().into_owned() - ]), - "added file contents were not the same" - ); - - // make sure the tempfile lives till here - drop(tempfile) - } -} diff --git a/tvix/nix_cli/src/main.rs b/tvix/nix_cli/src/main.rs deleted file mode 100644 index 40086e6f27ee..000000000000 --- a/tvix/nix_cli/src/main.rs +++ /dev/null @@ -1,3 +0,0 @@ -fn main() { - println!("Hello, tvix!"); -} -- cgit 1.4.1