π¬ fanquake commented on issue "valgrind: Conditional jump or move depends on uninitialised value(s)":
(https://github.com/bitcoin/bitcoin/issues/29635#issuecomment-2471216093)
> Also, the suppression won't work for libc++
We aren't using libc++ though?
(https://github.com/bitcoin/bitcoin/issues/29635#issuecomment-2471216093)
> Also, the suppression won't work for libc++
We aren't using libc++ though?
π¬ jb55 commented on issue "Fatal LevelDB error: Corruption: block checksum mismatch on Linux ext4 SATA SSDs":
(https://github.com/bitcoin/bitcoin/issues/30692#issuecomment-2471226445)
fwiw I ran a full memtest for 4 hours and had no memory issues, so at this point I have no idea. it seems to be running stable for now.
(https://github.com/bitcoin/bitcoin/issues/30692#issuecomment-2471226445)
fwiw I ran a full memtest for 4 hours and had no memory issues, so at this point I have no idea. it seems to be running stable for now.
π¬ jb55 commented on issue "Fatal LevelDB error: Corruption: block checksum mismatch on Linux ext4 SATA SSDs":
(https://github.com/bitcoin/bitcoin/issues/30692#issuecomment-2471227779)
@laanwj suggested to post the utxoset when it gets corrupted again, will do that when it does.
(https://github.com/bitcoin/bitcoin/issues/30692#issuecomment-2471227779)
@laanwj suggested to post the utxoset when it gets corrupted again, will do that when it does.
π theStack approved a pull request: "Ephemeral Dust"
(https://github.com/bitcoin/bitcoin/pull/30239#pullrequestreview-2430361343)
re-ACK 5c2e291060cca3be500f3af0f6f2d3fd2177a7c9
(https://github.com/bitcoin/bitcoin/pull/30239#pullrequestreview-2430361343)
re-ACK 5c2e291060cca3be500f3af0f6f2d3fd2177a7c9
π¬ instagibbs commented on pull request "Ephemeral Dust":
(https://github.com/bitcoin/bitcoin/pull/30239#discussion_r1838568129)
mmm, seems like a test of generateblock to me? going to leave as is
(https://github.com/bitcoin/bitcoin/pull/30239#discussion_r1838568129)
mmm, seems like a test of generateblock to me? going to leave as is
π instagibbs opened a pull request: "policy: ephemeral dust followups"
(https://github.com/bitcoin/bitcoin/pull/31279)
Here are the parent PR's comments that should be addressed by this PR:
https://github.com/bitcoin/bitcoin/pull/30239/files#r1834529646
https://github.com/bitcoin/bitcoin/pull/30239/files#r1831247308
https://github.com/bitcoin/bitcoin/pull/30239/files#r1832622481
https://github.com/bitcoin/bitcoin/pull/30239/files#r1831195216
https://github.com/bitcoin/bitcoin/pull/30239#discussion_r1835805164
https://github.com/bitcoin/bitcoin/pull/30239#discussion_r1835805164
https://github.com/bitcoin
...
(https://github.com/bitcoin/bitcoin/pull/31279)
Here are the parent PR's comments that should be addressed by this PR:
https://github.com/bitcoin/bitcoin/pull/30239/files#r1834529646
https://github.com/bitcoin/bitcoin/pull/30239/files#r1831247308
https://github.com/bitcoin/bitcoin/pull/30239/files#r1832622481
https://github.com/bitcoin/bitcoin/pull/30239/files#r1831195216
https://github.com/bitcoin/bitcoin/pull/30239#discussion_r1835805164
https://github.com/bitcoin/bitcoin/pull/30239#discussion_r1835805164
https://github.com/bitcoin
...
π¬ ariard commented on pull request "Halt processing of unrequested transactions v2":
(https://github.com/bitcoin/bitcoin/pull/30572#issuecomment-2471402156)
> I was seeing were mostly non-segwit transactions - not unsolicited transactions. The problematic commit seems to be https://github.com/bitcoin/bitcoin/commit/c7bc444d02d34273a0a42806745e6a32b3146143.
@0xB10C Thanks, thatβs interesting. Iβll dig into the `TxRequester`, if due to switching from iterating by status to iterate by peers, if the processing for non-segwit transactions and segwit transactions is always the same.
βββββ
>Between 2024-10-29T22:50:51 and 2024-10-29T23:54 the nod
...
(https://github.com/bitcoin/bitcoin/pull/30572#issuecomment-2471402156)
> I was seeing were mostly non-segwit transactions - not unsolicited transactions. The problematic commit seems to be https://github.com/bitcoin/bitcoin/commit/c7bc444d02d34273a0a42806745e6a32b3146143.
@0xB10C Thanks, thatβs interesting. Iβll dig into the `TxRequester`, if due to switching from iterating by status to iterate by peers, if the processing for non-segwit transactions and segwit transactions is always the same.
βββββ
>Between 2024-10-29T22:50:51 and 2024-10-29T23:54 the nod
...
β οΈ southunitedraza opened an issue: "https://fb.gg/play/bravoslotsgame/d/1090680746136943/"
(https://github.com/bitcoin/bitcoin/issues/31280)
https://fb.gg/play/bravoslotsgame/d/1090680746136943/
(https://github.com/bitcoin/bitcoin/issues/31280)
https://fb.gg/play/bravoslotsgame/d/1090680746136943/
β
fanquake closed an issue: "https://fb.gg/play/bravoslotsgame/d/1090680746136943/"
(https://github.com/bitcoin/bitcoin/issues/31280)
(https://github.com/bitcoin/bitcoin/issues/31280)
:lock: fanquake locked an issue: "https://fb.gg/play/bravoslotsgame/d/1090680746136943/"
(https://github.com/bitcoin/bitcoin/issues/31280)
(https://github.com/bitcoin/bitcoin/issues/31280)
π€ fjahr reviewed a pull request: "wallet, rpc: Settxfeerate"
(https://github.com/bitcoin/bitcoin/pull/31278#pullrequestreview-2430637810)
Did you try to follow the approach suggested here: https://github.com/bitcoin/bitcoin/issues/31088#issuecomment-2413394142? If not, can you say why it didn't work?
(https://github.com/bitcoin/bitcoin/pull/31278#pullrequestreview-2430637810)
Did you try to follow the approach suggested here: https://github.com/bitcoin/bitcoin/issues/31088#issuecomment-2413394142? If not, can you say why it didn't work?
π¬ fjahr commented on pull request "wallet, rpc: Settxfeerate":
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838736610)
Examples copy+pasted but not updated
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838736610)
Examples copy+pasted but not updated
π¬ fjahr commented on pull request "wallet, rpc: Settxfeerate":
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838745844)
Why "will be deprecated"? Why not actually deprecate it?
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838745844)
Why "will be deprecated"? Why not actually deprecate it?
π¬ fjahr commented on pull request "wallet, rpc: Settxfeerate":
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838750890)
I don't think that's the right description
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838750890)
I don't think that's the right description
π¬ fjahr commented on pull request "wallet, rpc: Settxfeerate":
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838736093)
Why 2014-2022?
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838736093)
Why 2014-2022?
π¬ fjahr commented on pull request "wallet, rpc: Settxfeerate":
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838749101)
Do we set deprecated rpc methods to hidden? I'm not sure I have seen that before.
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838749101)
Do we set deprecated rpc methods to hidden? I'm not sure I have seen that before.
π¬ ismaelsadeeq commented on pull request "Policy: Report reason inputs are non standard":
(https://github.com/bitcoin/bitcoin/pull/29060#issuecomment-2471554421)
> The release note helps a lot. To err on the side of caution, it seems appropriate to include a `-deprecatedrpc=` option, to enable a period of deprecation for users.
Thanks for your review @tdb3 will address this comment soon.
(https://github.com/bitcoin/bitcoin/pull/29060#issuecomment-2471554421)
> The release note helps a lot. To err on the side of caution, it seems appropriate to include a `-deprecatedrpc=` option, to enable a period of deprecation for users.
Thanks for your review @tdb3 will address this comment soon.
π¬ polespinasa commented on pull request "wallet, rpc: Settxfeerate":
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838753630)
thanks for the observation, will correct it
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838753630)
thanks for the observation, will correct it
π¬ polespinasa commented on pull request "wallet, rpc: Settxfeerate":
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838754267)
have to update, used a template from an old test
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838754267)
have to update, used a template from an old test
π¬ polespinasa commented on pull request "wallet, rpc: Settxfeerate":
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838754885)
Avoid incompatibility problems with programs actually using this RPC call.
(https://github.com/bitcoin/bitcoin/pull/31278#discussion_r1838754885)
Avoid incompatibility problems with programs actually using this RPC call.