Introduce udeps to remove unused dependencies

parallel-finance · 0x8f701 · Jun 1, 2022 · May 30, 2022 · May 31, 2022 · May 31, 2022
commit 1b5190469503130da4e8d8bf76f180e627af0067
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Makefile b/Makefile
@@ -19,6 +19,7 @@ init: submodules
 	git config advice.ignoredHook false
 	git config core.hooksPath .githooks
 	curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y --default-toolchain nightly-2022-04-24 --component rust-src --component rustfmt --component clippy --target wasm32-unknown-unknown
+	cargo install cargo-udeps --locked 
 	cd scripts/helper && yarn
 	cd scripts/polkadot-launch && yarn
 
@@ -143,6 +144,10 @@ fmt:
 	SKIP_WASM_BUILD= cargo fmt --all
 	cd scripts/helper && yarn format
 
+.PHONY: udeps
+udeps:
+	SKIP_WASM_BUILD= cargo udeps -q --all-targets
+
 .PHONY: resources
 resources:
 	docker run --rm parallelfinance/parallel:$(DOCKER_TAG) export-genesis-state --chain $(CHAIN) > ./resources/para-$(PARA_ID)-genesis

diff --git a/node/parallel/Cargo.toml b/node/parallel/Cargo.toml
@@ -77,7 +77,6 @@ sp-trie             = { git = 'https://github.com/paritytech/substrate.git', bra
 cumulus-client-cli                      = { git = 'https://github.com/paritytech/cumulus.git', branch = 'polkadot-v0.9.22' }
 cumulus-client-collator                 = { git = 'https://github.com/paritytech/cumulus.git', branch = 'polkadot-v0.9.22' }
 cumulus-client-consensus-aura           = { git = 'https://github.com/paritytech/cumulus.git', branch = 'polkadot-v0.9.22' }
-cumulus-client-consensus-relay-chain    = { git = 'https://github.com/paritytech/cumulus.git', branch = 'polkadot-v0.9.22' }
 cumulus-client-network                  = { git = 'https://github.com/paritytech/cumulus.git', branch = 'polkadot-v0.9.22' }
 cumulus-client-service                  = { git = 'https://github.com/paritytech/cumulus.git', branch = 'polkadot-v0.9.22' }
 cumulus-primitives-core                 = { git = 'https://github.com/paritytech/cumulus.git', branch = 'polkadot-v0.9.22' }

diff --git a/pallets/bridge/Cargo.toml b/pallets/bridge/Cargo.toml
@@ -16,7 +16,6 @@ scale-info         = { version = '2.1', default-features = false, features = ['d
 
 pallet-assets     = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 pallet-balances   = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
-pallet-collective = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 pallet-membership = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 primitives        = { package = 'parallel-primitives', path = '../../primitives', default-features = false }
 sp-runtime        = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
@@ -37,7 +36,6 @@ std                = [
   'frame-system/std',
   'frame-benchmarking/std',
   'primitives/std',
-  'pallet-collective/std',
   'pallet-assets/std',
   'pallet-membership/std',
   'pallet-balances/std',

diff --git a/pallets/streaming/Cargo.toml b/pallets/streaming/Cargo.toml
@@ -15,7 +15,6 @@ frame-system       = { git = 'https://github.com/paritytech/substrate.git', bran
 orml-traits        = { version = '0.4.1-dev', default-features = false }
 pallet-assets      = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 pallet-balances    = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
-pallet-prices      = { path = '../prices', default-features = false }
 pallet-timestamp   = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 primitives         = { package = 'parallel-primitives', path = '../../primitives', default-features = false }
 scale-info         = { version = '2.1', default-features = false, features = ['derive'] }
@@ -44,7 +43,6 @@ std                = [
   'sp-std/std',
   'pallet-assets/std',
   'pallet-balances/std',
-  'pallet-prices/std',
   'pallet-assets/std',
   'pallet-timestamp/std',
   'serde',

diff --git a/pallets/xcm-helper/Cargo.toml b/pallets/xcm-helper/Cargo.toml
@@ -31,7 +31,6 @@ cumulus-pallet-xcm              = { git = 'https://github.com/paritytech/cumulus
 cumulus-pallet-xcmp-queue       = { git = 'https://github.com/paritytech/cumulus.git', branch = 'polkadot-v0.9.22' }
 kusama-runtime                  = { git = 'https://github.com/paritytech/polkadot.git', branch = 'release-v0.9.22', features = ['runtime-benchmarks'] }
 orml-xcm-support                = { version = '0.4.1-dev' }
-orml-xtokens                    = { version = '0.4.1-dev' }
 pallet-babe                     = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22' }
 pallet-balances                 = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22' }
 pallet-session                  = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22' }

diff --git a/runtime/heiko/Cargo.toml b/runtime/heiko/Cargo.toml
@@ -40,7 +40,6 @@ pallet-preimage                            = { git = 'https://github.com/parityt
 pallet-proxy                               = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 pallet-scheduler                           = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 pallet-session                             = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
-pallet-sudo                                = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 pallet-timestamp                           = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 pallet-transaction-payment                 = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 pallet-transaction-payment-rpc-runtime-api = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
@@ -164,7 +163,6 @@ std                = [
   'pallet-crowdloans/std',
   'pallet-xcm/std',
   'pallet-timestamp/std',
-  'pallet-sudo/std',
   'pallet-transaction-payment/std',
   'pallet-transaction-payment-rpc-runtime-api/std',
   'parachain-info/std',
@@ -225,7 +223,6 @@ try-runtime        = [
   'frame-system/try-runtime',
   'pallet-balances/try-runtime',
   'pallet-timestamp/try-runtime',
-  'pallet-sudo/try-runtime',
   'pallet-transaction-payment/try-runtime',
   'pallet-multisig/try-runtime',
   'pallet-utility/try-runtime',
@@ -254,3 +251,8 @@ try-runtime        = [
   'pallet-xcm-helper/try-runtime',
   'pallet-asset-registry/try-runtime',
 ]
+
+[package.metadata.cargo-udeps.ignore]
+normal = [
+  'frame-try-runtime',
+]
diff --git a/runtime/heiko/src/lib.rs b/runtime/heiko/src/lib.rs
@@ -867,11 +867,6 @@ impl pallet_transaction_payment::Config for Runtime {
     type LengthToFee = ConstantMultiplier<Balance, TransactionByteFee>;
 }
 
-// impl pallet_sudo::Config for Runtime {
-//     type Event = Event;
-//     type Call = Call;
-// }
-
 #[derive(
     Copy,
     Clone,
@@ -2012,7 +2007,6 @@ construct_runtime!(
         Identity: pallet_identity::{Pallet, Call, Storage, Event<T>} = 8,
 
         // Governance
-        // Sudo: pallet_sudo::{Pallet, Call, Storage, Config<T>, Event<T>} = 10,
         Democracy: pallet_democracy::{Pallet, Call, Storage, Config<T>, Event<T>} = 11,
         GeneralCouncil: pallet_collective::<Instance1>::{Pallet, Call, Storage, Origin<T>, Event<T>, Config<T>} = 12,
         TechnicalCommittee: pallet_collective::<Instance2>::{Pallet, Call, Storage, Origin<T>, Event<T>, Config<T>} = 13,

diff --git a/runtime/kerria/Cargo.toml b/runtime/kerria/Cargo.toml
@@ -256,3 +256,8 @@ try-runtime        = [
   'pallet-xcm-helper/try-runtime',
   'pallet-asset-registry/try-runtime',
 ]
+
+[package.metadata.cargo-udeps.ignore]
+normal = [
+  'frame-try-runtime',
+]
diff --git a/runtime/parallel/Cargo.toml b/runtime/parallel/Cargo.toml
@@ -40,7 +40,6 @@ pallet-preimage                            = { git = 'https://github.com/parityt
 pallet-proxy                               = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 pallet-scheduler                           = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 pallet-session                             = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
-pallet-sudo                                = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 pallet-timestamp                           = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 pallet-transaction-payment                 = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 pallet-transaction-payment-rpc-runtime-api = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
@@ -163,7 +162,6 @@ std                = [
   'pallet-balances/std',
   'pallet-xcm/std',
   'pallet-timestamp/std',
-  'pallet-sudo/std',
   'pallet-transaction-payment/std',
   'pallet-transaction-payment-rpc-runtime-api/std',
   'parachain-info/std',
@@ -226,7 +224,6 @@ try-runtime        = [
   'frame-system/try-runtime',
   'pallet-balances/try-runtime',
   'pallet-timestamp/try-runtime',
-  'pallet-sudo/try-runtime',
   'pallet-transaction-payment/try-runtime',
   'pallet-multisig/try-runtime',
   'pallet-utility/try-runtime',
@@ -256,3 +253,8 @@ try-runtime        = [
   'pallet-xcm-helper/try-runtime',
   'pallet-asset-registry/try-runtime',
 ]
+
+[package.metadata.cargo-udeps.ignore]
+normal = [
+  'frame-try-runtime',
+]
diff --git a/runtime/parallel/src/lib.rs b/runtime/parallel/src/lib.rs
@@ -882,11 +882,6 @@ impl pallet_transaction_payment::Config for Runtime {
     type LengthToFee = ConstantMultiplier<Balance, TransactionByteFee>;
 }
 
-// impl pallet_sudo::Config for Runtime {
-//     type Event = Event;
-//     type Call = Call;
-// }
-
 #[derive(
     Copy,
     Clone,
@@ -2013,7 +2008,6 @@ construct_runtime!(
         Identity: pallet_identity::{Pallet, Call, Storage, Event<T>} = 8,
 
         // Governance
-        // Sudo: pallet_sudo::{Pallet, Call, Storage, Config<T>, Event<T>} = 10,
         Democracy: pallet_democracy::{Pallet, Call, Storage, Config<T>, Event<T>} = 11,
         GeneralCouncil: pallet_collective::<Instance1>::{Pallet, Call, Storage, Origin<T>, Event<T>, Config<T>} = 12,
         TechnicalCommittee: pallet_collective::<Instance2>::{Pallet, Call, Storage, Origin<T>, Event<T>, Config<T>} = 13,

diff --git a/runtime/vanilla/Cargo.toml b/runtime/vanilla/Cargo.toml
@@ -260,3 +260,8 @@ try-runtime        = [
   'pallet-xcm-helper/try-runtime',
   'pallet-asset-registry/try-runtime',
 ]
+
+[package.metadata.cargo-udeps.ignore]
+normal = [
+  'frame-try-runtime',
+]
diff --git a/support/Cargo.toml b/support/Cargo.toml
@@ -10,17 +10,15 @@ targets = ['x86_64-unknown-linux-gnu']
 [dependencies]
 serde               = { version = '1.0.136', features = ['derive'], optional = true }
 codec               = { package = 'parity-scale-codec', version = '3.0.0', features = ['max-encoded-len'], default-features = false }
-frame-support       = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
-frame-system        = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 log                 = { version = "0.4", default-features = false }
 num-bigint          = { default-features = false, version = '0.4' }
 num-traits          = { default-features = false, version = '0.2' }
-primitives          = { package = 'parallel-primitives', path = '../primitives', default-features = false }
 scale-info          = { version = '2.1', default-features = false, features = ['derive'] }
 sp-core             = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 sp-io               = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 sp-runtime          = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 sp-std              = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
+frame-support       = { git = 'https://github.com/paritytech/substrate.git', branch = 'polkadot-v0.9.22', default-features = false }
 substrate-fixed     = { git = "https://github.com/encointer/substrate-fixed", default-features = false }
 
 [dev-dependencies]
@@ -32,11 +30,9 @@ std     = [
   'serde',
   'codec/std',
   'frame-support/std',
-  'frame-system/std',
   'sp-runtime/std',
   'scale-info/std',
   'sp-std/std',
-  'primitives/std',
   'num-bigint/std',
   'num-traits/std',
   'sp-core/std',
@@ -46,3 +42,8 @@ try-runtime = ['frame-support/try-runtime']
 
 [lib]
 doctest = false
+
+[package.metadata.cargo-udeps.ignore]
+normal = [
+  'frame-support',
+]