refactor: move schema under #[cfg(schema)] in workspace

.github/test.sh

@@ -6,6 +6,7 @@ pushd borsh
 cargo test --no-run
 cargo test
 cargo test --no-default-features
+cargo test --no-default-features --features schema
 cargo test --no-default-features --features hashbrown,rc
 cargo test --features rc
 popd

borsh-derive/Cargo.toml

@@ -18,8 +18,12 @@ proc-macro = true
 
 [dependencies]
 borsh-derive-internal = { path = "../borsh-derive-internal", version = "0.11.0" }
-borsh-schema-derive-internal = { path = "../borsh-schema-derive-internal", version = "0.11.0" }
+borsh-schema-derive-internal = { path = "../borsh-schema-derive-internal", version = "0.11.0", optional = true }
 syn = { version = "2", features = ["full", "fold"] }
 proc-macro-crate = "1"
 proc-macro2 = "1"
 quote = "1"
+
+[features]
+default = []
+schema = ["borsh-schema-derive-internal"]

borsh-derive/src/lib.rs

@@ -6,6 +6,7 @@ use proc_macro_crate::FoundCrate;
 use syn::{Ident, ItemEnum, ItemStruct, ItemUnion};
 
 use borsh_derive_internal::*;
+#[cfg(feature = "schema")]
 use borsh_schema_derive_internal::*;
 
 #[proc_macro_derive(BorshSerialize, attributes(borsh_skip))]
@@ -58,6 +59,7 @@ pub fn borsh_deserialize(input: TokenStream) -> TokenStream {
     })
 }
 
+#[cfg(feature = "schema")]
 #[proc_macro_derive(BorshSchema, attributes(borsh_skip))]
 pub fn borsh_schema(input: TokenStream) -> TokenStream {
     let name = &crate_name("borsh").unwrap();

borsh/Cargo.toml

@@ -21,7 +21,7 @@ path = "src/lib.rs"
 [[bin]]
 name = "generate_schema_schema"
 path = "src/generate_schema_schema.rs"
-required-features = ["std"]
+required-features = ["std", "schema"]
 
 [build-dependencies]
 cfg_aliases = "0.1.0"
@@ -40,6 +40,7 @@ borsh = { path = ".", default_features = false, features = ["bytes", "bson"] }
 insta = "1.29.0"
 
 [features]
-default = ["std"]
+default = ["std", "schema"]
+schema = ["borsh-derive/schema"]
 std = []
 rc = []

borsh/src/lib.rs

@@ -46,18 +46,24 @@
 #[cfg(not(feature = "std"))]
 extern crate alloc;
 
-pub use borsh_derive::{BorshDeserialize, BorshSchema, BorshSerialize};
+#[cfg(feature = "schema")]
+pub use borsh_derive::BorshSchema;
+pub use borsh_derive::{BorshDeserialize, BorshSerialize};
 
 pub mod de;
 
 // See `hash_collections` alias definition in build.rs
+#[cfg(feature = "schema")]
 pub mod schema;
+#[cfg(feature = "schema")]
 pub mod schema_helpers;
 pub mod ser;
 
 pub use de::BorshDeserialize;
 pub use de::{from_reader, from_slice};
+#[cfg(feature = "schema")]
 pub use schema::BorshSchema;
+#[cfg(feature = "schema")]
 pub use schema_helpers::{try_from_slice_with_schema, try_to_vec_with_schema};
 pub use ser::helpers::{to_vec, to_writer};
 pub use ser::BorshSerialize;

borsh/tests/smoke.rs

@@ -1,5 +1,4 @@
 #![cfg_attr(not(feature = "std"), no_std)]
-#![cfg(hash_collections)]
 // Smoke tests that ensure that we don't accidentally remove top-level
 // re-exports in a minor release.
 
@@ -8,8 +7,11 @@ extern crate alloc;
 #[cfg(not(feature = "std"))]
 use alloc::vec;
 
-use borsh::{self, from_slice, try_from_slice_with_schema, BorshSchema};
+use borsh::{self, from_slice};
+#[cfg(feature = "schema")]
+use borsh::{try_from_slice_with_schema, BorshSchema};
 
+#[cfg(feature = "schema")]
 #[test]
 fn test_to_vec() {
     let value = 42u8;

borsh/tests/test_schema_enums.rs

@@ -1,5 +1,6 @@
 #![cfg_attr(not(feature = "std"), no_std)]
 #![allow(dead_code)] // Local structures do not have their fields used.
+#![cfg(feature = "schema")]
 
 #[cfg(feature = "std")]
 use std::collections::BTreeMap;

borsh/tests/test_schema_nested.rs

@@ -1,6 +1,7 @@
 #![cfg_attr(not(feature = "std"), no_std)]
 #![cfg(hash_collections)]
 #![allow(dead_code)] // Local structures do not have their fields used.
+#![cfg(feature = "schema")]
 
 use borsh::schema::*;
 #[cfg(feature = "hashbrown")]

borsh/tests/test_schema_primitives.rs

@@ -1,5 +1,6 @@
 #![cfg_attr(not(feature = "std"), no_std)]
 #![cfg(hash_collections)]
+#![cfg(feature = "schema")]
 
 #[cfg(not(feature = "std"))]
 extern crate alloc;

borsh/tests/test_schema_structs.rs

@@ -1,4 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
+#![cfg(feature = "schema")]
 
 use borsh::schema::*;
 

borsh/tests/test_schema_tuple.rs

@@ -1,4 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
+#![cfg(feature = "schema")]
 
 #[cfg(feature = "std")]
 use std::collections::BTreeMap;