diff options
| author | merge-script <fanquake@gmail.com> | 2024-04-10 12:03:06 +0200 |
|---|---|---|
| committer | merge-script <fanquake@gmail.com> | 2024-04-10 12:03:06 +0200 |
| commit | 0a9cfd175259391ec29b0dfe87578d46b508611e (patch) | |
| tree | d54759a5acc2ea8675de083bb8c79a59ea40b838 /depends | |
| parent | e31956980e16ad3d619022e572bdf55a4eae8716 (diff) | |
| parent | d5a715536e497c160a2520f81334aab6c7490213 (diff) | |
| download | bitcoin-0a9cfd175259391ec29b0dfe87578d46b508611e.tar.xz | |
Merge bitcoin/bitcoin#28981: Replace Boost.Process with cpp-subprocess
d5a715536e497c160a2520f81334aab6c7490213 build: remove boost::process dependency for building external signer support (Sebastian Falbesoner)
70434b1c443d9251a880d0193af771f574c40617 external_signer: replace boost::process with cpp-subprocess (Sebastian Falbesoner)
cc8b9875b104c31f0a5b5e4195a8278ec55f35f7 Add `cpp-subprocess` header-only library (Hennadii Stepanov)
Pull request description:
Closes https://github.com/bitcoin/bitcoin/issues/24907.
This PR is based on **theStack**'s [work](https://github.com/bitcoin/bitcoin/issues/24907#issuecomment-1466087049).
The `subprocess.hpp` header has been sourced from the [upstream repo](https://github.com/arun11299/cpp-subprocess) with the only modification being the removal of convenience functions, which are not utilized in our codebase.
Windows-related changes will be addressed in subsequent follow-ups.
ACKs for top commit:
achow101:
reACK d5a715536e497c160a2520f81334aab6c7490213
Sjors:
re-tACK d5a715536e497c160a2520f81334aab6c7490213
theStack:
Light re-ACK d5a715536e497c160a2520f81334aab6c7490213
fanquake:
ACK d5a715536e497c160a2520f81334aab6c7490213 - with the expectation that this code is going to be maintained as our own. Next PRs should:
Tree-SHA512: d7fb6fecc3f5792496204190afb7d85b3e207b858fb1a75efe483c05260843b81b27d14b299323bb667c990e87a07197059afea3796cf218ed8b614086bd3611
Diffstat (limited to 'depends')
0 files changed, 0 insertions, 0 deletions