Compare commits

...

4 commits

Author SHA1 Message Date
Hennadii Stepanov
fdd1f9798b
Merge 2083e9cf05 into c5e44a0435 2025-04-29 11:56:22 +02:00
merge-script
c5e44a0435
Merge bitcoin/bitcoin#32369: test: Use the correct node for doubled keypath test
Some checks are pending
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
CI / test each commit (push) Waiting to run
CI / macOS 14 native, arm64, no depends, sqlite only, gui (push) Waiting to run
32d55e28af test: Use the correct node for doubled keypath test (Ava Chow)

Pull request description:

  #29124 had a silent merge conflict with #32350 which resulted in it using the wrong node. Fix the test to use the correct v22 node.

ACKs for top commit:
  maflcko:
    lgtm ACK 32d55e28af
  rkrux:
    ACK 32d55e28af
  BrandonOdiwuor:
    Code Review ACK 32d55e28af

Tree-SHA512: 1e0231985beb382b16e1d608c874750423d0502388db0c8ad450b22d17f9d96f5e16a6b44948ebda5efc750f62b60d0de8dd20131f449427426a36caf374af92
2025-04-29 09:59:42 +01:00
Ava Chow
32d55e28af test: Use the correct node for doubled keypath test 2025-04-28 14:44:17 -07:00
Hennadii Stepanov
2083e9cf05
cmake: Fix -pthread flags in summary 2025-01-23 18:40:24 +00:00
2 changed files with 13 additions and 2 deletions

View file

@ -32,7 +32,18 @@ endfunction()
function(get_target_interface var config target property)
get_target_property(result ${target} INTERFACE_${property})
if(result)
evaluate_generator_expressions(result "${config}")
# On systems where pthread functionality is not provided by
# the C library implementation, the CMake FindThreads module
# sets the Threads::Threads target's compile options to
# generator expressions that evaluate to `-pthread` in this
# project.
# To improve the readability of the configuration summary,
# we skip these generator expressions.
if(${target} STREQUAL "Threads::Threads" AND ${property} STREQUAL "COMPILE_OPTIONS")
set(result -pthread)
else()
evaluate_generator_expressions(result "${config}")
endif()
list(JOIN result " " result)
else()
set(result)

View file

@ -87,7 +87,7 @@ class BackwardsCompatibilityTest(BitcoinTestFramework):
# 0.21.x and 22.x would both produce bad derivation paths when topping up an inactive hd chain
# Make sure that this is being automatically cleaned up by migration
node_master = self.nodes[1]
node_v22 = self.nodes[self.num_nodes - 5]
node_v22 = self.nodes[self.num_nodes - 3]
wallet_name = "bad_deriv_path"
node_v22.createwallet(wallet_name=wallet_name, descriptors=False)
bad_deriv_wallet = node_v22.get_wallet_rpc(wallet_name)