Make env data logging optional #17699

pull sipa wants to merge 1 commits into bitcoin:master from sipa:201912_randlog changing 1 files +2 −2

sipa commented at 2:35 AM on December 9, 2019: member

The dynamic env feeding logging is a bit chatty, make it dependent on -debug=rand.
Make env data logging optional 0ccad08fb2
fanquake added the label Utils/log/libs on Dec 9, 2019
practicalswift commented at 7:27 AM on December 9, 2019: contributor

ACK 0ccad08fb25e00fcf41ffbad6ce2501e363a0033 -- less noise is good and diff looks correct
laanwj commented at 8:35 AM on December 9, 2019: member

ACK 0ccad08fb25e00fcf41ffbad6ce2501e363a0033
promag commented at 8:38 AM on December 9, 2019: member

ACK 0ccad08fb25e00fcf41ffbad6ce2501e363a0033.
jonatack commented at 8:44 AM on December 9, 2019: member
ACK 0ccad08fb25e00fcf41ffbad6ce2501e363a0033, was considering to propose this.

Edit: built and tested on running bitcoind, e.g.
```
    bitcoin-cli logging '["rand"]'
    bitcoin-cli logging '["rand"]' '["all"]'
```
laanwj referenced this in commit b6fb899ead on Dec 9, 2019
laanwj merged this on Dec 9, 2019
laanwj closed this on Dec 9, 2019
deadalnix referenced this in commit 717a0e24b5 on May 31, 2020
MarcoFalke locked this on Dec 16, 2021

Contributors

Labels