Upgrade dependencies whenever “easy” (#3556)

* Update all dependencies

* Upgrade dependencies whenever “easy”

“easy” means that there are no major changes required.

* Fix build and bump paste dependency to 0.1.6

* Remove dead code

* Re-add = dependency for futures-preview

* Add missing std features for runtime-io

* Remove git dependencies

as updated versions have been published to crates.io

* try to debug bug

* For sr-io, "std" should imply "no_oom" and "no_panic_handler".

Otherwise, rustc complains (correctly) about duplicate lang items.

* Add missing "runtime-io/std" features

* Fix compilation errors

* Prevent duplicate lang items

Rust does not allow duplicate lang items.  When compiled without the
`std` feature, `sr-io` defines two lang items.  Therefore, `sr-io`
compiled without `feature = "std"` must not be linked with `std`.

However, `pwasm-utils` and `wasmi-validation` both bring in `std` unless
compiled with `default-features = "false"`.  This caused a duplicate
lang item error.  Building both with `default-features = "false"`
prevents this error.  When building with `feature = "std"`, they should
both be built with the `std` feature, so this feature needs to be
explicitly depended on.

* Bump `impl_version`

* Make tests pass

Three tests used 1 less gas than they had previously.

* Try to un-break build

* Add a Cargo.lock file

* Revert offchain code

* Revert "Revert offchain code"

This reverts commit d216d08cc6ca0344614669c1d24cde3aa5c0d4e2.

* Don’t try to send a body with a GET request

without adding a Transfer-Encoding or Content-Length header.

This has always been wrong, but hyperium/hyper#1925 hid the bug until
hyper was upgraded to 0.12.35.

* Change some more GET requests to POST requests

* Fix excess line width and remove an `extern crate`

* Delete commented-out extern crate

Co-Authored-By: Sergei Pepyakin <sergei@parity.io>

* Fix regression in Cargo.toml files

dev-dependencies need `default-features = false`, too.

* Bump parity-wasm dependency

* Bump `futures-preview`

* Apply suggestions from code review

Co-Authored-By: Bastian Köcher <bkchr@users.noreply.github.com>

* Update Cargo.lock files

* Apply suggestions from code review

Co-Authored-By: Bastian Köcher <bkchr@users.noreply.github.com>

* Update core/service/src/chain_ops.rs

Co-Authored-By: Sergei Pepyakin <sergei@parity.io>

substrate/core/service/Cargo.toml

@@ -5,20 +5,20 @@ authors = ["Parity Technologies <admin@parity.io>"]
 edition = "2018"
 
 [dependencies]
-derive_more = "0.14.0"
-futures = "0.1.17"
-futures03 = { package = "futures-preview", version = "0.3.0-alpha.18", features = ["compat"] }
+derive_more = "0.15.0"
+futures = "0.1.29"
+futures03 = { package = "futures-preview", version = "0.3.0-alpha.19", features = ["compat"] }
 parking_lot = "0.9.0"
-lazy_static = "1.0"
-log = "0.4"
-slog = {version = "^2", features = ["nested-values"]}
-tokio-executor = "0.1.7"
-tokio-timer = "0.2"
-exit-future = "0.1"
-serde = { version = "1.0" }
-serde_json = "1.0"
-sysinfo = "0.9.0"
-target_info = "0.1"
+lazy_static = "1.4.0"
+log = "0.4.8"
+slog = { version = "2.5.2", features = ["nested-values"] }
+tokio-executor = "0.1.8"
+tokio-timer = "0.2.11"
+exit-future = "0.1.4"
+serde = "1.0.101"
+serde_json = "1.0.40"
+sysinfo = "0.9.5"
+target_info = "0.1.0"
 keystore = { package = "substrate-keystore", path = "../../core/keystore" }
 sr-io = { path = "../../core/sr-io" }
 sr-primitives = { path = "../../core/sr-primitives" }

substrate/core/service/src/chain_ops.rs

@@ -21,6 +21,7 @@ use chain_spec::{ChainSpec, RuntimeGenesis, Extension};
 
 /// Defines the logic for an operation exporting blocks within a range.
 #[macro_export]
+/// Export blocks
 macro_rules! export_blocks {
 ($client:ident, $exit:ident, $output:ident, $from:ident, $to:ident, $json:ident) => {{
 	let mut block = $from;
@@ -77,6 +78,7 @@ macro_rules! export_blocks {
 
 /// Defines the logic for an operation importing blocks from some known import.
 #[macro_export]
+/// Import blocks
 macro_rules! import_blocks {
 ($block:ty, $client:ident, $queue:ident, $exit:ident, $input:ident) => {{
 	use consensus_common::import_queue::{IncomingBlock, Link, BlockImportError, BlockImportResult};
@@ -120,7 +122,7 @@ macro_rules! import_blocks {
 	}
 
 	let (exit_send, exit_recv) = std::sync::mpsc::channel();
-	::std::thread::spawn(move || {
+	std::thread::spawn(move || {
 		let _ = $exit.wait();
 		let _ = exit_send.send(());
 	});

substrate/core/service/src/lib.rs

@@ -629,7 +629,7 @@ impl<TBl, TCl, TSc, TNetStatus, TNet, TTxPool, TOc> Future for
 
 		// Polling all the `to_poll` futures.
 		while let Some(pos) = self.to_poll.iter_mut().position(|t| t.poll().map(|t| t.is_ready()).unwrap_or(true)) {
-			self.to_poll.remove(pos);
+			let _ = self.to_poll.remove(pos);
 		}
 
 		// The service future never ends.

substrate/core/service/test/Cargo.toml

@@ -5,13 +5,13 @@ authors = ["Parity Technologies <admin@parity.io>"]
 edition = "2018"
 
 [dependencies]
-tempdir = "0.3"
-tokio = "0.1.7"
-futures = "0.1"
-futures03 = { package = "futures-preview", version = "=0.3.0-alpha.18", features = ["compat"] }
-log = "0.4"
-env_logger = "0.6"
-fdlimit = "0.1"
+tempdir = "0.3.7"
+tokio = "0.1.22"
+futures = "0.1.29"
+log = "0.4.8"
+env_logger = "0.6.2"
+fdlimit = "0.1.1"
+futures03 = { package = "futures-preview", version = "=0.3.0-alpha.19", features = ["compat"] }
 service = { package = "substrate-service", path = "../../../core/service" }
 network = { package = "substrate-network", path = "../../../core/network" }
 consensus = { package = "substrate-consensus-common", path = "../../../core/consensus/common" }