💬 instagibbs commented on pull request "fuzz: Minor improvements to tx_package_eval target":
(https://github.com/bitcoin/bitcoin/pull/28825#discussion_r1391191028)
will do if I touch again

💬 hebasto commented on pull request "depends: Build the `native_capnp` and `capnp` packages with CMake":
(https://github.com/bitcoin/bitcoin/pull/28856#issuecomment-1808284816)
> Any reason to not change `native_capnp` at the same time?

Done.

👍 theStack approved a pull request: "doc: rewrite explanation for `-par=`"
(https://github.com/bitcoin/bitcoin/pull/28858#pullrequestreview-1727479651)
ACK d799ea26edfd63434b6d1cf55500de184dc67fac

💬 Riahiamirreza commented on pull request "rpc: show P2(W)SH redeemScript in getrawtransaction #27637":
(https://github.com/bitcoin/bitcoin/pull/27638#issuecomment-1808308529)
```feature_abortnode.py | ✖ Failed | 1 s
feature_bip68_sequence.py | ✖ Failed | 1 s
feature_cltv.py | ✖ Failed | 1 s
feature_coinstatsindex.py | ✖ Failed | 2 s
feature_config_args.py | ✖ Failed | 10 s
feature_csv_activation.py | ✖ Failed | 1 s
feature_dersig.py
...

💬 Sjors commented on pull request "build: LLD based macOS toolchain":
(https://github.com/bitcoin/bitcoin/pull/21778#issuecomment-1808312187)
Guix building df71f448cdaf429f54367ab251e21e628c1903d3 is having issues for me:

```
Extracting libevent...
/home/guix/depends-sources-cache/libevent-2.1.12-stable.tar.gz: OK
Preprocessing libevent...
Configuring libevent...
checking for a BSD-compatible install... /home/guix/.guix-profile/bin/install -c
checking whether build environment is sane... yes
checking for x86_64-apple-darwin-strip... no
checking for strip... strip
configure: WARNING: using cross tools not prefixed with host
...

💬 fanquake commented on pull request "build: LLD based macOS toolchain":
(https://github.com/bitcoin/bitcoin/pull/21778#issuecomment-1808313723)
> Can add more details...

I'd suggest reviewing the base PRs instead, for now.

💬 maflcko commented on pull request "rpc: show P2(W)SH redeemScript in getrawtransaction #27637":
(https://github.com/bitcoin/bitcoin/pull/27638#issuecomment-1808316213)
> These are the currently failing tests.

Failing why? Without any information, there is nothing we can do here.

Recall that the macOS CI passes on this pull request currently.

This looks like a local problem on your side.

💬 sipa commented on pull request "rpc: show P2(W)SH redeemScript in getrawtransaction #27637":
(https://github.com/bitcoin/bitcoin/pull/27638#discussion_r1391227222)
This code seems to assume that the 4th scriptSig push is the redeemScript. That won't always be correct; it's the last one.

💬 sipa commented on pull request "rpc: show P2(W)SH redeemScript in getrawtransaction #27637":
(https://github.com/bitcoin/bitcoin/pull/27638#discussion_r1391224005)
I don't think this is correct. P2SH spends are identified by the *spent scriptPubKey* having the right form. This will, I suspect, not have any effect; the scriptSig is generally push-only (and non-standard otherwise), so it'll never match the P2SH template.

💬 sipa commented on pull request "test, refactor: Magic bytes array followup":
(https://github.com/bitcoin/bitcoin/pull/28857#issuecomment-1808326518)
utACK 1e5b86171e81ab4b022b9746bb06e1968ecf4086

💬 Riahiamirreza commented on pull request "rpc: show P2(W)SH redeemScript in getrawtransaction #27637":
(https://github.com/bitcoin/bitcoin/pull/27638#issuecomment-1808327245)
> Failing why? Without any information, there is nothing we can do here.
As the exception suggests it's due the addition of the new field `redeemScript`.
Am I right?
```
{a.riahi@bitcoin} > python3.10 ./test/functional/rpc_generate.py
2023-11-13T14:57:56.863000Z TestFramework (INFO): PRNG seed is: 2176820945159495690
2023-11-13T14:57:56.863000Z TestFramework (INFO): Initializing test directory /tmp/bitcoin_func_test_nt1lv9on
2023-11-13T14:57:57.137000Z TestFramework (INFO): Test rpc gener
...

💬 theuni commented on pull request "guix: update time-machine":
(https://github.com/bitcoin/bitcoin/pull/28580#issuecomment-1808336780)
> LLVM 16 & 17 are now available. This is ready for review. This bump also contains the recently released [Mes 0.25](https://www.mail-archive.com/info-gnu@gnu.org/msg03232.html), which should improve the bootstrapping experience for RISC-V. cc @laanwj who upstreamed a bunch of the changes there.

This is _major_ and unblocks a bunch of toolchain bumps we've needed to do for years. Great work from everyone involved. Thanks @fanquake for pushing on the guix side.

💬 hebasto commented on pull request "guix: update time-machine":
(https://github.com/bitcoin/bitcoin/pull/28580#issuecomment-1808337570)
My Guix builds:
```
x86_64
13a7d1be447ecb614cf43034af4f7a3a7ce7dffbcdb6c1773bc939ba80587ef6 guix-build-92d12f1c8903/output/aarch64-linux-gnu/SHA256SUMS.part
5ab66c69b742a89b0aa52705e48563749cb72ebcc92745a4eb07df285a20c62a guix-build-92d12f1c8903/output/aarch64-linux-gnu/bitcoin-92d12f1c8903-aarch64-linux-gnu-debug.tar.gz
d3224eb0eb66bf4433ed8757667ed438e419db4240b06d76122e8754de241742 guix-build-92d12f1c8903/output/aarch64-linux-gnu/bitcoin-92d12f1c8903-aarch64-linux-gnu.tar.gz
9f5ecd1d
...

💬 Riahiamirreza commented on pull request "rpc: show P2(W)SH redeemScript in getrawtransaction #27637":
(https://github.com/bitcoin/bitcoin/pull/27638#discussion_r1391241361)
Is there any convenient way to detect redeemScript in the script?

👍 hebasto approved a pull request: "guix: update time-machine"
(https://github.com/bitcoin/bitcoin/pull/28580#pullrequestreview-1727523917)
ACK 92d12f1c890350f40d8e5d5c6a59d5c172ea7550.

💬 Riahiamirreza commented on pull request "rpc: show P2(W)SH redeemScript in getrawtransaction #27637":
(https://github.com/bitcoin/bitcoin/pull/27638#discussion_r1391244227)
So can I always expect the last `scriptSig` push to be the `redeemScript` if it has one?

💬 theuni commented on pull request "build: remove `-bind_at_load` usage":
(https://github.com/bitcoin/bitcoin/pull/28783#issuecomment-1808343838)
utACK 3c61c60b90db1b6a77b3804784430fcd57b447b6.

💬 sipa commented on pull request "rpc: show P2(W)SH redeemScript in getrawtransaction #27637":
(https://github.com/bitcoin/bitcoin/pull/27638#discussion_r1391246186)
Yes, BIP16 states that the redeemScript is the last push of the scriptSig.

💬 sipa commented on pull request "rpc: show P2(W)SH redeemScript in getrawtransaction #27637":
(https://github.com/bitcoin/bitcoin/pull/27638#issuecomment-1808346467)
@Riahiamirreza To fix that error you'll need to add your new field to the documentation of the affected RPCs.

💬 TheCharlatan commented on pull request "depends: latest config.guess & config.sub":
(https://github.com/bitcoin/bitcoin/pull/28781#issuecomment-1808354390)
ACK 49a92579c705831c7ffbcfb24cdf17a94e9a11a0

Checked the content through https://www.gnu.org/software/gettext/manual/html_node/config_002eguess.html

Guix build (x86_64):
```
83eba00a50fe023bce7066676dc15ae5eed1670a4ae3afe89d6c58d371786551 guix-build-49a92579c705/output/aarch64-linux-gnu/SHA256SUMS.part
6b49be1b06c77ff1265af5856a9e4475ee8dd70eee7556c0481a404e356443e5 guix-build-49a92579c705/output/aarch64-linux-gnu/bitcoin-49a92579c705-aarch64-linux-gnu-debug.tar.gz
b748937e55ffe6314b
...