From b3f8d66a6acc4428cb963edda9db20c624dcebdd Mon Sep 17 00:00:00 2001 From: Vincent Ambo Date: Sun, 12 Mar 2023 01:39:31 +0300 Subject: chore(tvix/eval): prune some dependencies & features * We no longer need backtrace-on-stack-overflow, as we no longer overflow the stack with the recent eval refactorings. This was weird voodoo anyways, introduced earlier to debug some cases where stack overflows occured. * default features of genawaiter crate are not needed, as we don't use their proc macros Change-Id: I346fc5a18d7f117ee805909a8be8f535b96be76c Reviewed-on: https://cl.tvl.fyi/c/depot/+/8263 Reviewed-by: flokli Tested-by: BuildkiteCI Reviewed-by: raitobezarius --- tvix/eval/Cargo.toml | 8 ++------ tvix/eval/src/vm/mod.rs | 9 --------- 2 files changed, 2 insertions(+), 15 deletions(-) (limited to 'tvix/eval') diff --git a/tvix/eval/Cargo.toml b/tvix/eval/Cargo.toml index 6bb68834ab93..e59249f1e70a 100644 --- a/tvix/eval/Cargo.toml +++ b/tvix/eval/Cargo.toml @@ -9,12 +9,11 @@ edition = "2021" name = "tvix_eval" [dependencies] -backtrace-on-stack-overflow = { version = "0.2.0", optional = true } builtin-macros = { path = "./builtin-macros", package = "tvix-eval-builtin-macros" } codemap = "0.1.3" codemap-diagnostic = "0.1.1" dirs = "4.0.0" -genawaiter = "0.99.1" +genawaiter = { version = "0.99.1", default_features = false } imbl = { version = "2.0", features = [ "serde" ] } lazy_static = "1.4.0" lexical-core = { version = "0.8.5", features = ["format", "parse-floats"] } @@ -44,7 +43,7 @@ git = "https://github.com/JamesGuthrie/test-generator.git" rev = "82e799979980962aec1aa324ec6e0e4cad781f41" [features] -default = ["impure", "arbitrary", "nix_tests", "backtrace_overflow"] +default = ["impure", "arbitrary", "nix_tests"] # Enables running the Nix language test suite from the original C++ # Nix implementation (at version 2.3) against Tvix. @@ -56,9 +55,6 @@ impure = [] # Enables Arbitrary impls for internal types (required to run tests) arbitrary = ["proptest", "test-strategy", "imbl/proptest"] -# For debugging use only; not appropriate for production use. -backtrace_overflow = ["backtrace-on-stack-overflow"] - [[bench]] name = "eval" harness = false diff --git a/tvix/eval/src/vm/mod.rs b/tvix/eval/src/vm/mod.rs index ca974dff1262..6ecd264d36a7 100644 --- a/tvix/eval/src/vm/mod.rs +++ b/tvix/eval/src/vm/mod.rs @@ -229,15 +229,6 @@ impl<'o> VM<'o> { globals: Rc, reasonable_span: LightSpan, ) -> Self { - // Backtrace-on-stack-overflow is some seriously weird voodoo and - // very unsafe. This double-guard prevents it from accidentally - // being enabled on release builds. - #[cfg(debug_assertions)] - #[cfg(feature = "backtrace_overflow")] - unsafe { - backtrace_on_stack_overflow::enable(); - }; - Self { nix_search_path, io_handle, -- cgit 1.4.1