Page MenuHomePhabricator

Revert "[backport#15639] bitcoin-wallet tool: Drop libbitcoin_server.a dependency"
ClosedPublic

Authored by majcosta on May 27 2020, 17:00.

Details

Summary

This reverts commit a7a55fb3ac4f7f8b6dd97fa51b4f2cd21a871022, turns out there's still work to be done before that dependency can be dropped

it also:

  • removes redundant target_link_libraries calls related to the wallet <-> server circular dependency
  • removes a libevent dependency from wallet since the only function that needed it was moved to src/util/url.cpp
  • added an explicit libevent dependency to util, and moved the code that finds libevent to account for that

Depends on D6256

Test Plan
../configure
make

cmake .. -GNinja
ninja

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

deadalnix requested changes to this revision.May 27 2020, 17:13
deadalnix added a subscriber: deadalnix.
deadalnix added inline comments.
src/CMakeLists.txt
584 ↗(On Diff #20646)

This is redundant witth he change you made in the wallet cmake file.

592 ↗(On Diff #20646)

This is unecessary because wallt-tool depend on wallet and wallet depend on server

src/Makefile.am
685 ↗(On Diff #20646)

likestamp

This revision now requires changes to proceed.May 27 2020, 17:13

removed redundant link commands and removed obsolete event dep

Snippet of first build failure:

2020-05-27T17:27:32.383000Z TestFramework (INFO): Test skipped
[17:28:19]i:		 [Bitcoin ABC functional tests] wallet_resendwallettransactions.py
[17:28:19]i:			 [wallet_resendwallettransactions.py] Test ignored: wallet_resendwallettransactions.py
[17:28:19]i:		 [Bitcoin ABC functional tests] [Test Output]
2020-05-27T17:28:00.145000Z TestFramework (WARNING): Test Skipped: wallet has not been compiled.
2020-05-27T17:28:00.195000Z TestFramework (INFO): Stopping nodes
2020-05-27T17:28:00.195000Z TestFramework (INFO): Cleaning up /work/build/test/tmp/test_runner_₿₵_  _20200527_172731/wallet_resendwallettransactions_68 on exit
2020-05-27T17:28:00.195000Z TestFramework (INFO): Test skipped
[17:28:19]i:		 [Bitcoin ABC functional tests] wallet_txn_clone.py
[17:28:19]i:			 [wallet_txn_clone.py] Test ignored: wallet_txn_clone.py
[17:28:19]i:		 [Bitcoin ABC functional tests] [Test Output]
2020-05-27T17:28:06.820000Z TestFramework (WARNING): Test Skipped: wallet has not been compiled.
2020-05-27T17:28:06.870000Z TestFramework (INFO): Stopping nodes
2020-05-27T17:28:06.871000Z TestFramework (INFO): Cleaning up /work/build/test/tmp/test_runner_₿₵_  _20200527_172731/wallet_txn_clone_86 on exit
2020-05-27T17:28:06.871000Z TestFramework (INFO): Test skipped
[17:28:19]i:		 [Bitcoin ABC functional tests] wallet_txn_clone.py --mineblock
[17:28:19]i:			 [wallet_txn_clone.py --mineblock] Test ignored: wallet_txn_clone.py --mineblock
[17:28:19]i:		 [Bitcoin ABC functional tests] [Test Output]
2020-05-27T17:28:00.233000Z TestFramework (WARNING): Test Skipped: wallet has not been compiled.
2020-05-27T17:28:00.283000Z TestFramework (INFO): Stopping nodes
2020-05-27T17:28:00.283000Z TestFramework (INFO): Cleaning up /work/build/test/tmp/test_runner_₿₵_  _20200527_172731/wallet_txn_clone_69 on exit
2020-05-27T17:28:00.283000Z TestFramework (INFO): Test skipped
[17:28:19]i:		 [Bitcoin ABC functional tests] wallet_txn_doublespend.py
[17:28:19]i:			 [wallet_txn_doublespend.py] Test ignored: wallet_txn_doublespend.py
[17:28:19]i:		 [Bitcoin ABC functional tests] [Test Output]
2020-05-27T17:28:14.047000Z TestFramework (WARNING): Test Skipped: wallet has not been compiled.
2020-05-27T17:28:14.097000Z TestFramework (INFO): Stopping nodes
2020-05-27T17:28:14.097000Z TestFramework (INFO): Cleaning up /work/build/test/tmp/test_runner_₿₵_  _20200527_172731/wallet_txn_doublespend_118 on exit
2020-05-27T17:28:14.097000Z TestFramework (INFO): Test skipped
[17:28:19]i:		 [Bitcoin ABC functional tests] wallet_txn_doublespend.py --mineblock
[17:28:19]i:			 [wallet_txn_doublespend.py --mineblock] Test ignored: wallet_txn_doublespend.py --mineblock
[17:28:19]i:		 [Bitcoin ABC functional tests] [Test Output]
2020-05-27T17:28:00.269000Z TestFramework (WARNING): Test Skipped: wallet has not been compiled.
2020-05-27T17:28:00.319000Z TestFramework (INFO): Stopping nodes
2020-05-27T17:28:00.319000Z TestFramework (INFO): Cleaning up /work/build/test/tmp/test_runner_₿₵_  _20200527_172731/wallet_txn_doublespend_70 on exit
2020-05-27T17:28:00.320000Z TestFramework (INFO): Test skipped
[17:28:19]i:		 [Bitcoin ABC functional tests] wallet_zapwallettxes.py
[17:28:19]i:			 [wallet_zapwallettxes.py] Test ignored: wallet_zapwallettxes.py
[17:28:19]i:		 [Bitcoin ABC functional tests] [Test Output]
2020-05-27T17:27:53.012000Z TestFramework (WARNING): Test Skipped: wallet has not been compiled.
2020-05-27T17:27:53.063000Z TestFramework (INFO): Stopping nodes
2020-05-27T17:27:53.063000Z TestFramework (INFO): Cleaning up /work/build/test/tmp/test_runner_₿₵_  _20200527_172731/wallet_zapwallettxes_51 on exit
2020-05-27T17:27:53.064000Z TestFramework (INFO): Test skipped
[17:28:19]i:		 [Bitcoin ABC functional tests] p2p_timeouts.py
[17:28:19]i:			 [p2p_timeouts.py] [Test Output]
2020-05-27T17:27:52.750000Z TestFramework (INFO): Initializing test directory /work/build/test/tmp/test_runner_₿₵_  _20200527_172731/p2p_timeouts_48
2020-05-27T17:27:58.064000Z TestFramework (ERROR): Assertion failed
Traceback (most recent call last):
  File "/work/test/functional/test_framework/test_framework.py", line 163, in main
    self.run_test()
  File "/work/test/functional/p2p_timeouts.py", line 84, in run_test
    assert not no_verack_node.is_connected
AssertionError
2020-05-27T17:27:58.747000Z TestFramework (INFO): Stopping nodes
2020-05-27T17:27:59.001000Z TestFramework (WARNING): Not cleaning up dir /work/build/test/tmp/test_runner_₿₵_  _20200527_172731/p2p_timeouts_48
2020-05-27T17:27:59.001000Z TestFramework (ERROR): Test failed. Test logging available at /work/build/test/tmp/test_runner_₿₵_  _20200527_172731/p2p_timeouts_48/test_framework.log
2020-05-27T17:27:59.001000Z TestFramework (ERROR): Hint: Call /work/test/functional/combine_logs.py '/work/build/test/tmp/test_runner_₿₵_  _20200527_172731/p2p_timeouts_48' to consolidate all logs
[17:28:24]E:	 [Step 1/2] Failed tests detected
[17:28:28]W:	 [Step 1/2] Process exited with code 1
[17:28:28]E:	 [Step 1/2] Process exited with code 1 (Step: Command Line)

Each failure log is accessible here:
Bitcoin ABC functional tests: p2p_timeouts.py

deadalnix requested changes to this revision.May 27 2020, 17:31
deadalnix added inline comments.
src/wallet/CMakeLists.txt
10 ↗(On Diff #20647)

?

This revision now requires changes to proceed.May 27 2020, 17:31

cleaned up event leftovers

add explicit libevent dependency to util library

This revision is now accepted and ready to land.May 27 2020, 18:42
This revision was landed with ongoing or failed builds.May 27 2020, 18:44
This revision was automatically updated to reflect the committed changes.