merge-script
80e6ad9e30
Merge bitcoin/bitcoin#31250 : wallet: Disable creating and loading legacy wallets
...
CI / test each commit (push) Waiting to run
CI / macOS 14 native, arm64, no depends, sqlite only, gui (push) Waiting to run
CI / macOS 14 native, arm64, fuzz (push) Waiting to run
CI / Windows native, VS 2022 (push) Waiting to run
CI / Windows native, fuzz, VS 2022 (push) Waiting to run
CI / Linux->Windows cross, no tests (push) Waiting to run
CI / Windows, test cross-built (push) Blocked by required conditions
CI / ASan + LSan + UBSan + integer, no depends, USDT (push) Waiting to run
17bb63f9f9
wallet: Disallow loading legacy wallets (Ava Chow)
9f04e02ffa
wallet: Disallow creating legacy wallets (Ava Chow)
6b247279b7
wallet: Disallow legacy wallet creation from the wallet tool (Ava Chow)
5e93b1fd6c
bench: Remove WalletLoadingLegacy benchmark (Ava Chow)
56f959d829
wallet: Remove wallettool salvage (Ava Chow)
7a41c939f0
wallet: Remove -format and bdb from wallet tool's createfromdump (Ava Chow)
c847dee148
test: remove legacy wallet functional tests (Ava Chow)
20a9173717
test: Remove legacy wallet tests from wallet_reindex.py (Ava Chow)
446d480cb2
test: Remove legacy wallet tests from wallet_backwards_compatibility.py (Ava Chow)
aff80298d0
test: wallet_signer.py bdb will be removed (Ava Chow)
f94f9399ac
test: Remove legacy wallet unit tests (Ava Chow)
d9ac9dbd8e
tests, gui: Use descriptors watchonly wallet for watchonly test (Ava Chow)
Pull request description:
To prepare for the deletion of legacy wallet code, disable creating or loading new legacy wallets.
Tests for the legacy wallet specifically are deleted.
Split from https://github.com/bitcoin/bitcoin/pull/28710
ACKs for top commit:
Sjors:
re-ACK 17bb63f9f9
pablomartin4btc:
re-ACK 17bb63f9f9
laanwj:
re-ACK 17bb63f9f9
Tree-SHA512: d7a86df1f71f12451b335f22f7c3f0394166ac3f8f5b81f6bbf0321026e2e8ed621576656c371d70e202df1be4410b2b1c1acb5d5f0c341e7b67aaa0ac792e7c
2025-04-25 13:11:24 +01:00
MarcoFalke
facb9b327b
scripted-diff: Use bpf_cflags
...
-BEGIN VERIFY SCRIPT-
ren() { sed --regexp-extended -i "s/$1/$2/g" $( git grep --extended-regexp -l "$1" ) ; }
ren 'cflags=\["-Wno-error=implicit-function-declaration"\]' 'cflags=bpf_cflags()'
-END VERIFY SCRIPT-
2025-04-24 10:47:57 +02:00
MarcoFalke
fa0c1baaf8
test: Add imports for util bpf_cflags
...
This is required for the next commit.
2025-04-24 10:47:44 +02:00
Ava Chow
c847dee148
test: remove legacy wallet functional tests
...
Removes all legacy wallet specific functional tests.
Also removes the --descriptor and --legacy-wallet options as these are
no longer necessary with the legacy wallet removed.
2025-04-23 12:10:30 -07:00
0xb10c
35ae6ff60f
test: don't use bpf_usdt_readarg_p
...
Instead of using the undocumented bcc helper bpf_usdt_readarg_p(),
use bpf_usdt_readarg() [1] and bpf_probe_read_user{_str}() [2, 3] as
documented in the bcc USDT reference guide [1].
Note that the bpf_probe_read_user() documentation says the following:
> For safety, all user address space memory reads must pass through bpf_probe_read_user().
It's assumed that using bpf_usdt_readarg_p() caused a lifetime issue.
See https://github.com/bitcoin/bitcoin/issues/27380#issuecomment-2286505348
With bpf_usdt_readarg() and bpf_probe_read_user(), this doesn't seem
to be a problem anymore. See https://github.com/bitcoin/bitcoin/issues/27380#issuecomment-2528671656
[1]: https://github.com/iovisor/bcc/blob/master/docs/reference_guide.md#6-usdt-probes
[2]: https://github.com/iovisor/bcc/blob/master/docs/reference_guide.md#10-bpf_probe_read_user
[3]: https://github.com/iovisor/bcc/blob/master/docs/reference_guide.md#11-bpf_probe_read_user_str
2025-02-12 16:21:35 +01:00
MarcoFalke
faaf3e53f0
test: [refactor] Fix F841 flake8
2024-08-29 07:01:54 +02:00
Hennadii Stepanov
a0473442d1
scripted-diff: Add __file__
argument to BitcoinTestFramework.init()
...
-BEGIN VERIFY SCRIPT-
sed -i -e 's/\s*().main\s*()/(__file__).main()/' $(git ls-files test/functional/*.py)
sed -i -e 's/def __init__(self)/def __init__(self, test_file)/' test/functional/test_framework/test_framework.py
-END VERIFY SCRIPT-
2024-07-16 22:06:47 +01:00
Richard Myers
2d58629ee6
wallet: fix coin selection tracing to return -1 when no change pos
2024-01-20 14:56:41 +01:00
willcl-ark
4077e43bf6
test: fix usdt undeclared function errors on mantis
...
Recently usage of undeclared functions became an error rather than a
warning, in C2x. https://reviews.llvm.org/D122983?id=420290
This change has migrated into the build tools of Ubuntu 23.10 which now
causes the USDT tests to fail to compile, see
https://github.com/bitcoin/bitcoin/issues/28600
Fix this by setting `-Wno-error=implicit-function-declaration` for the
tracing programs.
2023-10-09 21:01:38 +01:00
MacroFake
555519d082
test: Remove wallet option from non-wallet tests
...
Review note: The changes are complete, because self.options.descriptors
is set to None in parse_args (test_framework.py).
A value of None implies -disablewallet, see the previous commit.
So if a call to add_wallet_options is missing, it will lead to a test
failure when the wallet is compiled in.
2022-11-10 17:19:13 +01:00
Andrew Chow
ab5af9ca72
test: Add test for coinselection tracepoints
2022-04-21 11:17:00 -04:00