-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Bitcoin 0.12 cleanup PRs 2 #4268
Conversation
The `generate` RPC has no default `numblocks` and a numeric value is required.
* Introduce new constant MIN_CHANGE and use it instead of the hardcoded "CENT" * Add test case for MIN_CHANGE * Introduce new constant for -mintxfee default: DEFAULT_TRANSACTION_MINFEE = 1000
Zcash: Only includes non-QT parts
* Don't allow off-by-one or more * Make clear dust is coupled with minRelayTxFee * Check rounding for odd values
This PR only requires one review, as it's all general changes that don't affect consensus or risk user funds. |
@zkbot r+ |
📌 Commit 136e1c0 has been approved by |
⌛ Testing commit 136e1c0 with merge 8713d73daf5bc13f633986ef08d8920d4a64cbb9... |
💔 Test failed - pr-merge |
@zkbot retry |
⌛ Testing commit 136e1c0 with merge 2da77edbfed83c8add78e4cefe172d2e4fea066c... |
The retry will fail, because the test failed due to the "absurdly high fee" bound being altered (and changing the error message). When the retry fails, I'll push the fix and re-r+. |
💔 Test failed - pr-merge |
Fix pushed. @zkbot r+ |
📌 Commit f93db12 has been approved by |
Bitcoin 0.12 cleanup PRs 2 Cherry-picked from the following upstream PRs: - bitcoin/bitcoin#6631 - bitcoin/bitcoin#6664 - Only the first commit (we already had the second through bitcoin/bitcoin#6825). - bitcoin/bitcoin#6669 - bitcoin/bitcoin#6887 - Only the non-QT parts. - bitcoin/bitcoin#6962 - bitcoin/bitcoin#6822 - Only first and third commits (we already had the second through an earlier PR). - bitcoin/bitcoin#7136 - Excludes Travis CI changes, and fixes to documents we don't have anymore. - bitcoin/bitcoin#7084 - bitcoin/bitcoin#7509 - bitcoin/bitcoin#7617 - bitcoin/bitcoin#7726 Part of #2074.
Cherry-picked from the following upstream PRs:
Part of #2074.