Skip to content

Some adjustments to versi tooling#167

Closed
michalkucharczyk wants to merge 3 commits intomainfrom
mku-versi-fixes
Closed

Some adjustments to versi tooling#167
michalkucharczyk wants to merge 3 commits intomainfrom
mku-versi-fixes

Conversation

@michalkucharczyk
Copy link
Copy Markdown

Adjustments to the versi tooling which is a follow-up of paritytech/polkadot-sdk#1256.

},
"configuration": {
"config": {
"scheduling_lookahead": 2,
Copy link
Copy Markdown
Author

@michalkucharczyk michalkucharczyk Mar 5, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any corresponding parameter that replaces this? (best to see the diff w/o whitespaces).

@michalkucharczyk
Copy link
Copy Markdown
Author

cc: @altonen

@altonen
Copy link
Copy Markdown

altonen commented Mar 5, 2024

I wonder is there something wrong with merging Versi overrides with Rococo chainspec. If I call build-spec for chainspec-rococo-staging-plain.json, it works but if I call build-spec for chainspec-plain.json, I get the following error:

2024-03-05 14:11:27 Building chain spec
Error:
   0: Other: Error parsing spec file: expected value at line 253 column 5

  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ BACKTRACE ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
                                ⋮ 5 frames hidden ⋮
   6: <core::result::Result<T,F> as core::ops::try_trait::FromResidual<core::result::Result<core::convert::Infallible,E>>>::from_residual::h2448ea279a46891e
      at /rustc/82e1608dfa6e0b5569232559e3d385fea5a93112/library/core/src/result.rs:1963
   7: polkadot::main::h207b72b9e2003354
      at /builds/parity/mirrors/polkadot-sdk/polkadot/src/main.rs:31
   8: core::ops::function::FnOnce::call_once::hc88137fa2e13fa37
      at /rustc/82e1608dfa6e0b5569232559e3d385fea5a93112/library/core/src/ops/function.rs:250
                                ⋮ 14 frames hidden ⋮

Are you able to reproduce this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants