-
Notifications
You must be signed in to change notification settings - Fork 35.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[26.x] Backports #29011
[26.x] Backports #29011
Conversation
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. Code CoverageFor detailed information about the code coverage, see the test coverage report. ReviewsSee the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update. |
Github-Pull: bitcoin#29023 Rebased-From: 7d4e47d
Github-Pull: bitcoin#29023 Rebased-From: ca59375
a17e975
to
188e023
Compare
ACK 188e023 Checked diffs between Rebased-from commit hash and these commits themselves. |
188e023
to
de8f082
Compare
Note that any tidy job failure here is currently related to #28992 (comment) (and should be fixed in future). |
de8f082
to
2cf9b1b
Compare
987d3d6
to
aae721f
Compare
7e90b59 has a typo in the commit message |
aae721f
to
cace1c6
Compare
cace1c6
to
48772b0
Compare
Github-Pull: bitcoin#28992 Rebased-From: fa83b65
Github-Pull: bitcoin#28992 Rebased-From: fad2392
The crash would happen when querying a mempool transaction with verbosity=2, while pruning. Github-Pull: bitcoin#29003 Rebased-From: 494a926
This fails on master without the previous commit. Github-Pull: bitcoin#29003 Rebased-From: 9075a44
Co-authored-by: furszy <matiasfurszyfer@protonmail.com> Github-Pull: bitcoin#28994 Rebased-From: 5cea25b
Useful for understanding what is going on internally when the software is running. Debug issues, and provide more accurate feedback to users. Github-Pull: bitcoin#28994 Rebased-From: 0c57557
Verify the transaction creation process does not produce a BnB solution when SFFO is enabled. This is currently problematic because it could require a change output. And BnB is specialized on changeless solutions. Co-authored-by: Andrew Chow <achow101@gmail.com> Co-authored-by: Murch <murch@murch.one> Github-Pull: bitcoin#28994 Rebased-From: 05e5ff1
Github-Pull: bitcoin#28994 Rebased-From: 576bee8
In the following-up commit, the wallet birth time will also be modified by the transactions scanning process. When a tx older than all descriptor's timestamp is detected. Github-Pull: bitcoin#28920 Rebased-From: b4306e3
As the user could have imported a descriptor with a newer timestamp (by blindly setting 'timestamp=now'), the wallet needs to update the birth time when it detects a transaction older than the oldest descriptor timestamp. Github-Pull: bitcoin#28920 Rebased-From: 75fbf44
To avoid scanning blocks, as assumed by a wallet with no generated keys or imported scripts, the default value for the birth time needs to be set to the maximum int64_t value. Once the first key is generated or the first script is imported, the legacy SPKM will update the birth time automatically. Github-Pull: bitcoin#28920 Rebased-From: 6f49737
Verifying the wallet updates the birth time accordingly when it detects a transaction with a time older than the oldest descriptor timestamp. This could happen when the user blindly imports a descriptor with 'timestamp=now'. Github-Pull: bitcoin#28920 Rebased-From: 83c6644
And add coverage for it Github-Pull: bitcoin#28920 Rebased-From: 1ce45ba
Homebrew attempts to check for outdated dependents or those with broken linkage. Such behavior might lead to failures when Homebrew updates them on old macOS images. This change prevents such behavior. Github-Pull: bitcoin#29080 Rebased-From: 43c3246
Github-Pull: bitcoin#29176 Rebased-From: faebf1d
48772b0
to
7b79e54
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 7b79e54
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 7b79e54, matches mine
I'm planning to merge this now and open another batch of backports |
Backports for
26.x
. Currently:HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK
to avoid unrelated failures #29080