git.delta.rocks / unique-network / refs/commits / 37e76bc01ac1

difftreelog

fix make runtime build on 0.9.42

Yaroslav Bolyukin2023-05-22parent: #6f7886b.patch.diff
in: master

11 files changed

modifiedCargo.lockdiffbeforeafterboth
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -13701,6 +13701,7 @@
 dependencies = [
  "app-promotion-rpc",
  "fc-db",
+ "fc-mapping-sync",
  "fc-rpc",
  "fc-rpc-core",
  "fp-rpc",
modifiedruntime/common/config/ethereum.rsdiffbeforeafterboth
--- a/runtime/common/config/ethereum.rs
+++ b/runtime/common/config/ethereum.rs
@@ -4,7 +4,7 @@
 	traits::{FindAuthor},
 	parameter_types, ConsensusEngineId,
 };
-use sp_runtime::{RuntimeAppPublic, Perbill};
+use sp_runtime::{RuntimeAppPublic, Perbill, traits::ConstU32};
 use crate::{
 	runtime_common::{
 		config::sponsoring::DefaultSponsoringRateLimit,
@@ -84,6 +84,8 @@
 	type OnChargeTransaction = pallet_evm::EVMCurrencyAdapter<Balances, DealWithFees>;
 	type TransactionValidityHack = pallet_evm_transaction_payment::TransactionValidityHack<Self>;
 	type FindAuthor = EthereumFindAuthor<Aura>;
+	type Timestamp = crate::Timestamp;
+	type WeightInfo = pallet_evm::weights::SubstrateWeight<Self>;
 }
 
 impl pallet_evm_migration::Config for Runtime {
@@ -99,6 +101,9 @@
 	type RuntimeEvent = RuntimeEvent;
 	type StateRoot = pallet_ethereum::IntermediateStateRoot<Self>;
 	type PostLogContent = PostBlockAndTxnHashes;
+	// Space for revert reason. Ethereum transactions are not cheap, and overall size is much less
+	// than the substrate tx size, so we can afford this
+	type ExtraDataLength = ConstU32<32>;
 }
 
 parameter_types! {
modifiedruntime/common/config/substrate.rsdiffbeforeafterboth
--- a/runtime/common/config/substrate.rs
+++ b/runtime/common/config/substrate.rs
@@ -36,7 +36,7 @@
 use pallet_transaction_payment::{Multiplier, ConstFeeMultiplier};
 use crate::{
 	runtime_common::DealWithFees, Runtime, RuntimeEvent, RuntimeCall, RuntimeOrigin, PalletInfo,
-	System, Balances, Treasury, SS58Prefix, Version,
+	System, Balances, SS58Prefix, Version,
 };
 use up_common::{types::*, constants::*};
 
@@ -142,10 +142,15 @@
 	type Balance = Balance;
 	/// The ubiquitous event type.
 	type RuntimeEvent = RuntimeEvent;
-	type DustRemoval = Treasury;
+	// FIXME: Is () the new treasury?
+	type DustRemoval = ();
 	type ExistentialDeposit = ExistentialDeposit;
 	type AccountStore = System;
 	type WeightInfo = pallet_balances::weights::SubstrateWeight<Self>;
+	type HoldIdentifier = [u8; 16];
+	type FreezeIdentifier = [u8; 16];
+	type MaxHolds = ();
+	type MaxFreezes = ();
 }
 
 parameter_types! {
modifiedruntime/common/config/xcm/mod.rsdiffbeforeafterboth
after · runtime/common/config/xcm/mod.rs
1// Copyright 2019-2022 Unique Network (Gibraltar) Ltd.2// This file is part of Unique Network.34// Unique Network is free software: you can redistribute it and/or modify5// it under the terms of the GNU General Public License as published by6// the Free Software Foundation, either version 3 of the License, or7// (at your option) any later version.89// Unique Network is distributed in the hope that it will be useful,10// but WITHOUT ANY WARRANTY; without even the implied warranty of11// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the12// GNU General Public License for more details.1314// You should have received a copy of the GNU General Public License15// along with Unique Network. If not, see <http://www.gnu.org/licenses/>.1617use frame_support::{18	traits::{Everything, Nothing, Get, ConstU32, ProcessMessageError},19	parameter_types,20};21use frame_system::EnsureRoot;22use pallet_xcm::XcmPassthrough;23use polkadot_parachain::primitives::Sibling;24use xcm::latest::{prelude::*, Weight, MultiLocation};25use xcm::v3::Instruction;26use xcm_builder::{27	AccountId32Aliases, EnsureXcmOrigin, FixedWeightBounds, ParentAsSuperuser, RelayChainAsNative,28	SiblingParachainAsNative, SiblingParachainConvertsVia, SignedAccountId32AsNative,29	SignedToAccountId32, SovereignSignedViaLocation, ParentIsPreset,30};31use xcm_executor::{XcmExecutor, traits::ShouldExecute};32use sp_std::marker::PhantomData;33use crate::{34	Runtime, RuntimeCall, RuntimeEvent, RuntimeOrigin, ParachainInfo, ParachainSystem, PolkadotXcm,35	XcmpQueue, xcm_barrier::Barrier, RelayNetwork, AllPalletsWithSystem, Balances,36};3738use up_common::types::AccountId;3940#[cfg(feature = "foreign-assets")]41pub mod foreignassets;4243#[cfg(not(feature = "foreign-assets"))]44pub mod nativeassets;4546#[cfg(feature = "foreign-assets")]47pub use foreignassets as xcm_assets;4849#[cfg(not(feature = "foreign-assets"))]50pub use nativeassets as xcm_assets;5152use xcm_assets::{AssetTransactor, IsReserve, Trader};5354parameter_types! {55	pub const RelayLocation: MultiLocation = MultiLocation::parent();56	pub RelayOrigin: RuntimeOrigin = cumulus_pallet_xcm::Origin::Relay.into();57	pub UniversalLocation: InteriorMultiLocation = (58		GlobalConsensus(crate::RelayNetwork::get()),59		Parachain(ParachainInfo::get().into()),60	).into();61	pub SelfLocation: MultiLocation = MultiLocation::new(1, X1(Parachain(ParachainInfo::get().into())));6263	// One XCM operation is 1_000_000 weight - almost certainly a conservative estimate.64	pub UnitWeightCost: Weight = Weight::from_parts(1_000_000, 1000); // ?65	pub const MaxInstructions: u32 = 100;66}6768/// Type for specifying how a `MultiLocation` can be converted into an `AccountId`. This is used69/// when determining ownership of accounts for asset transacting and when attempting to use XCM70/// `Transact` in order to determine the dispatch Origin.71pub type LocationToAccountId = (72	// The parent (Relay-chain) origin converts to the default `AccountId`.73	ParentIsPreset<AccountId>,74	// Sibling parachain origins convert to AccountId via the `ParaId::into`.75	SiblingParachainConvertsVia<Sibling, AccountId>,76	// Straight up local `AccountId32` origins just alias directly to `AccountId`.77	AccountId32Aliases<RelayNetwork, AccountId>,78);7980/// No local origins on this chain are allowed to dispatch XCM sends/executions.81pub type LocalOriginToLocation = (SignedToAccountId32<RuntimeOrigin, AccountId, RelayNetwork>,);8283/// The means for routing XCM messages which are not for local execution into the right message84/// queues.85pub type XcmRouter = (86	// Two routers - use UMP to communicate with the relay chain:87	cumulus_primitives_utility::ParentAsUmp<ParachainSystem, PolkadotXcm, ()>,88	// ..and XCMP to communicate with the sibling chains.89	XcmpQueue,90);9192/// This is the type we use to convert an (incoming) XCM origin into a local `Origin` instance,93/// ready for dispatching a transaction with Xcm's `Transact`. There is an `OriginKind` which can94/// biases the kind of local `Origin` it will become.95pub type XcmOriginToTransactDispatchOrigin = (96	// Sovereign account converter; this attempts to derive an `AccountId` from the origin location97	// using `LocationToAccountId` and then turn that into the usual `Signed` origin. Useful for98	// foreign chains who want to have a local sovereign account on this chain which they control.99	SovereignSignedViaLocation<LocationToAccountId, RuntimeOrigin>,100	// Native converter for Relay-chain (Parent) location; will converts to a `Relay` origin when101	// recognised.102	RelayChainAsNative<RelayOrigin, RuntimeOrigin>,103	// Native converter for sibling Parachains; will convert to a `SiblingPara` origin when104	// recognised.105	SiblingParachainAsNative<cumulus_pallet_xcm::Origin, RuntimeOrigin>,106	// Superuser converter for the Relay-chain (Parent) location. This will allow it to issue a107	// transaction from the Root origin.108	ParentAsSuperuser<RuntimeOrigin>,109	// Native signed account converter; this just converts an `AccountId32` origin into a normal110	// `Origin::Signed` origin of the same 32-byte value.111	SignedAccountId32AsNative<RelayNetwork, RuntimeOrigin>,112	// Xcm origins can be represented natively under the Xcm pallet's Xcm origin.113	XcmPassthrough<RuntimeOrigin>,114);115116pub trait TryPass {117	fn try_pass<Call>(118		origin: &MultiLocation,119		message: &mut [Instruction<Call>],120	) -> Result<(), ProcessMessageError>;121}122123#[impl_trait_for_tuples::impl_for_tuples(30)]124impl TryPass for Tuple {125	fn try_pass<Call>(126		origin: &MultiLocation,127		message: &mut [Instruction<Call>],128	) -> Result<(), ProcessMessageError> {129		for_tuples!( #(130			Tuple::try_pass(origin, message)?;131		)* );132133		Ok(())134	}135}136137/// Deny executing the XCM if it matches any of the Deny filter regardless of anything else.138/// If it passes the Deny, and matches one of the Allow cases then it is let through.139pub struct DenyThenTry<Deny, Allow>(PhantomData<Deny>, PhantomData<Allow>)140where141	Deny: TryPass,142	Allow: ShouldExecute;143144impl<Deny, Allow> ShouldExecute for DenyThenTry<Deny, Allow>145where146	Deny: TryPass,147	Allow: ShouldExecute,148{149	fn should_execute<Call>(150		origin: &MultiLocation,151		message: &mut [Instruction<Call>],152		max_weight: Weight,153		weight_credit: &mut Weight,154	) -> Result<(), ProcessMessageError> {155		Deny::try_pass(origin, message)?;156		Allow::should_execute(origin, message, max_weight, weight_credit)157	}158}159160pub type Weigher = FixedWeightBounds<UnitWeightCost, RuntimeCall, MaxInstructions>;161162pub struct XcmExecutorConfig<T>(PhantomData<T>);163impl<T> xcm_executor::Config for XcmExecutorConfig<T>164where165	T: pallet_configuration::Config,166{167	type RuntimeCall = RuntimeCall;168	type XcmSender = XcmRouter;169	// How to withdraw and deposit an asset.170	type AssetTransactor = AssetTransactor;171	type OriginConverter = XcmOriginToTransactDispatchOrigin;172	type IsReserve = IsReserve;173	type IsTeleporter = (); // Teleportation is disabled174	type UniversalLocation = UniversalLocation;175	type Barrier = Barrier;176	type Weigher = Weigher;177	type Trader = Trader<T>;178	type ResponseHandler = PolkadotXcm;179	type SubscriptionService = PolkadotXcm;180	type PalletInstancesInfo = AllPalletsWithSystem;181	type MaxAssetsIntoHolding = ConstU32<8>;182183	type AssetTrap = PolkadotXcm;184	type AssetClaims = PolkadotXcm;185	type AssetLocker = ();186	type AssetExchanger = ();187	type FeeManager = ();188	type MessageExporter = ();189	type UniversalAliases = Nothing;190	type CallDispatcher = RuntimeCall;191192	// Deny all XCM Transacts.193	type SafeCallFilter = Nothing;194}195196#[cfg(feature = "runtime-benchmarks")]197parameter_types! {198	pub ReachableDest: Option<MultiLocation> = Some(Parent.into());199}200201impl pallet_xcm::Config for Runtime {202	type RuntimeEvent = RuntimeEvent;203	type SendXcmOrigin = EnsureXcmOrigin<RuntimeOrigin, ()>;204	type XcmRouter = XcmRouter;205	type ExecuteXcmOrigin = EnsureXcmOrigin<RuntimeOrigin, LocalOriginToLocation>;206	type XcmExecuteFilter = Everything;207	type XcmExecutor = XcmExecutor<XcmExecutorConfig<Self>>;208	type XcmTeleportFilter = Everything;209	type XcmReserveTransferFilter = Everything;210	type Weigher = FixedWeightBounds<UnitWeightCost, RuntimeCall, MaxInstructions>;211	type RuntimeOrigin = RuntimeOrigin;212	type RuntimeCall = RuntimeCall;213	const VERSION_DISCOVERY_QUEUE_SIZE: u32 = 100;214	type AdvertisedXcmVersion = pallet_xcm::CurrentXcmVersion;215	type UniversalLocation = UniversalLocation;216	type Currency = Balances;217	type CurrencyMatcher = ();218	type TrustedLockers = ();219	type SovereignAccountOf = LocationToAccountId;220	type MaxLockers = ConstU32<8>;221	type WeightInfo = crate::weights::xcm::SubstrateWeight<Runtime>;222	type AdminOrigin = EnsureRoot<AccountId>;223	#[cfg(feature = "runtime-benchmarks")]224	type ReachableDest = ReachableDest;225}226227impl cumulus_pallet_xcm::Config for Runtime {228	type RuntimeEvent = RuntimeEvent;229	type XcmExecutor = XcmExecutor<XcmExecutorConfig<Self>>;230}231232impl cumulus_pallet_xcmp_queue::Config for Runtime {233	type WeightInfo = ();234	type RuntimeEvent = RuntimeEvent;235	type XcmExecutor = XcmExecutor<XcmExecutorConfig<Self>>;236	type ChannelInfo = ParachainSystem;237	type VersionWrapper = PolkadotXcm;238	type ExecuteOverweightOrigin = frame_system::EnsureRoot<AccountId>;239	type ControllerOrigin = EnsureRoot<AccountId>;240	type ControllerOriginConverter = XcmOriginToTransactDispatchOrigin;241	type PriceForSiblingDelivery = ();242}243244impl cumulus_pallet_dmp_queue::Config for Runtime {245	type RuntimeEvent = RuntimeEvent;246	type XcmExecutor = XcmExecutor<XcmExecutorConfig<Self>>;247	type ExecuteOverweightOrigin = frame_system::EnsureRoot<AccountId>;248}
modifiedruntime/common/config/xcm/nativeassets.rsdiffbeforeafterboth
--- a/runtime/common/config/xcm/nativeassets.rs
+++ b/runtime/common/config/xcm/nativeassets.rs
@@ -102,7 +102,8 @@
 	for UsingOnlySelfCurrencyComponents<WeightToFee, AssetId, AccountId, Currency, OnUnbalanced>
 {
 	fn new() -> Self {
-		Self(0, Zero::zero(), PhantomData)
+		// FIXME: benchmark
+		Self(Weight::from_parts(0, 0), Zero::zero(), PhantomData)
 	}
 
 	fn buy_weight(&mut self, _weight: Weight, payment: Assets) -> Result<Assets, XcmError> {
modifiedruntime/common/runtime_apis.rsdiffbeforeafterboth
--- a/runtime/common/runtime_apis.rs
+++ b/runtime/common/runtime_apis.rs
@@ -35,7 +35,7 @@
     ) => {
         use sp_std::prelude::*;
         use sp_api::impl_runtime_apis;
-        use sp_core::{crypto::KeyTypeId, OpaqueMetadata, H256, U256, H160, Bytes};
+        use sp_core::{crypto::KeyTypeId, OpaqueMetadata, H256, U256, H160};
         use sp_runtime::{
             Permill,
             traits::Block as BlockT,
@@ -250,6 +250,14 @@
                 fn metadata() -> OpaqueMetadata {
                     OpaqueMetadata::new(Runtime::metadata().into())
                 }
+
+                fn metadata_at_version(version: u32) -> Option<OpaqueMetadata> {
+                    Runtime::metadata_at_version(version)
+                }
+
+                fn metadata_versions() -> sp_std::vec::Vec<u32> {
+                    Runtime::metadata_versions()
+                }
             }
 
             impl sp_block_builder::BlockBuilder<Block> for Runtime {
@@ -311,7 +319,7 @@
                 fn account_code_at(address: H160) -> Vec<u8> {
                     use pallet_evm::OnMethodCall;
                     <Runtime as pallet_evm::Config>::OnMethodCall::get_code(&address)
-                        .unwrap_or_else(|| EVM::account_codes(address))
+                        .unwrap_or_else(|| pallet_evm::AccountCodes::<Runtime>::get(address))
                 }
 
                 fn author() -> H160 {
@@ -321,7 +329,7 @@
                 fn storage_at(address: H160, index: U256) -> H256 {
                     let mut tmp = [0u8; 32];
                     index.to_big_endian(&mut tmp);
-                    EVM::account_storages(address, H256::from_slice(&tmp[..]))
+                    pallet_evm::AccountStorages::<Runtime>::get(address, H256::from_slice(&tmp[..]))
                 }
 
                 #[allow(clippy::redundant_closure)]
@@ -401,15 +409,15 @@
                 }
 
                 fn current_transaction_statuses() -> Option<Vec<TransactionStatus>> {
-                    Ethereum::current_transaction_statuses()
+                    pallet_ethereum::CurrentTransactionStatuses::<Runtime>::get()
                 }
 
                 fn current_block() -> Option<pallet_ethereum::Block> {
-                    Ethereum::current_block()
+                    pallet_ethereum::CurrentBlock::<Runtime>::get()
                 }
 
                 fn current_receipts() -> Option<Vec<pallet_ethereum::Receipt>> {
-                    Ethereum::current_receipts()
+                    pallet_ethereum::CurrentReceipts::<Runtime>::get()
                 }
 
                 fn current_all() -> (
@@ -418,9 +426,9 @@
                     Option<Vec<TransactionStatus>>
                 ) {
                     (
-                        Ethereum::current_block(),
-                        Ethereum::current_receipts(),
-                        Ethereum::current_transaction_statuses()
+                        pallet_ethereum::CurrentBlock::<Runtime>::get(),
+                        pallet_ethereum::CurrentReceipts::<Runtime>::get(),
+                        pallet_ethereum::CurrentTransactionStatuses::<Runtime>::get()
                     )
                 }
 
@@ -653,12 +661,12 @@
 
             impl up_pov_estimate_rpc::PovEstimateApi<Block> for Runtime {
                 #[allow(unused_variables)]
-                fn pov_estimate(uxt: Bytes) -> ApplyExtrinsicResult {
+                fn pov_estimate(uxt: Vec<u8>) -> ApplyExtrinsicResult {
                     #[cfg(feature = "pov-estimate")]
                     {
                         use codec::Decode;
 
-                        let uxt_decode = <<Block as BlockT>::Extrinsic as Decode>::decode(&mut &*uxt)
+                        let uxt_decode = <<Block as BlockT>::Extrinsic as Decode>::decode(&mut &uxt)
                             .map_err(|_| DispatchError::Other("failed to decode the extrinsic"));
 
                         let uxt = match uxt_decode {
modifiedruntime/common/weights/xcm.rsdiffbeforeafterboth
--- a/runtime/common/weights/xcm.rs
+++ b/runtime/common/weights/xcm.rs
@@ -244,5 +244,9 @@
 			.saturating_add(T::DbWeight::get().reads(9_u64))
 			.saturating_add(T::DbWeight::get().writes(4_u64))
 	}
+
+	fn force_suspension() -> Weight {
+		Default::default()
+	}
 }
 
modifiedruntime/opal/Cargo.tomldiffbeforeafterboth
--- a/runtime/opal/Cargo.toml
+++ b/runtime/opal/Cargo.toml
@@ -220,7 +220,7 @@
 orml-xcm-support = { workspace = true }
 pallet-aura = { workspace = true }
 pallet-authorship = { workspace = true }
-pallet-balances = { workspace = true }
+pallet-balances = { features = ["insecure_zero_ed"], workspace = true }
 pallet-preimage = { workspace = true }
 pallet-session = { workspace = true }
 pallet-sudo = { workspace = true }
modifiedruntime/quartz/Cargo.tomldiffbeforeafterboth
--- a/runtime/quartz/Cargo.toml
+++ b/runtime/quartz/Cargo.toml
@@ -212,7 +212,7 @@
 orml-xcm-support = { workspace = true }
 pallet-aura = { workspace = true }
 pallet-authorship = { workspace = true }
-pallet-balances = { workspace = true }
+pallet-balances = { features = ["insecure_zero_ed"], workspace = true }
 pallet-preimage = { workspace = true }
 pallet-session = { workspace = true }
 pallet-sudo = { workspace = true }
modifiedruntime/tests/Cargo.tomldiffbeforeafterboth
--- a/runtime/tests/Cargo.toml
+++ b/runtime/tests/Cargo.toml
@@ -9,37 +9,37 @@
 refungible = []
 
 [dependencies]
-up-data-structs = { default-features = false, path = "../../primitives/data-structs" }
+up-data-structs = { workspace = true }
 
-sp-core = { git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.42" }
-sp-io = { git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.42" }
-sp-runtime = { git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.42" }
-sp-std = { git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.42" }
+sp-core = { workspace = true }
+sp-io = { workspace = true }
+sp-runtime = { workspace = true }
+sp-std = { workspace = true }
 
-frame-support = { git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.42" }
-frame-system = { git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.42" }
+frame-support = { workspace = true }
+frame-system = { workspace = true }
 
-pallet-balances = { git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.42" }
-pallet-timestamp = { git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.42" }
-pallet-transaction-payment = { git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.42" }
+pallet-balances = { features = ["insecure_zero_ed"], workspace = true }
+pallet-timestamp = { workspace = true }
+pallet-transaction-payment = { workspace = true }
 
-pallet-ethereum = { default-features = false, git = "https://github.com/uniquenetwork/unique-frontier", branch = "unique-polkadot-v0.9.42" }
-pallet-evm = { default-features = false, git = "https://github.com/uniquenetwork/unique-frontier", branch = "unique-polkadot-v0.9.42" }
+pallet-ethereum = { workspace = true }
+pallet-evm = { workspace = true }
 
-pallet-common.path = "../../pallets/common"
-pallet-fungible.path = "../../pallets/fungible"
-pallet-nonfungible.path = "../../pallets/nonfungible"
-pallet-refungible.path = "../../pallets/refungible"
-pallet-structure.path = "../../pallets/structure"
-pallet-unique.path = "../../pallets/unique"
+pallet-common = { workspace = true }
+pallet-fungible = { workspace = true }
+pallet-nonfungible = { workspace = true }
+pallet-refungible = { workspace = true }
+pallet-structure = { workspace = true }
+pallet-unique = { workspace = true }
 
-pallet-evm-coder-substrate.path = "../../pallets/evm-coder-substrate"
+pallet-evm-coder-substrate = { workspace = true }
 
-parity-scale-codec = { version = "3.2.2", default-features = false, features = ["derive"] }
-scale-info = "*"
+codec = { workspace = true }
+scale-info = { workspace = true }
 
 evm-coder = { workspace = true }
-up-sponsorship = { default-features = false, git = "https://github.com/uniquenetwork/pallet-sponsoring", branch = "polkadot-v0.9.42" }
+up-sponsorship = { workspace = true }
 xcm = { workspace = true }
 pallet-xcm = { workspace = true }
 pallet-configuration = { workspace = true }
modifiedruntime/unique/Cargo.tomldiffbeforeafterboth
--- a/runtime/unique/Cargo.toml
+++ b/runtime/unique/Cargo.toml
@@ -211,7 +211,7 @@
 orml-xcm-support = { workspace = true }
 pallet-aura = { workspace = true }
 pallet-authorship = { workspace = true }
-pallet-balances = { workspace = true }
+pallet-balances = { features = ["insecure_zero_ed"], workspace = true }
 pallet-preimage = { workspace = true }
 pallet-session = { workspace = true }
 pallet-sudo = { workspace = true }