Migrate rpc-servers, serializer, service and sr-io to the 2018 edition (

paritytech#1732)
freedomfromfiat · Feb 8, 2019 · d4eb4ba · d4eb4ba
1 parent ccd178d
commit d4eb4ba
Show file tree

Hide file tree

Showing 30 changed files with 62 additions and 133 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/core/cli/Cargo.toml b/core/cli/Cargo.toml
@@ -13,7 +13,6 @@ log = "0.4"
 atty = "0.2"
 regex = "1"
 time = "0.1"
-slog = "^2"
 ansi_term = "0.11"
 lazy_static = "1.0"
 app_dirs = "1.2"

diff --git a/core/cli/src/informant.rs b/core/cli/src/informant.rs
@@ -27,7 +27,6 @@ use network::{SyncState, SyncProvider};
 use client::{backend::Backend, BlockchainEvents};
 use substrate_telemetry::telemetry;
 use log::{debug, info, warn};
-use slog::slog_info;
 
 use runtime_primitives::generic::BlockId;
 use runtime_primitives::traits::{Header, As};

diff --git a/core/client/Cargo.toml b/core/client/Cargo.toml
@@ -11,7 +11,6 @@ log = { version = "0.4", optional = true }
 parking_lot = { version = "0.7.1", optional = true }
 hex = { package = "hex-literal", version = "0.1", optional = true }
 futures = { version = "0.1.17", optional = true }
-slog = { version = "^2", optional = true }
 heapsize = { version = "0.4", optional = true }
 consensus = { package = "substrate-consensus-common", path = "../consensus/common", optional = true }
 executor = { package = "substrate-executor", path = "../executor", optional = true }
@@ -49,7 +48,6 @@ std = [
 	"log",
 	"hex",
 	"futures",
-	"slog",
 	"heapsize",
 	"executor",
 	"runtime-primitives/std",

diff --git a/core/client/src/client.rs b/core/client/src/client.rs
@@ -63,7 +63,6 @@ use crate::genesis;
 use consensus;
 use substrate_telemetry::telemetry;
 
-use slog::slog_info;
 use log::{info, trace, warn};
 use error_chain::bail;
 

diff --git a/core/client/src/light/backend.rs b/core/client/src/light/backend.rs
@@ -501,6 +501,7 @@ mod tests {
 		}
 	}
 
+	#[test]
 	fn light_aux_store_is_updated_via_non_importing_op() {
 		let backend = Backend::new(Arc::new(DummyBlockchain::new(DummyStorage::new())));
 		let mut op = ClientBackend::<Block, Blake2Hasher>::begin_operation(&backend).unwrap();

diff --git a/core/primitives/src/lib.rs b/core/primitives/src/lib.rs
@@ -21,6 +21,8 @@
 #![cfg_attr(not(feature = "std"), no_std)]
 #![cfg_attr(not(feature = "std"), feature(alloc))]
 
+/// Creates a vector of given pairs and calls `collect` on the iterator from it.
+/// Can be used to create a `HashMap`.
 #[macro_export]
 macro_rules! map {
 	($( $name:expr => $value:expr ),*) => (

diff --git a/core/rpc-servers/Cargo.toml b/core/rpc-servers/Cargo.toml
@@ -2,11 +2,12 @@
 name = "substrate-rpc-servers"
 version = "0.1.0"
 authors = ["Parity Technologies <admin@parity.io>"]
+edition = "2018"
 
 [dependencies]
-jsonrpc-http-server = "10.0.1"
-jsonrpc-pubsub = "10.0.1"
-jsonrpc-ws-server = "10.0.1"
+http = { package = "jsonrpc-http-server", version = "10.0.1" }
+pubsub = { package = "jsonrpc-pubsub", version = "10.0.1" }
+ws = { package = "jsonrpc-ws-server", version = "10.0.1" }
 log = "0.4"
 serde = "1.0"
 substrate-rpc = { path = "../rpc", version = "0.1" }

diff --git a/core/rpc-servers/src/lib.rs b/core/rpc-servers/src/lib.rs
@@ -18,18 +18,10 @@
 
 #[warn(missing_docs)]
 
-pub extern crate substrate_rpc as apis;
-
-extern crate jsonrpc_http_server as http;
-extern crate jsonrpc_pubsub as pubsub;
-extern crate jsonrpc_ws_server as ws;
-extern crate serde;
-extern crate sr_primitives;
-
-#[macro_use]
-extern crate log;
+pub use substrate_rpc as apis;
 
 use std::io;
+use log::error;
 use sr_primitives::{traits::{Block as BlockT, NumberFor}, generic::SignedBlock};
 
 /// Maximal payload accepted by RPC servers

diff --git a/core/serializer/Cargo.toml b/core/serializer/Cargo.toml
@@ -2,6 +2,7 @@
 name = "substrate-serializer"
 version = "0.1.0"
 authors = ["Parity Technologies <admin@parity.io>"]
+edition = "2018"
 
 [dependencies]
 serde = { version = "1.0", default-features = false }

diff --git a/core/serializer/src/lib.rs b/core/serializer/src/lib.rs
@@ -21,9 +21,6 @@
 
 #![warn(missing_docs)]
 
-extern crate serde;
-extern crate serde_json;
-
 pub use serde_json::{from_str, from_slice, from_reader, Result, Error};
 
 const PROOF: &str = "Serializers are infallible; qed";

diff --git a/core/service/Cargo.toml b/core/service/Cargo.toml
@@ -2,6 +2,7 @@
 name = "substrate-service"
 version = "0.3.0"
 authors = ["Parity Technologies <admin@parity.io>"]
+edition = "2018"
 
 [dependencies]
 futures = "0.1.17"
@@ -16,19 +17,19 @@ serde = "1.0"
 serde_json = "1.0"
 serde_derive = "1.0"
 target_info = "0.1"
-substrate-keystore = { path = "../../core/keystore" }
+keystore = { package = "substrate-keystore", path = "../../core/keystore" }
 sr-io = { path = "../../core/sr-io" }
-sr-primitives = { path = "../../core/sr-primitives" }
-substrate-primitives = { path = "../../core/primitives" }
-substrate-consensus-common = { path = "../../core/consensus/common" }
-substrate-network = { path = "../../core/network" }
-substrate-client = { path = "../../core/client" }
-substrate-client-db = { path = "../../core/client/db" }
+runtime_primitives = { package = "sr-primitives", path = "../../core/sr-primitives" }
+primitives = { package = "substrate-primitives", path = "../../core/primitives" }
+consensus_common = { package = "substrate-consensus-common", path = "../../core/consensus/common" }
+network = { package = "substrate-network", path = "../../core/network" }
+client = { package = "substrate-client", path = "../../core/client" }
+client_db = { package = "substrate-client-db", path = "../../core/client/db" }
 parity-codec = "3.0"
 substrate-executor = { path = "../../core/executor" }
-substrate-transaction-pool = { path = "../../core/transaction-pool" }
-substrate-rpc-servers = { path = "../../core/rpc-servers" }
-substrate-telemetry = { path = "../../core/telemetry" }
+transaction_pool = { package = "substrate-transaction-pool", path = "../../core/transaction-pool" }
+rpc = { package = "substrate-rpc-servers", path = "../../core/rpc-servers" }
+tel = { package = "substrate-telemetry", path = "../../core/telemetry" }
 
 [dev-dependencies]
 substrate-test-client = { path = "../test-client" }
diff --git a/core/service/src/chain_ops.rs b/core/service/src/chain_ops.rs
@@ -18,18 +18,19 @@
 
 use std::{self, io::{Read, Write}};
 use futures::Future;
+use log::{info, warn};
 
 use runtime_primitives::generic::{SignedBlock, BlockId};
 use runtime_primitives::traits::{As, Block, Header};
 use consensus_common::import_queue::{ImportQueue, IncomingBlock, Link};
 use network::message;
 
 use consensus_common::BlockOrigin;
-use components::{self, Components, ServiceFactory, FactoryFullConfiguration, FactoryBlockNumber, RuntimeGenesis};
-use new_client;
-use codec::{Decode, Encode};
-use error;
-use chain_spec::ChainSpec;
+use crate::components::{self, Components, ServiceFactory, FactoryFullConfiguration, FactoryBlockNumber, RuntimeGenesis};
+use crate::new_client;
+use parity_codec::{Decode, Encode};
+use crate::error;
+use crate::chain_spec::ChainSpec;
 
 /// Export a range of blocks to a binary stream.
 pub fn export_blocks<F, E, W>(

diff --git a/core/service/src/chain_spec.rs b/core/service/src/chain_spec.rs
@@ -19,10 +19,11 @@
 use std::collections::HashMap;
 use std::fs::File;
 use std::path::PathBuf;
+use serde_derive::{Serialize, Deserialize};
 use primitives::storage::{StorageKey, StorageData};
 use runtime_primitives::{BuildStorage, StorageMap, ChildrenStorageMap};
 use serde_json as json;
-use components::RuntimeGenesis;
+use crate::components::RuntimeGenesis;
 use network::Multiaddr;
 
 enum GenesisSource<G> {

diff --git a/core/service/src/components.rs b/core/service/src/components.rs
@@ -19,18 +19,18 @@
 use std::{sync::Arc, net::SocketAddr, marker::PhantomData, ops::Deref, ops::DerefMut};
 use serde::{Serialize, de::DeserializeOwned};
 use tokio::runtime::TaskExecutor;
-use chain_spec::ChainSpec;
+use crate::chain_spec::ChainSpec;
 use client_db;
 use client::{self, Client, runtime_api::{Metadata, TaggedTransactionQueue}};
-use {error, Service, maybe_start_server};
+use crate::{error, Service, maybe_start_server};
 use consensus_common::import_queue::ImportQueue;
 use network::{self, OnDemand};
 use substrate_executor::{NativeExecutor, NativeExecutionDispatch};
 use transaction_pool::txpool::{self, Options as TransactionPoolOptions, Pool as TransactionPool};
 use runtime_primitives::{
 	BuildStorage, traits::{Block as BlockT, Header as HeaderT, ProvideRuntimeApi}, generic::BlockId
 };
-use config::Configuration;
+use crate::config::Configuration;
 use primitives::{Blake2Hasher, H256};
 use rpc::{self, apis::system::SystemInfo};
 use parking_lot::Mutex;
@@ -535,7 +535,7 @@ impl<Factory: ServiceFactory> Components for LightComponents<Factory> {
 #[cfg(test)]
 mod tests {
 	use super::*;
-	use codec::Encode;
+	use parity_codec::Encode;
 	use consensus_common::BlockOrigin;
 	use substrate_test_client::{
 		self,

diff --git a/core/service/src/config.rs b/core/service/src/config.rs
@@ -18,7 +18,7 @@
 
 use std::net::SocketAddr;
 use transaction_pool;
-use chain_spec::ChainSpec;
+use crate::chain_spec::ChainSpec;
 pub use client::ExecutionStrategy;
 pub use client_db::PruningMode;
 pub use network::config::{NetworkConfiguration, Roles};

diff --git a/core/service/src/error.rs b/core/service/src/error.rs
@@ -24,6 +24,7 @@ use client;
 use network;
 use keystore;
 use consensus_common;
+use error_chain::*;
 
 error_chain! {
 	foreign_links {

diff --git a/core/service/src/lib.rs b/core/service/src/lib.rs
@@ -19,39 +19,6 @@
 
 #![warn(unused_extern_crates)]
 
-extern crate futures;
-extern crate exit_future;
-extern crate serde;
-extern crate serde_json;
-extern crate parking_lot;
-extern crate substrate_keystore as keystore;
-extern crate substrate_primitives as primitives;
-extern crate sr_primitives as runtime_primitives;
-extern crate substrate_consensus_common as consensus_common;
-extern crate substrate_network as network;
-extern crate substrate_executor;
-extern crate substrate_client as client;
-extern crate substrate_client_db as client_db;
-extern crate parity_codec as codec;
-extern crate substrate_transaction_pool as transaction_pool;
-extern crate substrate_rpc_servers as rpc;
-extern crate target_info;
-extern crate tokio;
-
-#[macro_use]
-extern crate substrate_telemetry as tel;
-#[macro_use]
-extern crate error_chain;
-#[macro_use]
-extern crate slog;	// needed until we can reexport `slog_info` from `substrate_telemetry`
-#[macro_use]
-extern crate log;
-#[macro_use]
-extern crate serde_derive;
-
-#[cfg(test)]
-extern crate substrate_test_client;
-
 mod components;
 mod error;
 mod chain_spec;
@@ -63,6 +30,7 @@ use std::net::SocketAddr;
 use std::collections::HashMap;
 #[doc(hidden)]
 pub use std::{ops::Deref, result::Result, sync::Arc};
+use log::{info, warn, debug};
 use futures::prelude::*;
 use keystore::Store as Keystore;
 use client::BlockchainEvents;
@@ -72,7 +40,8 @@ use exit_future::Signal;
 #[doc(hidden)]
 pub use tokio::runtime::TaskExecutor;
 use substrate_executor::NativeExecutor;
-use codec::{Encode, Decode};
+use parity_codec::{Encode, Decode};
+use tel::telemetry;
 
 pub use self::error::{ErrorKind, Error};
 pub use config::{Configuration, Roles, PruningMode};

diff --git a/core/service/test/Cargo.toml b/core/service/test/Cargo.toml
@@ -2,6 +2,7 @@
 name = "substrate-service-test"
 version = "0.3.0"
 authors = ["Parity Technologies <admin@parity.io>"]
+edition = "2018"
 
 [dependencies]
 tempdir = "0.3"
@@ -10,9 +11,9 @@ futures = "0.1"
 log = "0.4"
 env_logger = "0.6"
 fdlimit = "0.1"
-substrate-service = { path = "../../../core/service" }
-substrate-network = { path = "../../../core/network" }
-substrate-consensus-common = { path = "../../../core/consensus/common" }
-substrate-primitives = { path = "../../../core/primitives" }
-substrate-client = { path = "../../../core/client" }
+service = { package = "substrate-service", path = "../../../core/service" }
+network = { package = "substrate-network", path = "../../../core/network" }
+consensus = { package = "substrate-consensus-common", path = "../../../core/consensus/common" }
+primitives = { package = "substrate-primitives", path = "../../../core/primitives" }
+client = { package = "substrate-client", path = "../../../core/client" }
 sr-primitives = { path = "../../../core/sr-primitives" }
diff --git a/core/service/test/src/lib.rs b/core/service/test/src/lib.rs
@@ -16,23 +16,11 @@
 
 //! Service integration test utils.
 
-#[macro_use]
-extern crate log;
-extern crate tempdir;
-extern crate tokio;
-extern crate futures;
-extern crate env_logger;
-extern crate fdlimit;
-extern crate substrate_service as service;
-extern crate substrate_network as network;
-extern crate substrate_primitives as primitives;
-extern crate substrate_client as client;
-extern crate substrate_consensus_common as consensus;
-extern crate sr_primitives;
 use std::iter;
 use std::sync::Arc;
 use std::net::Ipv4Addr;
 use std::time::Duration;
+use log::info;
 use futures::{Future, Stream};
 use tempdir::TempDir;
 use tokio::runtime::Runtime;

diff --git a/core/sr-io/Cargo.toml b/core/sr-io/Cargo.toml
@@ -3,32 +3,33 @@ name = "sr-io"
 version = "0.1.0"
 authors = ["Parity Technologies <admin@parity.io>"]
 build = "build.rs"
+edition = "2018"
 
 [build-dependencies]
 rustc_version = "0.2"
 
 [dependencies]
-sr-std = { path = "../sr-std", default-features = false }
-substrate-primitives = { path = "../primitives", default-features = false }
+rstd = { package = "sr-std", path = "../sr-std", default-features = false }
+primitives = { package = "substrate-primitives", path = "../primitives", default-features = false }
 parity-codec = { version = "3.0", default-features = false }
 hash-db = { version = "0.11", default-features = false }
 libsecp256k1 = { version = "0.2.1", optional = true }
 tiny-keccak = { version = "1.4.2", optional = true }
 environmental = { version = "~1.0", optional = true }
 substrate-state-machine = { path = "../state-machine", optional = true }
-substrate-trie = { path = "../trie", optional = true }
+trie = { package = "substrate-trie", path = "../trie", optional = true }
 
 [features]
 default = ["std"]
 std = [
-	"substrate-primitives/std",
+	"primitives/std",
 	"parity-codec/std",
-	"sr-std/std",
+	"rstd/std",
 
 	"hash-db/std",
 	"environmental",
 	"substrate-state-machine",
-	"substrate-trie",
+	"trie",
 	"libsecp256k1",
 	"tiny-keccak"
 ]
-Original file line number
+Diff line change
@@ Expand Up / @@ -501,6 +501,7 @@ mod tests { @@
     		}
     	}
+    	#[test]
     	fn light_aux_store_is_updated_via_non_importing_op() {
     		let backend = Backend::new(Arc::new(DummyBlockchain::new(DummyStorage::new())));
     		let mut op = ClientBackend::<Block, Blake2Hasher>::begin_operation(&backend).unwrap();
@@ Expand Down @@