Bitcoin Core Github
44 subscribers
120K links
Download Telegram
💬 maflcko commented on pull request "assumeutxo: Drop block height from metadata":
(https://github.com/bitcoin/bitcoin/pull/30598#discussion_r1712660234)
Done as part of https://github.com/bitcoin/bitcoin/pull/30624
👍 tdb3 approved a pull request: "cli: restrict multiple exclusive argument usage in bitcoin-cli"
(https://github.com/bitcoin/bitcoin/pull/30148#pullrequestreview-2231464882)
cr re ACK c8e6771af002eaf15567794fcdc57fdb0e3fb140
Diff showed minor changes to comment header of `CheckMultipleCLIArgs()`
💬 agroce commented on issue "fuzz: `script`: Assertion `!extract_destination_ret' failed.":
(https://github.com/bitcoin/bitcoin/issues/30615#issuecomment-2282202407)
This actually leads to a q I was wondering about for OSS-Fuzz: are there configs for appropriate targets with smaller max-lens?
💬 fjahr commented on pull request "doc: Remove outdated nTx faking comment":
(https://github.com/bitcoin/bitcoin/pull/30624#issuecomment-2282212130)
ACK fa04511e44bc6c740ff1c4f912f689b2bc3d1288
🤔 stratospher reviewed a pull request: "addrman: change internal id counting to int64_t"
(https://github.com/bitcoin/bitcoin/pull/30568#pullrequestreview-2231495237)
Concept ACK. there's also if you retouch. https://github.com/bitcoin/bitcoin/blob/c2d15d993ef06d97d4c117012bda6efa3dcbac45/src/test/fuzz/addrman.cpp#L189
💬 gmaxwell commented on pull request "validation: write chainstate to disk every hour":
(https://github.com/bitcoin/bitcoin/pull/30611#issuecomment-2282238485)
I didn't comment on IBD because I figured a hit per hour isn't that significant in the context of IBD and likely pays for itself in improving crash robustness. I'm a little surprised that the effect is quite so small, but it does probably improve performance too by making writes a little more concurrent.

Sipa: good point on the injection locking, I've previously wondered if the the 24 hour behavior could sync up.
💬 stratospher commented on pull request "test: Add test for rpcwhitelistdefault":
(https://github.com/bitcoin/bitcoin/pull/29858#discussion_r1712687061)
3123eea: I wonder if [`replace_in_config`](https://github.com/bitcoin/bitcoin/blob/c2d15d993ef06d97d4c117012bda6efa3dcbac45/test/functional/test_framework/test_node.py#L441) can be used here?
💬 gmaxwell commented on pull request "Halt processing of unrequested transactions v2":
(https://github.com/bitcoin/bitcoin/pull/30572#issuecomment-2282241950)
@mzumsande Good point-- though the logic in the comment ought not apply when the transaction in the orphan pool comes from the same peer, in that case the extra request would be purely redundant.
📝 fjahr opened a pull request: "chainparams: Handle Testnet4 in GetNetworkForMagic"
(https://github.com/bitcoin/bitcoin/pull/30625)
Found during testing: The recently introduced `GetNetworkForMagic()` doesn't handle Testnet4 yet.
💬 fjahr commented on pull request "validation: assumeutxo params mainnet":
(https://github.com/bitcoin/bitcoin/pull/28553#issuecomment-2282275885)
re-ACK 1610643c8b37a9f674b236cfa79abf8f8aaf1410

Verified only rebase via `git range-diff master fb543c9f733610b8fadbfd6f1c71f1bb520de985 1610643c8b37a9f674b236cfa79abf8f8aaf1410` and re-tested loading and syncing with the new snapshot torrent.
💬 hebasto commented on pull request "build: Introduce CMake-based build system":
(https://github.com/bitcoin/bitcoin/pull/30454#issuecomment-2282278895)
@fanquake in https://github.com/bitcoin/bitcoin/pull/30454#issuecomment-2277722262:
> Had a look at a Guix build. Stripping the macOS binaries is broken:
>
> ```shell
> -- Installing: /distsrc-base/distsrc-ad2140d4d8cc-arm64-apple-darwin/installed/bitcoin-ad2140d4d8cc/bin/bitcoind
> /root/.guix-profile/bin/llvm-strip: error: unknown argument '-u'
> ```

Fixed in https://github.com/hebasto/bitcoin/pull/321.
⚠️ Vick145 opened an issue: "https://t.me/OfficialBabyOtter"
(https://github.com/bitcoin/bitcoin/issues/30626)
### Motivation

Welcome John to the BabyOtter World

We're a wild gang of degen enthusiasts on a mission to rocket BabyOTTer to at least 300x! Count on us!! Let's blast off!!!

''OTTER'' the biggest hype SOLANA

$BabyOtter CA : EUYJtrgfuqnsqX27EDPSbS3LtoXV9vfGagF8ssUWsX5w

| NO TEAM TOKEN |
| 0% FEE |
|CONTRACT SAFU |
|LP BURNED & CONTRACT RENOUNCED |

‼️Read the pinned messages!
‼️ADM NEVER DM FIRST!

### Possible solution

Welcome John to the BabyOtter World
...
:lock: hebasto locked an issue: "."
(https://github.com/bitcoin/bitcoin/issues/30626)
👍 tdb3 approved a pull request: "chainparams: Handle Testnet4 in GetNetworkForMagic"
(https://github.com/bitcoin/bitcoin/pull/30625#pullrequestreview-2231690134)
cr ACK b0ec8716bf27335686471e0ae4c6a34f9a08f33c
💬 theStack commented on pull request "contrib: add tool to convert compact-serialized UTXO set to SQLite database":
(https://github.com/bitcoin/bitcoin/pull/27432#issuecomment-2282299449)
Rebased on master, with the following changes:
* adapted to the new snapshot format (VERSION=2, see #30598)
* extended the netmagic table with Testnet4 (see #29775)
* extended the test to also create a pay-to-anchor UTXO (see #30352)
* print out the supposed snapshot height after processing all coins, by tracking the maximum height (see https://github.com/bitcoin/bitcoin/pull/30598#issuecomment-2276350922)
💬 allanperlee commented on issue "use MiniWallet in functional test `rpc_signrawtransactionwithkey.py` for funding txs":
(https://github.com/bitcoin/bitcoin/issues/30600#issuecomment-2282304362)
Hi @theStack, I could soon start this issue
⚠️ sahil5d opened an issue: "Evaluate Unappreciated Evidence that Satoshi Nakamoto was Elon Musk"
(https://github.com/bitcoin/bitcoin/issues/30627)
### Please describe the feature you'd like to see added.

Add to the knowledge of Satoshi Nakamoto the following information:

## Unappreciated Evidence that Satoshi Nakamoto was Elon Musk

### An IP address in Los Angeles

On Satoshi: "This is not a TOR exit node which implies that this is the IP address used by Satoshi on 2009–01–10 and he was in Van Nuys [Los Angeles] on this day."
 https://whoissatoshi.wordpress.com/2016/02/20/satoshi-in-california/

On Musk: "The jet is registered
...
achow101 closed an issue: "Evaluate Unappreciated Evidence that Satoshi Nakamoto was Elon Musk"
(https://github.com/bitcoin/bitcoin/issues/30627)
:lock: achow101 locked an issue: "Evaluate Unappreciated Evidence that Satoshi Nakamoto was Elon Musk"
(https://github.com/bitcoin/bitcoin/issues/30627)