Page MenuHomePhabricator
Feed Advanced Search

May 20 2019

markblundeberg updated the summary of D3075: Merge #11872: [rpc] createrawtransaction: Accept sorted outputs.
May 20 2019, 16:12
Herald added a reviewer for D3075: Merge #11872: [rpc] createrawtransaction: Accept sorted outputs: deadalnix.
May 20 2019, 16:11
markblundeberg accepted D3074: Fix undefined behavior in avalanche.cpp.
May 20 2019, 15:10
markblundeberg added a comment to D3074: Fix undefined behavior in avalanche.cpp.

In src/tests/lcg.h it should pick up the same problem, does it?

May 20 2019, 14:09
markblundeberg updated the diff for D3072: Merge #11877: Improve createrawtransaction functional tests.

rebased for D3071

May 20 2019, 14:02
markblundeberg added a comment to D3073: [RPC] Serialize in getblock without cs_main.

Hey, just a heads up that this partly overlaps with D3021. Also when you're backporting things from Bitcoin Core, please make sure to mark them as backports and mention the PR number in the description.

May 20 2019, 13:51
markblundeberg updated the diff for D3072: Merge #11877: Improve createrawtransaction functional tests.

relint

May 20 2019, 13:44
markblundeberg added a comment to T639: Backport Core PR 11877 (tests).

Does not actually depend on that...

May 20 2019, 07:40
markblundeberg removed a parent task for T638: Backport Core PR 11738 (fix hang in sendrawtransaction): T639: Backport Core PR 11877 (tests).
May 20 2019, 07:40
markblundeberg removed a subtask for T639: Backport Core PR 11877 (tests): T638: Backport Core PR 11738 (fix hang in sendrawtransaction).
May 20 2019, 07:40
markblundeberg added a comment to D3072: Merge #11877: Improve createrawtransaction functional tests.

nb: needs to be re-linted apparently

May 20 2019, 07:31
markblundeberg added a revision to T639: Backport Core PR 11877 (tests): D3072: Merge #11877: Improve createrawtransaction functional tests.
May 20 2019, 07:29
markblundeberg added a task to D3072: Merge #11877: Improve createrawtransaction functional tests: T639: Backport Core PR 11877 (tests).
May 20 2019, 07:29
Herald added a reviewer for D3072: Merge #11877: Improve createrawtransaction functional tests: deadalnix.
May 20 2019, 07:28
Herald added a reviewer for D3071: Merge #12278: Add special error for genesis coinbase to getrawtransaction: deadalnix.
May 20 2019, 06:47

May 19 2019

markblundeberg updated the summary of D2924: [txindex] Activate new transaction index code that runs in background.
May 19 2019, 19:15
markblundeberg updated the diff for D3021: Merge #12151: rpc: Remove cs_main lock from blockToJSON and blockheaderToJSON.

rebased & separated out config removal into D3070

May 19 2019, 17:37
Herald added a reviewer for D3070: Remove config argument from blockToJSON: deadalnix.
May 19 2019, 17:26
markblundeberg updated the diff for D3014: Merge #10095: refactor: Move GetDifficulty out of `rpc/server.h`.

rebase

May 19 2019, 17:17

May 16 2019

markblundeberg accepted D3035: New -includeconf argument for including external configuration files.

OK :)

May 16 2019, 13:13

May 14 2019

markblundeberg requested review of D3014: Merge #10095: refactor: Move GetDifficulty out of `rpc/server.h`.

Please write a proper test plan.

May 14 2019, 23:08
markblundeberg updated the test plan for D3014: Merge #10095: refactor: Move GetDifficulty out of `rpc/server.h`.
May 14 2019, 23:07
markblundeberg updated the diff for D3014: Merge #10095: refactor: Move GetDifficulty out of `rpc/server.h`.

rebase for retest (same random rpc_listtransactions.py failure occured earlier in D2750)

May 14 2019, 22:47
markblundeberg updated the diff for D3014: Merge #10095: refactor: Move GetDifficulty out of `rpc/server.h`.

update per Fabien comments

May 14 2019, 17:37
markblundeberg added inline comments to D3014: Merge #10095: refactor: Move GetDifficulty out of `rpc/server.h`.
May 14 2019, 17:35
markblundeberg requested review of D3021: Merge #12151: rpc: Remove cs_main lock from blockToJSON and blockheaderToJSON.
May 14 2019, 17:18
markblundeberg added inline comments to D3021: Merge #12151: rpc: Remove cs_main lock from blockToJSON and blockheaderToJSON.
May 14 2019, 16:12
markblundeberg added inline comments to D3021: Merge #12151: rpc: Remove cs_main lock from blockToJSON and blockheaderToJSON.
May 14 2019, 16:11
markblundeberg added a comment to D3026: Merge #10267: New -includeconf argument for including external configuration files.
In D3026#71602, @Fabien wrote:

Clearing my queue, see D3035

May 14 2019, 15:02
markblundeberg abandoned D3026: Merge #10267: New -includeconf argument for including external configuration files.
May 14 2019, 15:01

May 13 2019

markblundeberg updated the diff for D3021: Merge #12151: rpc: Remove cs_main lock from blockToJSON and blockheaderToJSON.

rebase ; change casting style per comments

May 13 2019, 20:00
markblundeberg updated the diff for D3014: Merge #10095: refactor: Move GetDifficulty out of `rpc/server.h`.

rebase

May 13 2019, 19:46
markblundeberg accepted D3037: Added missing release notes.
May 13 2019, 19:40
markblundeberg added a comment to D3013: [rpc] Move tojson.h into blockchain.h.

Okay, I see. That's weird -- still not sure *toJSON functions should go there, but okay.

May 13 2019, 19:17
markblundeberg accepted D3035: New -includeconf argument for including external configuration files.
May 13 2019, 14:58
markblundeberg added a comment to D3035: New -includeconf argument for including external configuration files.

Ah I see the extra clause is the fix you mention.

May 13 2019, 14:43
markblundeberg added a comment to D3035: New -includeconf argument for including external configuration files.

Which one should we do? ( D3026 ) 馃槀

May 13 2019, 14:34
markblundeberg added inline comments to D2964: Merge #9804: Fixes subscript 0 (&var[0]) where should use (var.data()) instead..
May 13 2019, 13:52
markblundeberg added a comment to D3013: [rpc] Move tojson.h into blockchain.h.

I think more and more stuff should be better organized. Core doesn't seem to care about separating stuff out much.

However, this is just my opinion. It seems Amaury shared it. I hope we don't prevent ourself from making needed organizational changes indefinitely because of backports.

May 13 2019, 13:48
markblundeberg updated the diff for D3013: [rpc] Move tojson.h into blockchain.h.

add static

May 13 2019, 01:26
markblundeberg added a comment to D3013: [rpc] Move tojson.h into blockchain.h.

I think these should be left in place.

May 13 2019, 01:17

May 12 2019

markblundeberg updated subscribers of T651: Backport PR 11687 "External wallet files".
May 12 2019, 14:22Restricted Project
markblundeberg renamed T568: Backport PR12718 "[Tests] Require exact match in assert_start_raises_init_error" from Backport PR12718 to Backport PR12718 "[Tests] Require exact match in assert_start_raises_init_error".
May 12 2019, 13:47Restricted Project
markblundeberg triaged T651: Backport PR 11687 "External wallet files" as Normal priority.
May 12 2019, 13:17Restricted Project
markblundeberg added inline comments to D3012: Backport current GetDifficulty logic (& tests) from Core.
May 12 2019, 04:55
markblundeberg added a subtask for T540: Backport Core PR12755: T568: Backport PR12718 "[Tests] Require exact match in assert_start_raises_init_error".
May 12 2019, 01:45Restricted Project
markblundeberg added a parent task for T568: Backport PR12718 "[Tests] Require exact match in assert_start_raises_init_error": T540: Backport Core PR12755.
May 12 2019, 01:45Restricted Project
markblundeberg added a parent task for T447: Backport Core PR 12902: T540: Backport Core PR12755.
May 12 2019, 01:40Restricted Project
markblundeberg added a subtask for T540: Backport Core PR12755: T447: Backport Core PR 12902.
May 12 2019, 01:40Restricted Project
markblundeberg added a subtask for T650: Backport Core PR 13197 "util: warn about ignored recursive -includeconf calls": T540: Backport Core PR12755.
May 12 2019, 01:16
markblundeberg added a parent task for T540: Backport Core PR12755: T650: Backport Core PR 13197 "util: warn about ignored recursive -includeconf calls".
May 12 2019, 01:16Restricted Project
markblundeberg updated the diff for D3026: Merge #10267: New -includeconf argument for including external configuration files.

move release notes into main doc

May 12 2019, 01:06

May 11 2019

markblundeberg planned changes to D3026: Merge #10267: New -includeconf argument for including external configuration files.
May 11 2019, 23:55
markblundeberg added a comment to D3026: Merge #10267: New -includeconf argument for including external configuration files.

Note 13197 should be backported fairly soon too (T650)

May 11 2019, 23:55
markblundeberg triaged T650: Backport Core PR 13197 "util: warn about ignored recursive -includeconf calls" as Normal priority.
May 11 2019, 23:50
markblundeberg added a subtask for T650: Backport Core PR 13197 "util: warn about ignored recursive -includeconf calls": T541: Backport Core PR10267 to support includeconf.
May 11 2019, 23:50
markblundeberg added a parent task for T541: Backport Core PR10267 to support includeconf: T650: Backport Core PR 13197 "util: warn about ignored recursive -includeconf calls".
May 11 2019, 23:50Restricted Project
markblundeberg created T650: Backport Core PR 13197 "util: warn about ignored recursive -includeconf calls".
May 11 2019, 23:50
markblundeberg added a revision to T541: Backport Core PR10267 to support includeconf: D3026: Merge #10267: New -includeconf argument for including external configuration files.
May 11 2019, 23:43Restricted Project
markblundeberg added a task to D3026: Merge #10267: New -includeconf argument for including external configuration files: T541: Backport Core PR10267 to support includeconf.
May 11 2019, 23:43
Herald added a reviewer for D3026: Merge #10267: New -includeconf argument for including external configuration files: deadalnix.
May 11 2019, 23:43
markblundeberg accepted D3022: Merge #10045: [trivial] Fix typos in comments.
May 11 2019, 04:57

May 10 2019

markblundeberg accepted D2983: Merge #11351: Refactor: Modernize disallowed copy constructors/assignment.

Looks good, thanks!

May 10 2019, 18:00
markblundeberg updated the diff for D3011: Merge #9833: Trivial: fix comments referencing AppInit2.

fix oopsie noticed by Jason

May 10 2019, 16:17
markblundeberg updated the diff for D3010: update release notes.

update per Jason comments

May 10 2019, 16:14
markblundeberg added a comment to D3021: Merge #12151: rpc: Remove cs_main lock from blockToJSON and blockheaderToJSON.

Due to divergent histories, the blockchain_tests affected in original PR do not exist yet in ABC; this will be fixed in child diff (D3012)

May 10 2019, 16:13
markblundeberg updated the diff for D3012: Backport current GetDifficulty logic (& tests) from Core.

rebase onto D3021

May 10 2019, 16:10
markblundeberg updated the summary of D3014: Merge #10095: refactor: Move GetDifficulty out of `rpc/server.h`.
May 10 2019, 16:03
markblundeberg updated the summary of D3021: Merge #12151: rpc: Remove cs_main lock from blockToJSON and blockheaderToJSON.
May 10 2019, 16:02
Herald added a reviewer for D3021: Merge #12151: rpc: Remove cs_main lock from blockToJSON and blockheaderToJSON: deadalnix.
May 10 2019, 16:01
markblundeberg planned changes to D3012: Backport current GetDifficulty logic (& tests) from Core.
May 10 2019, 15:18
markblundeberg updated the summary of D3012: Backport current GetDifficulty logic (& tests) from Core.
May 10 2019, 14:43
markblundeberg added a comment to D3013: [rpc] Move tojson.h into blockchain.h.

note: TxToJSON is not in blockchain.cpp and doesn't need to be in a header either

May 10 2019, 05:30
markblundeberg updated the diff for D3012: Backport current GetDifficulty logic (& tests) from Core.

rebase onto 10095 backport

May 10 2019, 05:24
Herald added a reviewer for D3014: Merge #10095: refactor: Move GetDifficulty out of `rpc/server.h`: deadalnix.
May 10 2019, 05:19
Herald added a reviewer for D3013: [rpc] Move tojson.h into blockchain.h: deadalnix.
May 10 2019, 05:13
markblundeberg planned changes to D3012: Backport current GetDifficulty logic (& tests) from Core.
May 10 2019, 05:00
markblundeberg retitled D3012: Backport current GetDifficulty logic (& tests) from Core from Backport GetDifficulty logic from Core to Backport current GetDifficulty logic from Core.
May 10 2019, 04:27
Herald added a reviewer for D3012: Backport current GetDifficulty logic (& tests) from Core: deadalnix.
May 10 2019, 04:27
Herald added a reviewer for D3011: Merge #9833: Trivial: fix comments referencing AppInit2: deadalnix.
May 10 2019, 03:13
markblundeberg added a parent task for T648: Backport PR 12806 qa: Fix function names in feature_blocksdir: T631: Backport PR 15118 "Refactor block file logic".
May 10 2019, 02:57
markblundeberg added a subtask for T631: Backport PR 15118 "Refactor block file logic": T648: Backport PR 12806 qa: Fix function names in feature_blocksdir.
May 10 2019, 02:57
markblundeberg added a comment to T631: Backport PR 15118 "Refactor block file logic".

T648 too

May 10 2019, 02:57
Herald added a reviewer for D3010: update release notes: deadalnix.
May 10 2019, 02:52
markblundeberg closed T636: Backport PR 12653 "Allow to optional specify the directory for the blocks storage", a subtask of T631: Backport PR 15118 "Refactor block file logic", as Resolved.
May 10 2019, 02:44
markblundeberg closed T636: Backport PR 12653 "Allow to optional specify the directory for the blocks storage", a subtask of T648: Backport PR 12806 qa: Fix function names in feature_blocksdir, as Resolved.
May 10 2019, 02:44
markblundeberg closed T636: Backport PR 12653 "Allow to optional specify the directory for the blocks storage" as Resolved.
May 10 2019, 02:44
markblundeberg accepted D3009: Merge #9980: Fix mem access violation merkleblock.

still exists in this form in Core.

May 10 2019, 01:35
markblundeberg added inline comments to D2939: [backport PR12653] Allow to optional specify the directory for the blocks storage.
May 10 2019, 00:11
markblundeberg updated the diff for D2939: [backport PR12653] Allow to optional specify the directory for the blocks storage.

rebase ; braces

May 10 2019, 00:03
markblundeberg updated the task description for T636: Backport PR 12653 "Allow to optional specify the directory for the blocks storage".
May 10 2019, 00:00

May 9 2019

markblundeberg added a parent task for T568: Backport PR12718 "[Tests] Require exact match in assert_start_raises_init_error": T648: Backport PR 12806 qa: Fix function names in feature_blocksdir.
May 9 2019, 23:59Restricted Project
markblundeberg added a parent task for T636: Backport PR 12653 "Allow to optional specify the directory for the blocks storage": T648: Backport PR 12806 qa: Fix function names in feature_blocksdir.
May 9 2019, 23:59
markblundeberg added subtasks for T648: Backport PR 12806 qa: Fix function names in feature_blocksdir: T568: Backport PR12718 "[Tests] Require exact match in assert_start_raises_init_error", T636: Backport PR 12653 "Allow to optional specify the directory for the blocks storage".
May 9 2019, 23:59
markblundeberg triaged T648: Backport PR 12806 qa: Fix function names in feature_blocksdir as Low priority.
May 9 2019, 23:59
markblundeberg created T648: Backport PR 12806 qa: Fix function names in feature_blocksdir.
May 9 2019, 23:59
markblundeberg accepted D3008: Merge #10258: Fixed typo in documentation for merkleblock.h.
May 9 2019, 23:53
markblundeberg accepted D3007: Revert "Merge #10126: Compensate for memory peak at flush time".

I see, you mean that D1049 backported PR11022 already?

May 9 2019, 23:52
markblundeberg closed T633: Backport PR 12731 "Support serialization as another type without casting", a subtask of T630: Backport PR 13243 "Make reusable base class for auxiliary indices", as Resolved.
May 9 2019, 16:07
markblundeberg closed T633: Backport PR 12731 "Support serialization as another type without casting" as Resolved.
May 9 2019, 16:07