Page MenuHomePhabricator
Feed All Stories

Sep 3 2019

teamcity added a comment to D3987: wallet: Catch filesystem_error and raise InitError.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Sep 3 2019, 18:26
fpelliccioni created D3987: wallet: Catch filesystem_error and raise InitError.
Sep 3 2019, 18:25
teamcity added a comment to D3986: During IBD, when doing pruning, prune 10% extra to avoid pruning again soon after.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Sep 3 2019, 18:19
fpelliccioni created D3986: During IBD, when doing pruning, prune 10% extra to avoid pruning again soon after.
Sep 3 2019, 18:19
Nico Guiton <nico@bitframe.org> committed rABC3bdbe4821611: Merge #13159: Don't close old debug log file handle prematurely when trying to… (authored by Wladimir J. van der Laan <laanwj@gmail.com>).
Merge #13159: Don't close old debug log file handle prematurely when trying to…
Sep 3 2019, 17:49
Nico Guiton <nico@bitframe.org> closed D3979: Merge #13159: Don't close old debug log file handle prematurely when trying to re-open (on SIGHUP).
Sep 3 2019, 17:49
Nico Guiton <nico@bitframe.org> committed rABC1d91c62d1401: Merge #13148: logging: Fix potential use-after-free in LogPrintStr(...) (authored by Wladimir J. van der Laan <laanwj@gmail.com>).
Merge #13148: logging: Fix potential use-after-free in LogPrintStr(...)
Sep 3 2019, 17:34
Nico Guiton <nico@bitframe.org> closed D3976: Merge #13148: logging: Fix potential use-after-free in LogPrintStr(...).
Sep 3 2019, 17:34
teamcity added a comment to D3985: Add missing override keyword to DummySignatureCreator::CreateSig().

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Sep 3 2019, 13:58
deadalnix accepted D3985: Add missing override keyword to DummySignatureCreator::CreateSig().
Sep 3 2019, 13:57
teamcity added a comment to D3985: Add missing override keyword to DummySignatureCreator::CreateSig().

Build Bitcoin-ABC / Diffs / Diff Testing started.

Sep 3 2019, 13:48
Fabien created D3985: Add missing override keyword to DummySignatureCreator::CreateSig().
Sep 3 2019, 13:48
fpelliccioni abandoned D3983: Add linter: Enforce the source code file naming convention described in the developer notes.
Sep 3 2019, 12:58
teamcity added a comment to D3984: [LINTER] Enforce using C++ style for void parameters.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Sep 3 2019, 10:39
teamcity added a comment to D3984: [LINTER] Enforce using C++ style for void parameters.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Sep 3 2019, 10:29
Fabien created D3984: [LINTER] Enforce using C++ style for void parameters.
Sep 3 2019, 10:28
Fabien added a comment to D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.

See D3980

Sep 3 2019, 08:25
Fabien requested changes to D3981: Merge #13081: wallet: Add compile time checking for cs_wallet runtime locking assertions.

See D3980

Sep 3 2019, 08:24
Fabien requested changes to D3980: Merge #13077: Add compile time checking for all cs_KeyStore runtime locking assertions.

The test plan is incorrect. --enable-debug is aimed to detect races at runtime from the LOCK(), AssertLockHeld(), ... macros.
EXCLUSIVE_LOCKS_REQUIRED() and alike are compile-time directives used by Clang (only) when the -Wthread-safety-analysis compile flag is set.

Sep 3 2019, 08:17
Fabien accepted D3979: Merge #13159: Don't close old debug log file handle prematurely when trying to re-open (on SIGHUP).
Sep 3 2019, 07:58
Fabien requested changes to D3983: Add linter: Enforce the source code file naming convention described in the developer notes.

I'm afraid this is a duplicate of D3593.
My bad, I missed there was a backport.

Sep 3 2019, 07:46
teamcity added a comment to D3983: Add linter: Enforce the source code file naming convention described in the developer notes.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Sep 3 2019, 02:56
teamcity added a comment to D3983: Add linter: Enforce the source code file naming convention described in the developer notes.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Sep 3 2019, 02:45
fpelliccioni created D3983: Add linter: Enforce the source code file naming convention described in the developer notes.
Sep 3 2019, 02:45

Sep 2 2019

fpelliccioni committed rABC1bf9852e2c50: Default to defining endian-conversion DECLs in compat w/o config (authored by Matt Corallo <git@bluematt.me>).
Default to defining endian-conversion DECLs in compat w/o config
Sep 2 2019, 17:46
fpelliccioni closed D3966: Default to defining endian-conversion DECLs in compat w/o config.
Sep 2 2019, 17:46
fpelliccioni committed rABC51fbdf464a64: Consistently log CValidationState on failure (authored by Ben Woosley <ben.woosley@gmail.com>).
Consistently log CValidationState on failure
Sep 2 2019, 17:37
fpelliccioni closed D3967: Consistently log CValidationState on failure.
Sep 2 2019, 17:36
Fabien committed rABC9904779739dd: Remove deprecated features for the 0.20.x branch (authored by Fabien).
Remove deprecated features for the 0.20.x branch
Sep 2 2019, 07:48
Fabien closed D3978: Remove deprecated features for the 0.20.x branch.
Sep 2 2019, 07:48

Aug 31 2019

deadalnix accepted D3976: Merge #13148: logging: Fix potential use-after-free in LogPrintStr(...).
Aug 31 2019, 14:37
teamcity added a comment to D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Aug 31 2019, 00:52
teamcity added a comment to D3981: Merge #13081: wallet: Add compile time checking for cs_wallet runtime locking assertions.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Aug 31 2019, 00:48
teamcity added a comment to D3980: Merge #13077: Add compile time checking for all cs_KeyStore runtime locking assertions.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Aug 31 2019, 00:42
teamcity added a comment to D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 31 2019, 00:39
nakihito updated the summary of D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.
Aug 31 2019, 00:39
nakihito updated the summary of D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.
Aug 31 2019, 00:38
teamcity added a comment to D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 31 2019, 00:38
nakihito created D3982: Merge #13083: Add compile time checking for cs_main runtime locking assertions.
Aug 31 2019, 00:38
teamcity added a comment to D3981: Merge #13081: wallet: Add compile time checking for cs_wallet runtime locking assertions.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 31 2019, 00:37
nakihito created D3981: Merge #13081: wallet: Add compile time checking for cs_wallet runtime locking assertions.
Aug 31 2019, 00:37
teamcity added a comment to D3980: Merge #13077: Add compile time checking for all cs_KeyStore runtime locking assertions.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 31 2019, 00:34
nakihito created D3980: Merge #13077: Add compile time checking for all cs_KeyStore runtime locking assertions.
Aug 31 2019, 00:34

Aug 30 2019

teamcity added a comment to D3979: Merge #13159: Don't close old debug log file handle prematurely when trying to re-open (on SIGHUP).

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Aug 30 2019, 21:06
nakihito requested review of D3976: Merge #13148: logging: Fix potential use-after-free in LogPrintStr(...).

Fix completed by D3979.

Aug 30 2019, 20:54
nakihito updated the summary of D3979: Merge #13159: Don't close old debug log file handle prematurely when trying to re-open (on SIGHUP).
Aug 30 2019, 20:53
teamcity added a comment to D3979: Merge #13159: Don't close old debug log file handle prematurely when trying to re-open (on SIGHUP).

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 30 2019, 20:53
nakihito created D3979: Merge #13159: Don't close old debug log file handle prematurely when trying to re-open (on SIGHUP).
Aug 30 2019, 20:53
Nico Guiton <nico@bitframe.org> committed rABCf3cacd50640f: Merge #12507: Interrupt rescan on shutdown request (authored by Wladimir J. van der Laan <laanwj@gmail.com>).
Merge #12507: Interrupt rescan on shutdown request
Aug 30 2019, 20:28
Nico Guiton <nico@bitframe.org> closed D3975: Merge #12507: Interrupt rescan on shutdown request.
Aug 30 2019, 20:28
teamcity added a comment to D3968: Merge #12885: Reduce implementation code inside CScript.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Aug 30 2019, 18:59
teamcity added a comment to D3968: Merge #12885: Reduce implementation code inside CScript.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 30 2019, 18:48
nakihito updated the diff for D3968: Merge #12885: Reduce implementation code inside CScript.

Fixed variable type comparison warning.

Aug 30 2019, 18:47
teamcity added a comment to D3950: Merge #12953: Deprecate accounts.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Aug 30 2019, 18:17
teamcity added a comment to D3950: Merge #12953: Deprecate accounts.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 30 2019, 18:10
nakihito updated the diff for D3950: Merge #12953: Deprecate accounts.

Rebased.

Aug 30 2019, 18:10
deadalnix accepted D3978: Remove deprecated features for the 0.20.x branch.
Aug 30 2019, 18:03
teamcity added a comment to D3949: Merge #12892: [wallet] [rpc] introduce 'label' API for wallet.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Aug 30 2019, 18:02
teamcity added a comment to D3949: Merge #12892: [wallet] [rpc] introduce 'label' API for wallet.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 30 2019, 17:52
nakihito updated the diff for D3949: Merge #12892: [wallet] [rpc] introduce 'label' API for wallet.

Rebased and moved label and account function blocks inline with Core's placement.

Aug 30 2019, 17:51
teamcity added a comment to D3978: Remove deprecated features for the 0.20.x branch.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Aug 30 2019, 16:45
deadalnix changed the visibility for D3865: recognize bare multisigs as standard only when using minimal pushes.
Aug 30 2019, 16:37
teamcity added a comment to D3978: Remove deprecated features for the 0.20.x branch.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 30 2019, 16:33
Fabien created D3978: Remove deprecated features for the 0.20.x branch.
Aug 30 2019, 16:33
Mengerian added a revision to T653: Clean up past upgrades: D3160: Make SCRIPT_DISALLOW_SEGWIT_RECOVERY a standard flag.
Aug 30 2019, 15:30
Mengerian added a task to D3160: Make SCRIPT_DISALLOW_SEGWIT_RECOVERY a standard flag: T653: Clean up past upgrades.
Aug 30 2019, 15:30
deadalnix accepted D3732: Added a script to generate chainparams intermediate files.
Aug 30 2019, 14:06
deadalnix accepted D3975: Merge #12507: Interrupt rescan on shutdown request.
Aug 30 2019, 13:56
deadalnix requested changes to D3968: Merge #12885: Reduce implementation code inside CScript.

You got that wrong. The compiler is warning about it. You need to fix this.

Aug 30 2019, 13:54
Fabien committed rABCca85e129eea9: Make sure LC_ALL=C is set in all shell scripts (authored by practicalswift <practicalswift@users.noreply.github.com>).
Make sure LC_ALL=C is set in all shell scripts
Aug 30 2019, 13:50
Fabien closed D3964: Make sure LC_ALL=C is set in all shell scripts.
Aug 30 2019, 13:49
teamcity added a comment to D3964: Make sure LC_ALL=C is set in all shell scripts.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Aug 30 2019, 13:45
teamcity added a comment to D3965: [net] Tighten scope in net_processing.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Aug 30 2019, 13:39
teamcity added a comment to D3964: Make sure LC_ALL=C is set in all shell scripts.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 30 2019, 13:38
teamcity added a comment to D3964: Make sure LC_ALL=C is set in all shell scripts.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 30 2019, 13:37
Harbormaster failed remote builds in B7290: Diff 11044 for D3964: Make sure LC_ALL=C is set in all shell scripts!
Aug 30 2019, 13:36
teamcity added a comment to D3964: Make sure LC_ALL=C is set in all shell scripts.

Build Bitcoin-ABC / Diffs / Diff Testing failed.

Aug 30 2019, 13:36
teamcity added a comment to D3965: [net] Tighten scope in net_processing.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 30 2019, 13:26
deadalnix updated the diff for D3965: [net] Tighten scope in net_processing.

rebase

Aug 30 2019, 13:26
teamcity added a comment to D3964: Make sure LC_ALL=C is set in all shell scripts.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 30 2019, 13:26
Fabien updated the diff for D3964: Make sure LC_ALL=C is set in all shell scripts.

Rebase.

Aug 30 2019, 13:26
Fabien requested changes to D3965: [net] Tighten scope in net_processing.

Otherwise LGTM.

Aug 30 2019, 13:17
teamcity added a comment to D3963: Fix deprecated copy warnings in amount.h.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Aug 30 2019, 13:15
deadalnix committed rABC1cf3629759b9: Fix deprecated copy warnings in amount.h (authored by deadalnix).
Fix deprecated copy warnings in amount.h
Aug 30 2019, 13:11
deadalnix closed D3963: Fix deprecated copy warnings in amount.h.
Aug 30 2019, 13:11
deadalnix accepted D3967: Consistently log CValidationState on failure.
Aug 30 2019, 13:08
Fabien accepted D3963: Fix deprecated copy warnings in amount.h.
Aug 30 2019, 13:07
deadalnix retitled D3963: Fix deprecated copy warnings in amount.h from Enable deprecated copy warnings to Fix deprecated copy warnings in amount.h.
Aug 30 2019, 13:06
deadalnix accepted D3966: Default to defining endian-conversion DECLs in compat w/o config.

I'm not convinced this is really useful for us as we are moving toward cmake and always generating the config anyways, but it has value for consistency.

Aug 30 2019, 13:05
teamcity added a comment to D3963: Fix deprecated copy warnings in amount.h.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 30 2019, 13:00
deadalnix updated the diff for D3963: Fix deprecated copy warnings in amount.h.

Remove flags twiddling

Aug 30 2019, 13:00
Fabien committed rABC9e55cad08506: Bump version to 0.20.2 (authored by Fabien).
Bump version to 0.20.2
Aug 30 2019, 12:43
Fabien closed D3977: Bump version to 0.20.2.
Aug 30 2019, 12:43
deadalnix accepted D3977: Bump version to 0.20.2.
Aug 30 2019, 12:38
Fabien requested changes to D3949: Merge #12892: [wallet] [rpc] introduce 'label' API for wallet.

Please rebase on top of https://reviews.bitcoinabc.org/D3977

Aug 30 2019, 12:23
teamcity added a comment to D3977: Bump version to 0.20.2.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Aug 30 2019, 10:59
teamcity added a comment to D3977: Bump version to 0.20.2.

Build Bitcoin-ABC / Diffs / Diff Testing started.

Aug 30 2019, 10:50
Fabien created D3977: Bump version to 0.20.2.
Aug 30 2019, 10:49
nakihito planned changes to D3976: Merge #13148: logging: Fix potential use-after-free in LogPrintStr(...).

Not a complete fix. https://github.com/bitcoin/bitcoin/pull/13159 fixes remaining issues.

Aug 30 2019, 01:44
teamcity added a comment to D3976: Merge #13148: logging: Fix potential use-after-free in LogPrintStr(...).

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Aug 30 2019, 01:43