diff options
| author | fanquake <fanquake@gmail.com> | 2022-10-10 17:41:29 +0800 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2022-10-10 17:58:18 +0800 |
| commit | 866dd664a16de341262b1f1a7bfd46be7eb78a8e (patch) | |
| tree | 24285b19d0fd44a92680e7f114b34f48577ecf86 /src/policy | |
| parent | 869342f7fade69396c9966cfdbceca335b093f65 (diff) | |
| parent | 43b8777dc3e63f4a1b20a3cb23e44c1b9e32862b (diff) | |
| download | bitcoin-866dd664a16de341262b1f1a7bfd46be7eb78a8e.tar.xz | |
Merge bitcoin/bitcoin#26196: kernel: move RunCommandParseJSON to its own file
43b8777dc3e63f4a1b20a3cb23e44c1b9e32862b refactor: move run_command from util to common (Cory Fields)
192325a77d593e404e74ef5e204aed8801b4e66f kernel: move RunCommandParseJSON to its own file (Cory Fields)
Pull request description:
Because libbitcoinkernel does not include this new object, this has the side-effect of eliminating its unnecessary `boost::process` dependency.
This leaves libbitcoinkernel with 3 remaining boost dependencies:
- `boost::date_time` for `util/time.cpp`, which I'll separate out next. Exactly like this PR.
- `boost::signals2` for which I have a POC re-implementation here: https://github.com/theuni/bitcoin/commits/replace-boost-signals
- `boost::multi_index` which I'm not sure about yet.
ACKs for top commit:
ryanofsky:
Code review ACK 43b8777dc3e63f4a1b20a3cb23e44c1b9e32862b. Could consider squashing the two commits, so the code just moves once instead of twice.
fanquake:
ACK 43b8777dc3e63f4a1b20a3cb23e44c1b9e32862b
Tree-SHA512: f2a46cac34aaadfb8a1442316152ad354f6990021b82c78d80cae9fd43cd026209ffd62132eaa99d5d0f8cf34e996b6737d318a9d9a3f1d2ff8d17d697abf26d
Diffstat (limited to 'src/policy')
0 files changed, 0 insertions, 0 deletions