Skip to content

Commit a9401cc

Browse files
knstPastaPastaPasta
authored andcommitted
chore: update list of circular dependencies
It is partial de-circularisation of dependencies between that includes net_processing Classes that still depends on net_processing but should not: - llmq/dkgsessionmgr - llmq/signing - llmq/instantsend They have asynchronous processing and with current impl that's impossible to do
1 parent f4fd143 commit a9401cc

File tree

1 file changed

+12
-19
lines changed

1 file changed

+12
-19
lines changed

test/lint/lint-circular-dependencies.sh

Lines changed: 12 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -24,20 +24,18 @@ EXPECTED_CIRCULAR_DEPENDENCIES=(
2424
"wallet/wallet -> wallet/walletdb -> wallet/wallet"
2525
"node/coinstats -> validation -> node/coinstats"
2626
# Dash
27-
"coinjoin/server -> net_processing -> coinjoin/server"
27+
"coinjoin/server -> core_io -> evo/assetlocktx -> llmq/signing -> net_processing -> coinjoin/server"
28+
"coinjoin/coinjoin -> llmq/instantsend -> net_processing -> coinjoin/server -> coinjoin/coinjoin"
2829
"dsnotificationinterface -> llmq/chainlocks -> node/blockstorage -> dsnotificationinterface"
2930
"evo/cbtx -> evo/simplifiedmns -> evo/cbtx"
3031
"evo/deterministicmns -> llmq/commitment -> evo/deterministicmns"
3132
"evo/deterministicmns -> llmq/utils -> evo/deterministicmns"
32-
"evo/mnauth -> net_processing -> evo/mnauth"
3333
"governance/classes -> governance/governance -> governance/classes"
3434
"governance/governance -> governance/object -> governance/governance"
3535
"governance/governance -> masternode/sync -> governance/governance"
36-
"governance/governance -> net_processing -> governance/governance"
3736
"llmq/quorums -> llmq/utils -> llmq/quorums"
38-
"llmq/blockprocessor -> net_processing -> llmq/blockprocessor"
3937
"llmq/chainlocks -> llmq/instantsend -> llmq/chainlocks"
40-
"llmq/chainlocks -> net_processing -> llmq/chainlocks"
38+
"llmq/chainlocks -> llmq/instantsend -> net_processing -> llmq/chainlocks"
4139
"llmq/dkgsessionmgr -> net_processing -> llmq/dkgsessionmgr"
4240
"llmq/instantsend -> net_processing -> llmq/instantsend"
4341
"llmq/instantsend -> txmempool -> llmq/instantsend"
@@ -49,45 +47,40 @@ EXPECTED_CIRCULAR_DEPENDENCIES=(
4947
"masternode/payments -> validation -> masternode/payments"
5048
"masternode/sync -> validation -> masternode/sync"
5149
"net -> netmessagemaker -> net"
52-
"net_processing -> spork -> net_processing"
5350
"netaddress -> netbase -> netaddress"
5451
"qt/appearancewidget -> qt/guiutil -> qt/appearancewidget"
5552
"qt/bitcoinaddressvalidator -> qt/guiutil -> qt/bitcoinaddressvalidator"
5653
"qt/bitcoingui -> qt/guiutil -> qt/bitcoingui"
5754
"qt/guiutil -> qt/optionsdialog -> qt/guiutil"
5855
"qt/guiutil -> qt/qvalidatedlineedit -> qt/guiutil"
5956
"core_io -> evo/cbtx -> evo/simplifiedmns -> core_io"
60-
"evo/simplifiedmns -> llmq/blockprocessor -> net_processing -> evo/simplifiedmns"
57+
"core_io -> evo/assetlocktx -> llmq/signing -> net_processing -> governance/governance -> governance/object -> core_io"
6158
"llmq/dkgsession -> llmq/dkgsessionmgr -> llmq/dkgsessionhandler -> llmq/dkgsession"
6259
"logging -> util/system -> sync -> logging"
6360
"logging -> util/system -> stacktraces -> logging"
6461
"logging -> util/system -> util/getuniquepath -> random -> logging"
65-
"coinjoin/client -> net_processing -> coinjoin/context -> coinjoin/client"
66-
"coinjoin/context -> coinjoin/server -> net_processing -> coinjoin/context"
62+
"coinjoin/context -> coinjoin/server -> core_io -> evo/assetlocktx -> llmq/signing -> net_processing -> coinjoin/context"
6763
"qt/appearancewidget -> qt/guiutil -> qt/optionsdialog -> qt/appearancewidget"
6864
"qt/guiutil -> qt/optionsdialog -> qt/optionsmodel -> qt/guiutil"
6965

7066
"bloom -> evo/assetlocktx -> llmq/quorums -> net -> bloom"
67+
"bloom -> evo/assetlocktx -> llmq/signing -> net_processing -> governance/governance -> bloom"
68+
"bloom -> evo/assetlocktx -> llmq/signing -> net_processing -> merkleblock -> bloom"
7169
"banman -> bloom -> evo/assetlocktx -> llmq/quorums -> net -> banman"
72-
"banman -> bloom -> evo/assetlocktx -> llmq/quorums -> net_processing -> banman"
73-
"bloom -> evo/assetlocktx -> llmq/quorums -> net_processing -> merkleblock -> bloom"
70+
"banman -> bloom -> evo/assetlocktx -> llmq/signing -> net_processing -> banman"
7471

75-
"coinjoin/client -> net_processing -> coinjoin/client"
76-
"llmq/quorums -> net_processing -> llmq/quorums"
72+
"coinjoin/client -> coinjoin/coinjoin -> llmq/instantsend -> net_processing -> coinjoin/client"
73+
"coinjoin/client -> coinjoin/coinjoin -> llmq/instantsend -> net_processing -> coinjoin/context -> coinjoin/client"
7774
"llmq/dkgsession -> llmq/dkgsessionmgr -> llmq/dkgsession"
7875
"llmq/chainlocks -> validation -> llmq/chainlocks"
7976
"coinjoin/coinjoin -> llmq/chainlocks -> net -> coinjoin/coinjoin"
8077
"evo/deterministicmns -> llmq/utils -> net -> evo/deterministicmns"
8178
"policy/policy -> policy/settings -> policy/policy"
8279
"evo/specialtxman -> validation -> evo/specialtxman"
8380
"consensus/tx_verify -> evo/assetlocktx -> validation -> consensus/tx_verify"
84-
"consensus/tx_verify -> evo/assetlocktx -> llmq/quorums -> net_processing -> txmempool -> consensus/tx_verify"
85-
"evo/assetlocktx -> llmq/quorums -> net_processing -> txmempool -> evo/assetlocktx"
81+
"consensus/tx_verify -> evo/assetlocktx -> llmq/signing -> net_processing -> txmempool -> consensus/tx_verify"
82+
"evo/assetlocktx -> llmq/signing -> net_processing -> txmempool -> evo/assetlocktx"
8683

87-
"evo/simplifiedmns -> llmq/blockprocessor -> net_processing -> llmq/snapshot -> evo/simplifiedmns"
88-
"llmq/blockprocessor -> net_processing -> llmq/context -> llmq/blockprocessor"
89-
"llmq/blockprocessor -> net_processing -> llmq/snapshot -> llmq/blockprocessor"
90-
"llmq/chainlocks -> net_processing -> llmq/context -> llmq/chainlocks"
9184
"llmq/context -> llmq/dkgsessionmgr -> net_processing -> llmq/context"
9285
"llmq/dkgsession -> llmq/dkgsessionmgr -> llmq/quorums -> llmq/dkgsession"
9386
"llmq/dkgsessionmgr -> llmq/quorums -> llmq/dkgsessionmgr"

0 commit comments

Comments
 (0)