Page MenuHomePhabricator

Don't rename main thread at process level
ClosedPublic

Authored by Fabien on Mar 23 2020, 16:32.

Details

Summary
Set only the internal name.

Fixes #17036 for both `bitcoind` and `bitcoin-qt`.

This fixes Unix tools like ps or pkill that use the process name
(called bitcoin-init since D5540).

Backport of core PR17038.

Depends on D5540.

Test Plan
ninja check

bitcoind -daemon
ps -A | grep bitcoind # should return our bitcoind instance
sudo killall bitcoind
ps -A | grep bitcoind # should return nothing, kill successful

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR17038
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 9935
Build 17729: Default Diff Build & Tests
Build 17728: arc lint + arc unit

Event Timeline

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

This revision is now accepted and ready to land.Mar 23 2020, 17:33