Page MenuHomePhabricator
Feed All Stories

May 30 2020

teamcity added a comment to D6265: Add land bot dependencies to base image setup script.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) passed.

May 30 2020, 00:48
teamcity added a comment to D6265: Add land bot dependencies to base image setup script.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) passed.

May 30 2020, 00:48
majcosta updated the diff for D6296: [backport#15932] rpc: Use IsValidNumArgs in getblock.

updated test plan and reformatted and relayouted the cli command's help output

May 30 2020, 00:46
jasonbcox committed rABC7df5863a2333: Merge #15345: net: Correct comparison of addr count (authored by Wladimir J. van der Laan <laanwj@gmail.com>).
Merge #15345: net: Correct comparison of addr count
May 30 2020, 00:43
jasonbcox closed D6301: Merge #15345: net: Correct comparison of addr count.
May 30 2020, 00:43
jasonbcox retitled D6265: Add land bot dependencies to base image setup script from Add base image setup script for the land bot to Add land bot dependencies to base image setup script.
May 30 2020, 00:41
teamcity added a comment to D6265: Add land bot dependencies to base image setup script.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) started.

May 30 2020, 00:39
teamcity added a comment to D6265: Add land bot dependencies to base image setup script.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) started.

May 30 2020, 00:39
teamcity added a comment to D6265: Add land bot dependencies to base image setup script.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) started.

May 30 2020, 00:39
jasonbcox updated the diff for D6265: Add land bot dependencies to base image setup script.

Update same base image used for CI builds rather than introducing a new one.

May 30 2020, 00:39
majcosta updated the test plan for D6296: [backport#15932] rpc: Use IsValidNumArgs in getblock.
May 30 2020, 00:31
teamcity added a comment to D5956: Add -datadir option to allow specification of seeder's data directory.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) passed.

May 30 2020, 00:22
teamcity added a comment to D5956: Add -datadir option to allow specification of seeder's data directory.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) passed.

May 30 2020, 00:15
teamcity added a comment to D6303: Wrap nChainTx into GetChainTxCount.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) passed.

May 30 2020, 00:14
teamcity added a comment to D5956: Add -datadir option to allow specification of seeder's data directory.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) passed.

May 30 2020, 00:11
teamcity added a comment to D5956: Add -datadir option to allow specification of seeder's data directory.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) started.

May 30 2020, 00:10
teamcity added a comment to D5956: Add -datadir option to allow specification of seeder's data directory.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) started.

May 30 2020, 00:10
teamcity added a comment to D5956: Add -datadir option to allow specification of seeder's data directory.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) started.

May 30 2020, 00:10
nakihito added inline comments to D5956: Add -datadir option to allow specification of seeder's data directory.
May 30 2020, 00:10
nakihito updated the diff for D5956: Add -datadir option to allow specification of seeder's data directory.

Added some output that specifies the default directory and the directory in use on start up.

May 30 2020, 00:10
teamcity added a comment to D6303: Wrap nChainTx into GetChainTxCount.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) passed.

May 30 2020, 00:07
teamcity added a comment to D6303: Wrap nChainTx into GetChainTxCount.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) passed.

May 30 2020, 00:05
teamcity added a comment to D6303: Wrap nChainTx into GetChainTxCount.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) started.

May 30 2020, 00:03
teamcity added a comment to D6303: Wrap nChainTx into GetChainTxCount.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) started.

May 30 2020, 00:03
teamcity added a comment to D6303: Wrap nChainTx into GetChainTxCount.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) started.

May 30 2020, 00:03
deadalnix updated the diff for D6303: Wrap nChainTx into GetChainTxCount.

Also update getchaintxstats

May 30 2020, 00:02

May 29 2020

majcosta added inline comments to D6303: Wrap nChainTx into GetChainTxCount.
May 29 2020, 23:51
nakihito updated the summary of D6302: Merge #12330: Reduce scope of cs_wallet locks in listtransactions.
May 29 2020, 23:29
nakihito requested review of D6302: Merge #12330: Reduce scope of cs_wallet locks in listtransactions.
May 29 2020, 23:28
teamcity added a comment to D6303: Wrap nChainTx into GetChainTxCount.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) passed.

May 29 2020, 23:16
teamcity added a comment to D6289: Generate assumed blockchain and chainstate disk sizes when updating chainparams.

Build Bitcoin ABC Diffs / Diff Testing (build-ibd-no-assumevalid-checkpoint) passed.

May 29 2020, 23:13
teamcity added a comment to D6303: Wrap nChainTx into GetChainTxCount.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) passed.

May 29 2020, 23:06
teamcity added a comment to D6303: Wrap nChainTx into GetChainTxCount.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) started.

May 29 2020, 23:04
teamcity added a comment to D6303: Wrap nChainTx into GetChainTxCount.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) passed.

May 29 2020, 23:04
teamcity added a comment to D6303: Wrap nChainTx into GetChainTxCount.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) started.

May 29 2020, 23:02
teamcity added a comment to D6303: Wrap nChainTx into GetChainTxCount.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) started.

May 29 2020, 23:01
deadalnix requested review of D6303: Wrap nChainTx into GetChainTxCount.
May 29 2020, 23:00
teamcity added a comment to D6302: Merge #12330: Reduce scope of cs_wallet locks in listtransactions.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) passed.

May 29 2020, 22:53
deadalnix accepted D6301: Merge #15345: net: Correct comparison of addr count.
May 29 2020, 22:51
deadalnix committed rABC89778bcdb086: Remove CBlockIndex::SetNull (authored by deadalnix).
Remove CBlockIndex::SetNull
May 29 2020, 22:48
deadalnix closed D6290: Remove CBlockIndex::SetNull.
May 29 2020, 22:48
teamcity added a comment to D6302: Merge #12330: Reduce scope of cs_wallet locks in listtransactions.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) passed.

May 29 2020, 22:43
teamcity added a comment to D6301: Merge #15345: net: Correct comparison of addr count.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) passed.

May 29 2020, 22:43
teamcity added a comment to D6302: Merge #12330: Reduce scope of cs_wallet locks in listtransactions.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) started.

May 29 2020, 22:42
teamcity added a comment to D6302: Merge #12330: Reduce scope of cs_wallet locks in listtransactions.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) passed.

May 29 2020, 22:42
nakihito retitled D6302: Merge #12330: Reduce scope of cs_wallet locks in listtransactions from Merge #12330: Reduce scope of cs_main and cs_wallet locks in listtransactions to Merge #12330: Reduce scope of cs_wallet locks in listtransactions.
May 29 2020, 22:42
teamcity added a comment to D6302: Merge #12330: Reduce scope of cs_wallet locks in listtransactions.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) started.

May 29 2020, 22:40
teamcity added a comment to D6302: Merge #12330: Reduce scope of cs_wallet locks in listtransactions.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) started.

May 29 2020, 22:40
teamcity added a comment to D6301: Merge #15345: net: Correct comparison of addr count.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) passed.

May 29 2020, 22:40
nakihito planned changes to D6302: Merge #12330: Reduce scope of cs_wallet locks in listtransactions.
May 29 2020, 22:39
teamcity updated the summary of D6302: Merge #12330: Reduce scope of cs_wallet locks in listtransactions.
May 29 2020, 22:38
teamcity added a comment to D6302: Merge #12330: Reduce scope of cs_wallet locks in listtransactions.

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

May 29 2020, 22:38
nakihito requested review of D6302: Merge #12330: Reduce scope of cs_wallet locks in listtransactions.
May 29 2020, 22:38
teamcity added a comment to D6301: Merge #15345: net: Correct comparison of addr count.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) started.

May 29 2020, 22:34
teamcity added a comment to D6301: Merge #15345: net: Correct comparison of addr count.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) passed.

May 29 2020, 22:34
teamcity added a comment to D6301: Merge #15345: net: Correct comparison of addr count.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) started.

May 29 2020, 22:33
teamcity added a comment to D6301: Merge #15345: net: Correct comparison of addr count.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) started.

May 29 2020, 22:32
teamcity added a comment to D6301: Merge #15345: net: Correct comparison of addr count.

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

May 29 2020, 22:31
teamcity updated the summary of D6301: Merge #15345: net: Correct comparison of addr count.
May 29 2020, 22:31
jasonbcox requested review of D6301: Merge #15345: net: Correct comparison of addr count.
May 29 2020, 22:31
majcosta accepted D6290: Remove CBlockIndex::SetNull.
May 29 2020, 22:29
deadalnix committed rABCec2417bd70d6: [wallet] abort when attempting to fund a transaction above maxtxfee (authored by Sjors Provoost <sjors@sprovoost.nl>).
[wallet] abort when attempting to fund a transaction above maxtxfee
May 29 2020, 22:18
deadalnix closed D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.
May 29 2020, 22:18
teamcity added a comment to D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) passed.

May 29 2020, 22:00
teamcity added a comment to D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) passed.

May 29 2020, 21:54
jasonbcox added a comment to D6291: [SECP256K1] Fix -Wmaybe-uninitialized in tests.

I would rather avoid taking ownership of this. The part of libsecp256k1 we have ownership over are very well delimited and separated from the rest of the codebase. IMO this is a feature.

I would suggest you go about this by either:

  • Disable this warning for libsecp256k1 and report the problem upstream.
  • Submit a patch upstream.
May 29 2020, 21:53
jasonbcox planned changes to D6265: Add land bot dependencies to base image setup script.
May 29 2020, 21:52
jasonbcox accepted D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.
May 29 2020, 21:51
teamcity added a comment to D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) passed.

May 29 2020, 21:51
jasonbcox added inline comments to D6290: Remove CBlockIndex::SetNull.
May 29 2020, 21:51
teamcity added a comment to D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) started.

May 29 2020, 21:49
teamcity added a comment to D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) started.

May 29 2020, 21:49
teamcity added a comment to D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) started.

May 29 2020, 21:49
teamcity added a comment to D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) passed.

May 29 2020, 21:49
deadalnix updated the diff for D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.

BTC -> BCh

May 29 2020, 21:49
teamcity added a comment to D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) passed.

May 29 2020, 21:41
teamcity added a comment to D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.

Build Bitcoin ABC Diffs / Diff Testing (build-diff) started.

May 29 2020, 21:38
teamcity added a comment to D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) started.

May 29 2020, 21:35
teamcity added a comment to D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) passed.

May 29 2020, 21:35
jasonbcox requested changes to D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.
May 29 2020, 21:33
teamcity added a comment to D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.

Build Bitcoin ABC Diffs / Diff Testing (build-werror) started.

May 29 2020, 21:32
deadalnix requested changes to D6295: [backport#15139] util: Make ToLower and ToUpper take a char.
May 29 2020, 21:32
deadalnix updated the diff for D6268: [wallet] abort when attempting to fund a transaction above maxtxfee.

Update release notes

May 29 2020, 21:31
deadalnix added inline comments to D6290: Remove CBlockIndex::SetNull.
May 29 2020, 21:28
deadalnix requested changes to D6291: [SECP256K1] Fix -Wmaybe-uninitialized in tests.

I would rather avoid taking ownership of this. The part of libsecp256k1 we have ownership over are very well delimited and separated from the rest of the codebase. IMO this is a feature.

May 29 2020, 21:25
deadalnix committed rABCe19a6b1c5386: Pass Consensus::Params to load block index family of functions (authored by deadalnix).
Pass Consensus::Params to load block index family of functions
May 29 2020, 21:23
deadalnix closed D6293: Pass Consensus::Params to load block index family of functions.
May 29 2020, 21:23
deadalnix requested changes to D6296: [backport#15932] rpc: Use IsValidNumArgs in getblock.
May 29 2020, 21:17
deadalnix requested changes to D6294: [backport#14599] Use functions guaranteed to be locale independent.
May 29 2020, 21:16
jasonbcox added a comment to D6265: Add land bot dependencies to base image setup script.

Why would we want the land bot to use different dependencies that CI builds?

May 29 2020, 20:58
Fabien requested changes to D6264: Add arcanist land bot workflow.
May 29 2020, 20:50
nakihito committed rABC5cf3c08e993d: Fix WSL file locking by using flock instead of fcntl (authored by Samuel Dobson <dobsonsa68@gmail.com>).
Fix WSL file locking by using flock instead of fcntl
May 29 2020, 20:39
nakihito closed D6299: Fix WSL file locking by using flock instead of fcntl.
May 29 2020, 20:39
jasonbcox requested changes to D6294: [backport#14599] Use functions guaranteed to be locale independent.

Are the missed changes in LocaleDependenceLinter due to missing backports? If so, why not do those first?

May 29 2020, 20:26
jasonbcox committed rABCd5188dc324c4: Bump version to 0.21.9 (authored by jasonbcox).
Bump version to 0.21.9
May 29 2020, 20:21
jasonbcox closed D6298: Bump version to 0.21.9.
May 29 2020, 20:20
jasonbcox added a comment to D6298: Bump version to 0.21.9.

Should probably start including checking the seeder version?

May 29 2020, 20:19
jasonbcox accepted D6299: Fix WSL file locking by using flock instead of fcntl.
May 29 2020, 20:19
majcosta updated the test plan for D6297: [backport#15932] rpc: Serialize in getblock without cs_main.
May 29 2020, 20:14
majcosta requested review of D6296: [backport#15932] rpc: Use IsValidNumArgs in getblock.

expanded test plan

May 29 2020, 20:13