Replace ethcore-logger with env-logger.

Cargo.toml

@@ -9,7 +9,6 @@ authors = ["Parity Technologies <admin@parity.io>"]
 [dependencies]
 blooms-db = { path = "util/blooms-db" }
 log = "0.4"
-env_logger = "0.5"
 rustc-hex = "1.0"
 docopt = "1.0"
 clap = "2"
@@ -35,7 +34,7 @@ ethcore = { path = "ethcore", features = ["parity"] }
 parity-bytes = "0.1"
 ethcore-io = { path = "util/io" }
 ethcore-light = { path = "ethcore/light" }
-ethcore-logger = { path = "logger" }
+ethcore-logger = { path = "parity/logger" }
 ethcore-miner = { path = "miner" }
 ethcore-network = { path = "util/network" }
 ethcore-private-tx = { path = "ethcore/private-tx" }

ethcore/Cargo.toml

@@ -21,9 +21,9 @@ memorydb = "0.3.0"
 patricia-trie = "0.3.0"
 patricia-trie-ethereum = { path = "../util/patricia-trie-ethereum" }
 parity-crypto = "0.2"
+env_logger = { version = "0.5", optional = true }
 error-chain = { version = "0.12", default-features = false }
 ethcore-io = { path = "../util/io" }
-ethcore-logger = { path = "../logger" }
 ethcore-miner = { path = "../miner" }
 ethcore-stratum = { path = "../miner/stratum", optional = true }
 ethcore-transaction = { path = "./transaction" }
@@ -78,7 +78,6 @@ hardware-wallet = { path = "../accounts/hw" }
 fake-hardware-wallet = { path = "../accounts/fake-hardware-wallet" }
 
 [dev-dependencies]
-env_logger = "0.4"
 tempdir = "0.3"
 trie-standardmap = "0.1"
 criterion = "0.2"
@@ -105,7 +104,7 @@ evm-debug-tests = ["evm-debug", "evm/evm-debug-tests"]
 # EVM debug traces are printed.
 slow-blocks = []
 # Run JSON consensus tests.
-json-tests = ["ethcore-transaction/json-tests", "test-helpers", "tempdir", "to-pod-full"]
+json-tests = ["env_logger", "ethcore-transaction/json-tests", "test-helpers", "tempdir", "to-pod-full"]
 # Skip JSON consensus tests with pending issues.
 ci-skip-issue = []
 # Run memory/cpu heavy tests.

ethcore/private-tx/Cargo.toml

@@ -14,7 +14,6 @@ ethcore = { path = ".." }
 parity-bytes = "0.1"
 parity-crypto = "0.2"
 ethcore-io = { path = "../../util/io" }
-ethcore-logger = { path = "../../logger" }
 ethcore-miner = { path = "../../miner" }
 ethcore-transaction = { path = "../transaction" }
 ethereum-types = "0.4"
@@ -40,4 +39,5 @@ transaction-pool = "1.13.2"
 url = "1"
 
 [dev-dependencies]
+env_logger = "0.5"
 ethcore = { path = "..", features = ["test-helpers"] }

ethcore/private-tx/src/lib.rs

@@ -60,7 +60,7 @@ extern crate rlp_derive;
 #[cfg(test)]
 extern crate rand;
 #[cfg(test)]
-extern crate ethcore_logger;
+extern crate env_logger;
 
 pub use encryptor::{Encryptor, SecretStoreEncryptor, EncryptorConfig, NoopEncryptor};
 pub use private_transactions::{VerifiedPrivateTransaction, VerificationStore, PrivateTransactionSigningDesc, SigningStore};

ethcore/private-tx/tests/private_contract.rs

@@ -16,14 +16,14 @@
 
 //! Contract for private transactions tests.
 
-extern crate rustc_hex;
+extern crate env_logger;
 extern crate ethcore;
-extern crate ethkey;
-extern crate keccak_hash as hash;
 extern crate ethcore_io;
-extern crate ethcore_logger;
 extern crate ethcore_private_tx;
 extern crate ethcore_transaction;
+extern crate ethkey;
+extern crate keccak_hash as hash;
+extern crate rustc_hex;
 
 #[macro_use]
 extern crate log;
@@ -47,7 +47,7 @@ use ethcore_private_tx::{NoopEncryptor, Provider, ProviderConfig};
 #[test]
 fn private_contract() {
 	// This uses a simple private contract: contract Test1 { bytes32 public x; function setX(bytes32 _x) { x = _x; } }
-	ethcore_logger::init_log();
+	::env_logger::try_init().ok();
 	let client = generate_dummy_client(0);
 	let chain_id = client.signing_chain_id();
 	let key1 = KeyPair::from_secret(Secret::from("0000000000000000000000000000000000000000000000000000000000000011")).unwrap();

ethcore/src/json_tests/chain.rs

@@ -42,7 +42,7 @@ fn skip_test(name: &String) -> bool {
 }
 
 pub fn json_chain_test<H: FnMut(&str, HookType)>(json_data: &[u8], start_stop_hook: &mut H) -> Vec<String> {
-	::ethcore_logger::init_log();
+	let _ = ::env_logger::try_init();
 	let tests = ethjson::blockchain::Test::load(json_data).unwrap();
 	let mut failed = Vec::new();
 

ethcore/src/json_tests/difficulty.rs

@@ -22,7 +22,7 @@ use spec::Spec;
 use super::HookType;
 
 pub fn json_difficulty_test<H: FnMut(&str, HookType)>(json_data: &[u8], spec: Spec, start_stop_hook: &mut H) -> Vec<String> {
-	::ethcore_logger::init_log();
+	let _ = ::env_logger::try_init();
 	let tests = ethjson::test::DifficultyTest::load(json_data).unwrap();
 	let engine = &spec.engine;
 

ethcore/src/json_tests/state.rs

@@ -48,7 +48,7 @@ fn skip_test(subname: &str, chain: &String, number: usize) -> bool {
 }
 
 pub fn json_chain_test<H: FnMut(&str, HookType)>(json_data: &[u8], start_stop_hook: &mut H) -> Vec<String> {
-	::ethcore_logger::init_log();
+	let _ = ::env_logger::try_init();
 	let tests = ethjson::state::test::Test::load(json_data).unwrap();
 	let mut failed = Vec::new();
 

ethcore/src/lib.rs

@@ -67,7 +67,6 @@ extern crate ethcore_bloom_journal as bloom_journal;
 extern crate parity_crypto;
 extern crate ethcore_io as io;
 extern crate parity_bytes as bytes;
-extern crate ethcore_logger;
 extern crate ethcore_miner;
 #[cfg(feature = "stratum")]
 extern crate ethcore_stratum;
@@ -141,7 +140,7 @@ extern crate serde_derive;
 #[cfg_attr(test, macro_use)]
 extern crate evm;
 
-#[cfg(test)]
+#[cfg(any(test, feature = "env_logger"))]
 extern crate env_logger;
 
 pub extern crate ethstore;

ethcore/src/snapshot/tests/service.rs

@@ -36,8 +36,7 @@ use verification::queue::kind::blocks::Unverified;
 
 #[test]
 fn restored_is_equivalent() {
-	use ::ethcore_logger::init_log;
-	init_log();
+	::env_logger::try_init().ok();
 
 	const NUM_BLOCKS: u32 = 400;
 	const TX_PER: usize = 5;
@@ -153,7 +152,7 @@ fn guards_delete_folders() {
 
 #[test]
 fn keep_ancient_blocks() {
-	::env_logger::init().ok();
+	::env_logger::try_init().ok();
 
 	// Test variables
 	const NUM_BLOCKS: u64 = 500;
@@ -272,7 +271,7 @@ fn keep_ancient_blocks() {
 
 #[test]
 fn recover_aborted_recovery() {
-	::env_logger::init().ok();
+	::env_logger::try_init().ok();
 
 	const NUM_BLOCKS: u32 = 400;
 	let gas_prices = vec![1.into(), 2.into(), 3.into(), 999.into()];

ethcore/src/spec/spec.rs

@@ -1016,7 +1016,7 @@ mod tests {
 
 	#[test]
 	fn genesis_constructor() {
-		::ethcore_logger::init_log();
+		let _ = ::env_logger::try_init();
 		let spec = Spec::new_test_constructor();
 		let db = spec.ensure_db_good(get_temp_state_db(), &Default::default())
 			.unwrap();