Feed All Stories
Jun 8 2021
Jun 8 2021
Fabien committed rABCa1601abf3f0e: [CMAKE] Hint to user to the option flags when an optional dependency is not… (authored by Fabien).
[CMAKE] Hint to user to the option flags when an optional dependency is not…
deadalnix accepted D9645: [CMAKE] Hint to user to the option flags when an optional dependency is not found.
PiRK committed rABC1d26b7dbb65d: walletdb: Remove unused static functions from walletdb.h (authored by Andrew Chow <achow101-github@achow101.com>).
walletdb: Remove unused static functions from walletdb.h
PiRK committed rABCd37ccaecae8f: wallettool: Have RecoverDatabaseFile return errors and warnings (authored by Andrew Chow <achow101-github@achow101.com>).
wallettool: Have RecoverDatabaseFile return errors and warnings
PiRK committed rABC7e286043e41a: walletdb: Don't remove database transaction logs and instead error (authored by Andrew Chow <achow101-github@achow101.com>).
walletdb: Don't remove database transaction logs and instead error
PiRK committed rABC11d97206b17a: Call RecoverDatabaseFile directly from wallettool (authored by Andrew Chow <achow101-github@achow101.com>).
Call RecoverDatabaseFile directly from wallettool
Jun 7 2021
Jun 7 2021
Fabien requested review of D9644: [avalanche] Send avalanche proofs upon request.
Fabien updated the diff for D9643: [avalanche] Keep track of the peer creation time.
Use the right clock for the job
Mengerian added a comment to D9628: Add -uaclientname and -uaclientversion config options to set the client name and version of the user agent in the version message..
Why not have two flags that can be set, -uaclientname, and -uaclientversion ?
tobias_ruck updated the diff for D9628: Add -uaclientname and -uaclientversion config options to set the client name and version of the user agent in the version message..
Changed FormatSubVersion to FormatUserAgent and solely use FormatSubVersionUserAgent now
Fabien planned changes to D9643: [avalanche] Keep track of the peer creation time.
[refactor] deduplicate AmountFromValue
Fabien added inline comments to D9643: [avalanche] Keep track of the peer creation time.
Fabien requested review of D9642: [avalanche] Keep a rolling bloom filter of our recently announced proofs.
I don't know what caused the failure, I couldn't reproduce in a 100 loop on a similar debian, nor did it reproduce on CI.
Fabien added a comment to D9643: [avalanche] Keep track of the peer creation time.
Added the rational in the diff summary
Fabien updated the summary of D9643: [avalanche] Keep track of the peer creation time.
Fabien updated the summary of D9642: [avalanche] Keep a rolling bloom filter of our recently announced proofs.
deadalnix added a comment to D9643: [avalanche] Keep track of the peer creation time.
Why?
Fabien planned changes to D9642: [avalanche] Keep a rolling bloom filter of our recently announced proofs.
Investigating the build failure
teamcity added a comment to D9642: [avalanche] Keep a rolling bloom filter of our recently announced proofs.
Build Bitcoin ABC Diffs / Diff Testing (build-clang) failed.
Fabien requested review of D9643: [avalanche] Keep track of the peer creation time.
Fabien requested review of D9642: [avalanche] Keep a rolling bloom filter of our recently announced proofs.
teamcity added a comment to D9641: wallettool: Have RecoverDatabaseFile return errors and warnings.
Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) failed.
PiRK updated the diff for D9639: Call RecoverDatabaseFile directly from wallettool.
rebase (unrelated failure)
PiRK updated the diff for D9079: walletdb: Remove unused static functions from walletdb.h.
rebase to fix clang failure
PiRK updated the diff for D9641: wallettool: Have RecoverDatabaseFile return errors and warnings.
fix clang warning
teamcity added a comment to D9079: walletdb: Remove unused static functions from walletdb.h.
Build Bitcoin ABC Diffs / Diff Testing (build-clang) failed.
teamcity added a comment to D9641: wallettool: Have RecoverDatabaseFile return errors and warnings.
Build Bitcoin ABC Diffs / Diff Testing (build-clang) failed.
teamcity added a comment to D9639: Call RecoverDatabaseFile directly from wallettool.
Build Bitcoin ABC Diffs / Diff Testing (build-clang-tidy) failed.
PiRK updated the diff for D9079: walletdb: Remove unused static functions from walletdb.h.
PiRK updated the summary of D9641: wallettool: Have RecoverDatabaseFile return errors and warnings.
teamcity updated the summary of D9641: wallettool: Have RecoverDatabaseFile return errors and warnings.
PiRK requested review of D9641: wallettool: Have RecoverDatabaseFile return errors and warnings.
teamcity updated the summary of D9640: walletdb: Don't remove database transaction logs and instead error.
PiRK requested review of D9640: walletdb: Don't remove database transaction logs and instead error.
teamcity updated the summary of D9639: Call RecoverDatabaseFile directly from wallettool.
PiRK requested review of D9639: Call RecoverDatabaseFile directly from wallettool.
PiRK updated the summary of D9079: walletdb: Remove unused static functions from walletdb.h.
teamcity triaged T1642: Build Bitcoin-ABC / CashTab / CashTab Components Unit Tests is broken. as Unbreak Now! priority.
teamcity triaged T1641: Build Bitcoin-ABC / CashTab / CashTab Unit Tests is broken. as Unbreak Now! priority.
PiRK committed rABC4b6a468d3eb8: Split SetWalletFlags into Add/LoadWalletFlags (authored by Andrew Chow <achow101-github@achow101.com>).
Split SetWalletFlags into Add/LoadWalletFlags
PiRK committed rABCfee8cf69cb67: Split SetActiveScriptPubKeyMan into Add/LoadActiveScriptPubKeyMan (authored by Andrew Chow <achow101-github@achow101.com>).
Split SetActiveScriptPubKeyMan into Add/LoadActiveScriptPubKeyMan
PiRK committed rABC1147f7663bec: Split SetHDChain into AddHDChain and LoadHDChain (authored by Andrew Chow <achow101-github@achow101.com>).
Split SetHDChain into AddHDChain and LoadHDChain
Fabien requested changes to D9636: [testing] enable '-ecash' only on tests that use it.
Can you expand a bit of what the problem is from setting the flag globally ? Without more details it's hard to see how having the option repeated in and here in every test is better than setting it once for all.
Also if it increases the test coverage then the tests are wrong, because these tests are not testing the -ecash flag.
Jun 6 2021
Jun 6 2021
tobias_ruck retitled D9628: Add -uaclientname and -uaclientversion config options to set the client name and version of the user agent in the version message. from Add -useragent config option to explicitly set the User Agent in the version message. to Add -uaclient config option to set the client name and version of the user agent in the version message..
tobias_ruck updated the diff for D9628: Add -uaclientname and -uaclientversion config options to set the client name and version of the user agent in the version message..
Changed to -uaclient, which sets the client name and version of the user agent, staying compatible with -uacomment
majcosta updated the diff for D9638: [refactor] deduplicate AmountFromValue.
couldn't reproduce this failure. rebasing on latest master.
teamcity added a comment to D9638: [refactor] deduplicate AmountFromValue.
Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) failed.
majcosta abandoned D9635: [refactor] replace duplicated AmountFromValue function with Amount::Amount(UniValue T).
bummer. alright, how about D9638, then?
majcosta requested review of D9638: [refactor] deduplicate AmountFromValue.
majcosta updated the diff for D9636: [testing] enable '-ecash' only on tests that use it.
rebase on master
majcosta updated the summary of D9636: [testing] enable '-ecash' only on tests that use it.
Add FormatSubVersionUserAgent.
deadalnix added a comment to D9635: [refactor] replace duplicated AmountFromValue function with Amount::Amount(UniValue T).
This honestly seems like a step backward. The thing throws JSONRPC errors, so that most certainly belong in the RPC stack.
tobias_ruck requested review of D9637: Add FormatSubVersionUserAgent..
majcosta updated the diff for D9636: [testing] enable '-ecash' only on tests that use it.
rebase
majcosta updated the diff for D9636: [testing] enable '-ecash' only on tests that use it.
that flakiness in script_tests.cpp is unrelated, restarting build
teamcity added a comment to D9636: [testing] enable '-ecash' only on tests that use it.
Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) failed.
majcosta updated the diff for D9636: [testing] enable '-ecash' only on tests that use it.
fixed feature_notifications.py without wallet
teamcity added a comment to D9636: [testing] enable '-ecash' only on tests that use it.
Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) failed.
majcosta updated the summary of D9636: [testing] enable '-ecash' only on tests that use it.
majcosta updated the diff for D9636: [testing] enable '-ecash' only on tests that use it.
Depends on D9635
teamcity added a comment to D9636: [testing] enable '-ecash' only on tests that use it.
Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) failed.
majcosta requested review of D9636: [testing] enable '-ecash' only on tests that use it.
Jun 5 2021
Jun 5 2021
Fabien committed rABC13e9dcd14d5a: [avalanche] Fix threading issue in processor_tests.cpp (authored by Fabien).
[avalanche] Fix threading issue in processor_tests.cpp
Jun 4 2021
Jun 4 2021
Abandoning in favor of D9634
Fabien added a comment to D9633: Fix threading issue in processor_tests.cpp.
Please consider D9634 for an alternative solution
Fabien planned changes to D9365: [avalanche] Add an option to import a delegation.
Fabien planned changes to D9445: [avalanche] Check the delegation generated at startup is valid.
Fabien planned changes to D9471: [avalanche] Improve proof checking at startup.
Hopefully this won't be needed anymore