From 1bccee0878ffc80efe8741afdb5793ef9105aa35 Mon Sep 17 00:00:00 2001
From: DaniPopes <57450786+DaniPopes@users.noreply.github.com>
Date: Wed, 3 Jul 2024 21:47:17 +0200
Subject: [PATCH] fix(rust): define NDEBUG when debug assertions are off (#450)

---
 bindings/rust/build.rs   | 9 ++++-----
 bindings/rust/src/lib.rs | 2 +-
 2 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/bindings/rust/build.rs b/bindings/rust/build.rs
index 1b19f71ac..cd07487c5 100644
--- a/bindings/rust/build.rs
+++ b/bindings/rust/build.rs
@@ -25,20 +25,19 @@ fn main() {
     cc.include(blst_headers_dir.clone());
     cc.warnings(false);
     cc.file(c_src_dir.join("c_kzg_4844.c"));
+    #[cfg(not(debug_assertions))]
+    cc.define("NDEBUG", None);
 
     cc.try_compile("ckzg").expect("Failed to compile ckzg");
 
     #[cfg(feature = "generate-bindings")]
     {
         let header_path = c_src_dir.join("c_kzg_4844.h");
-        let bindings_out_path = concat!(
-            env!("CARGO_MANIFEST_DIR"),
-            "/bindings/rust/src/bindings/generated.rs"
-        );
+        let bindings_out_path = root_dir.join("bindings/rust/src/bindings/generated.rs");
         make_bindings(
             header_path.to_str().expect("valid header path"),
             blst_headers_dir.to_str().expect("valid blst header path"),
-            bindings_out_path.as_ref(),
+            &bindings_out_path,
         );
     }
 
diff --git a/bindings/rust/src/lib.rs b/bindings/rust/src/lib.rs
index bbc79f0e5..223f67767 100644
--- a/bindings/rust/src/lib.rs
+++ b/bindings/rust/src/lib.rs
@@ -19,8 +19,8 @@ pub use bindings::{
     C_KZG_RET as CkzgError,
 };
 
-#[cfg(feature = "ethereum_kzg_settings")]
 // Expose the default settings.
+#[cfg(feature = "ethereum_kzg_settings")]
 pub use ethereum_kzg_settings::{ethereum_kzg_settings, ethereum_kzg_settings_arc};
 
 // Expose the constants.