Merge pull request #47 from victor-dumitrescu/master

Make blst dependency optional

CHANGELOG.md

@@ -7,7 +7,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ### Added
 
-- Nothing.
+- Added `bls` feature flag to allow disabling dependency on `blst`
 
 ### Changed
 

crypto/Cargo.toml

@@ -27,13 +27,14 @@ strum_macros = "0.20"
 zeroize = { version = "1.5" }
 ed25519-dalek = { version = "2.0.0", default-features = false }
 cryptoxide = { version = "0.4.4", default-features = false, features = ["sha2", "blake2"] }
-blst = "=0.3.10"
+blst = { version = "=0.3.10", optional = true }
 
 proptest = { version = "1.1", optional = true }
 
 [dev-dependencies]
 serde_json = "1.0"
 
 [features]
-default = ["std"]
+default = ["std", "bls"]
+bls = ["blst"]
 std = ["rand/std", "num-bigint/rand", "libsecp256k1/std", "p256/std", "proptest"]

crypto/src/bls.rs

@@ -1,9 +1,12 @@
 // SPDX-FileCopyrightText: 2022-2023 TriliTech <[email protected]>
+// SPDX-FileCopyrightText: 2023 Nomadic Labs <[email protected]>
 //
 // SPDX-License-Identifier: MIT
 
 //! BLS support (min_pk).
 
+#![cfg(feature = "bls")]
+
 use crate::hash::BlsSignature;
 use crate::hash::ContractTz4Hash;
 use crate::hash::PublicKeyBls;

crypto/src/lib.rs

@@ -1,5 +1,6 @@
 // Copyright (c) SimpleStaking, Viable Systems and Tezedge Contributors
 // SPDX-CopyrightText: 2023 TriliTech <[email protected]>
+// SPDX-CopyrightText: 2023 Nomadic Labs <[email protected]>
 // SPDX-License-Identifier: MIT
 #![forbid(unsafe_code)]
 #![cfg_attr(feature = "fuzzing", feature(no_coverage))]
@@ -9,6 +10,7 @@ use thiserror::Error;
 #[macro_use]
 pub mod blake2b;
 pub mod base58;
+#[cfg(feature = "bls")]
 pub mod bls;
 #[macro_use]
 pub mod hash;

tezos-encoding/Cargo.toml

@@ -30,7 +30,7 @@ default-features = false
 path = "../tezos-encoding-derive"
 version = "0.5.1"
 
-[features]
-
 [dev-dependencies]
 serde_json = "1.0"
+
+[features]