tersec
tersec
``` 2024-10-08T03:19:48.2521970Z /github-runner/workspace/nimbus-eth2/nimbus-eth2/beacon_chain/spec/datatypes/electra.nim(306, 3) Error: redefinition of 'ExecutionRequests'; previous declaration here: /github-runner/workspace/nimbus-eth2/nimbus-eth2/beacon_chain/spec/datatypes/electra.nim(193, 3) ```
``` 2024-10-08T12:12:55.6224986Z /github-runner/workspace/nimbus-eth2/nimbus-eth2/beacon_chain/spec/datatypes/epbs.nim(37, 23) Error: undeclared identifier: 'PendingBalanceDeposit' 2024-10-08T12:12:55.6458434Z make: *** [Makefile:447: wss_sim] Error 1 -- 2024-10-08T12:12:56.6417447Z /github-runner/workspace/nimbus-eth2/nimbus-eth2/beacon_chain/spec/datatypes/epbs.nim(37, 23) Error: undeclared identifier: 'PendingBalanceDeposit' 2024-10-08T12:12:56.6684431Z make: *** [Makefile:447: ncli_db] Error 1...
Haven't checked in detail yet, but something went awry with that merge. Some conflicts that I don't think were there before: 
> Strange there are conflicts in vendor packages. They aren't conflicts in the vendor packages as such, but different versions of the packages specified somehow in the git repository history...
The files: - the binary executable `tests/consensus_spec/bellatrix/test_fixture_operations` (the one without the `.nim` extension); - the binary executable `tests/consensus_spec/electra/test_fixture_operations` (ditto, the one without the `.nim` extension) - the binary executable `beacon_chain/spec/helpers`...
Current build failure (locally reproducible via `make all_tests`): ``` /github-runner/workspace/nimbus-eth2/nimbus-eth2/tests/test_helpers.nim(276, 33) Error: type mismatch Expression: remove_flag(flag, flag_index) [1] flag: uint8 [2] flag_index: int Expected one of (first mismatch at [position]):...
``` /github-runner/workspace/nimbus-eth2/nimbus-eth2/vendor/nim-unittest2/unittest2.nim: In function ‘_ZN23test_message_signatures20runTestX60gensym450_E6string6string’: /github-runner/workspace/nimbus-eth2/nimbus-eth2/vendor/nim-unittest2/unittest2.nim:1101:15: error: stack usage might be 5658432 bytes [-Werror=stack-usage=] 1101 | proc runTest(suiteName, testName: string): TestStatus {.raises: [], gensym.} = | ^ /github-runner/workspace/nimbus-eth2/nimbus-eth2/vendor/nim-unittest2/unittest2.nim: In function...
Seems ORC-only. `refc` doesn't seem affected.
> Maybe it’s time to update this and remove the response body altogether for Fulu – and instead explicitly require the builder to publish the block and blobs? That feels...