Skip to content

qt: require Network for walletless Ord GUI build - #1

Closed
Joopooranski wants to merge 1 commit into
StanCan321:codex/ord-pr1-testsfrom
Joopooranski:joop-ord-qt-network
Closed

Joopooranski wants to merge 1 commit into
StanCan321:codex/ord-pr1-testsfrom
Joopooranski:joop-ord-qt-network

Conversation

@Joopooranski

Copy link
Copy Markdown

Follow-up to bitcoindogmode#12. The Ord GUI code links Qt6::Network even when wallet support is disabled, but the root CMake configuration currently discovers that Qt component only with ENABLE_WALLET=ON.

Reproduce on the Ord branch with cmake -S . -B build -DBUILD_GUI=ON -DENABLE_WALLET=OFF -DENABLE_IPC=OFF: generation fails because target Qt6::Network was not found. Include Network in the GUI component list unconditionally and remove the wallet-only append.

Validation on Ubuntu 24.04 with Qt 6.4.2: the same CMake configuration and generation succeed after this change. bitcoin.cpp.o and ordmanager.cpp.o compile successfully in the wallet-disabled build. The full bitcoin-qt link was not completed locally.

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.

1 participant