upgrade to polkadot-v1.0.0

[brenzi]

noteworthy causes for change:

• no more T::BlockNumber Moves Block to frame_system instead of construct_runtime and removes Header and BlockNumber paritytech/substrate#14437
• deprecated GenesisBuild: GenesisBuild<T,I> deprecated. BuildGenesisConfig added. paritytech/substrate#14306
• Moves Block to frame_system instead of construct_runtime and removes Header and BlockNumber paritytech/substrate#14437
• Replace system config Index for Nonce paritytech/substrate#14290
• frame: support for serde added paritytech/substrate#14261

once merged, reset branch polkadot-v1.0.0

[brenzi]

still getting error logs during tests, deemed insubstantial:
Jul 22 16:34:17.299 ERROR runtime::system: Logic error: Account already dead when reducing provider

[brenzi]

hmm, this may be relevant too, even if older: paritytech/substrate#14261

[clangenb]

looks good in general, some minor questions, and I might have spotted the missing serialize implementation error.

[clangenb]

Jul 22 16:34:17.299 ERROR runtime::system: Logic error: Account already dead when reducing provider

I made an issue of this. This is suspicious, in the worst case, it could mean that we forget to increment a provider count when we should, and as a result we might kill an account, which should not be killed yet. #333

[clangenb]

hmm, this may be relevant too, even if older: paritytech/substrate#14261

Hmm, this probably means that we need to enable the serde_derive feature here: https://github.com/encointer/encointer-node/blob/0e4b92736783483ec35d21ba09fc0d20fdad9346/runtime/Cargo.toml#L21, which we currently only do in std.

Unfortunately, I just tried it and it seems to introduce an std-leackage....

[clangenb]

hmm, this may be relevant too, even if older: paritytech/substrate#14261

Hmm, this probably means that we need to enable the serde_derive feature here: https://github.com/encointer/encointer-node/blob/0e4b92736783483ec35d21ba09fc0d20fdad9346/runtime/Cargo.toml#L21, which we currently only do in std.

Unfortunately, I just tried it and it seems to introduce an std-leackage....

I should have checked it more thoroughly, it does fix the runtime build. It is just the node, which also needs some fixes then. However, if the node requires this flag now in any case, it is better to enable encointer-primitive's serde_derive flag from within the pallets, such that the node does not need to care about it.

[clangenb]

LGTM