Bitcoin Core Github
44 subscribers
121K links
Download Telegram
💬 0xB10C commented on issue "CI, tracing: run tracepoint interface tests in the CI":
(https://github.com/bitcoin/bitcoin/issues/24782#issuecomment-1492963404)
The up coming Ubuntu 23.04 seems to be shipping with an up-to-date bpfcc: https://packages.ubuntu.com/lunar/python3-bpfcc
💬 0xB10C commented on pull request "Move log messages: tx enqueue to mempool, allocation to blockstorage":
(https://github.com/bitcoin/bitcoin/pull/27277#issuecomment-1492964604)
> CI network error https://cirrus-ci.com/task/4864754720702464?logs=ci#L61:
>
> `TimeoutError: [Errno 110] Connection timed out`

Note: The failed connections seems to be to the launchpad.net PPA with a newer bpfcc. Not using this newer PPA to install an up-to-date `bpfcc` causes warnings printed to stderr which means `interface_usdt_coinselection.py` is treated as failed. (The up-coming Ubuntu lunar seems to include an up-to-date bpfcc).
💬 DRAWCORE commented on issue "get block or transaction data via rpc api when in the ibd process":
(https://github.com/bitcoin/bitcoin/issues/27388#issuecomment-1492968313)
> Yes this should work, did you try it? Here's example commands starting bitcoind in a fresh datadir with `-txindex`:
>
> ```
> --> bitcoin-cli -datadir=/home/pinheadmz/bcd-test getblockcount
> 400
>
> --> bitcoin-cli -datadir=/home/pinheadmz/bcd-test getblock 000000007bc154e0fa7ea32218a72fe2c1bb9f86cf8c9ebf9a715ed27fdb229a
> {
> "hash": "000000007bc154e0fa7ea32218a72fe2c1bb9f86cf8c9ebf9a715ed27fdb229a",
> "confirmations": 9282,
> "strippedsize": 215,
> "size": 215,
> "we
...
💬 pinheadmz commented on issue "get block or transaction data via rpc api when in the ibd process":
(https://github.com/bitcoin/bitcoin/issues/27388#issuecomment-1492970384)
Yes with the rpc you can access anything that you have already downloaded
💬 DevAgrawal1112 commented on pull request "#24049 Issue: Update nScore datatype":
(https://github.com/bitcoin/bitcoin/pull/27386#issuecomment-1492981982)
@hebasto Yes, I had read previous discussions that are mention in [thread](https://github.com/bitcoin/bitcoin/pull/24090)
⚠️ Sjors opened an issue: "rpc: show P2(W)SH redeemScript in getrawtransaction (and friends)"
(https://github.com/bitcoin/bitcoin/issues/27391)
### Please describe the feature you'd like to see added.

I'd like to be able to get the (decompiled) P2SH `redeemScript` and P2WSH `witnessScript` when calling `getrawtransaction` and `getblock … 2`.

### Is your feature related to a problem, if so please describe it.

I found myself having a hard time counting sigops in the recently invalid block. For scriptPubKey it's easy:

```
# Raw OP_CHECKMULTISIG: 20 x 4 sigops each
bitcoin-cli getblock 00000000000000000002ec935e245f8ae70fc68cc828f05
...
🚀 hebasto merged a pull request: "Update translation source file for v25.0 string freeze"
(https://github.com/bitcoin-core/gui/pull/724)
💬 hebasto commented on pull request "Adjust plural forms for translations":
(https://github.com/bitcoin-core/gui/pull/716#issuecomment-1493123198)
Closed in favour of #724.
hebasto closed a pull request: "Adjust plural forms for translations"
(https://github.com/bitcoin-core/gui/pull/716)
💬 hebasto commented on pull request "Translatability fixups":
(https://github.com/bitcoin-core/gui/pull/599#issuecomment-1493123474)
Most of commits from this PR were merged as a part of #724.
💬 Ayush170-Future commented on pull request "addrman, refactor: improve stochastic test in `AddSingle`":
(https://github.com/bitcoin/bitcoin/pull/27319#issuecomment-1493166757)
Yeah, the left shift should be faster than the iterative approach. Yet I'm not sure how much of an improvement it will make in terms of efficiency.

But, there are other instances in the same file where the Left shift is already being used for the same purpose. As a result, I see no point in continuing to use the current iterative method for calculating the power of two. It's better to change it for the sake of consistency and better practice.
👍 Ayush170-Future approved a pull request: "addrman, refactor: improve stochastic test in `AddSingle`"
(https://github.com/bitcoin/bitcoin/pull/27319)
MarcoFalke closed an issue: "get block or transaction data via rpc api when in the ibd process"
(https://github.com/bitcoin/bitcoin/issues/27388)
🚀 fanquake merged a pull request: "net processing: #26140 follow-ups"
(https://github.com/bitcoin/bitcoin/pull/27379)
💬 fanquake commented on pull request "test: Remove python3.5 workaround":
(https://github.com/bitcoin/bitcoin/pull/27378#issuecomment-1493310359)
ACK fae66fceb3147385320593d1e15faf290b0f4caf

I agree. Not seeing any issues with this on macOS, so lets merge, and revisit if any issues occur.
🚀 fanquake merged a pull request: "test: Remove python3.5 workaround"
(https://github.com/bitcoin/bitcoin/pull/27378)
🚀 fanquake merged a pull request: "depends, doc: Document `NO_USDT` option"
(https://github.com/bitcoin/bitcoin/pull/27387)
fanquake closed an issue: "miniscript: comparison of integers of different signs"
(https://github.com/bitcoin/bitcoin/issues/27381)
🚀 fanquake merged a pull request: "miniscript: explicit cast instead of comparing integers of different signs"
(https://github.com/bitcoin/bitcoin/pull/27382)
💬 furszy commented on pull request "wallet, gui: bugfix, getAvailableBalance skips selected coins":
(https://github.com/bitcoin/bitcoin/pull/26699#issuecomment-1493373583)
rebased, conflict solved.