Allow per network configuration file #9402

pull rebroad wants to merge 1 commits into bitcoin:master from rebroad:PerNetworkConfig changing 6 files +35 −7

rebroad commented at 3:45 PM on December 21, 2016: contributor

Fixes #9374
rebroad force-pushed on Dec 21, 2016
rebroad renamed this:
~~Allow per network bitcoin.conf~~
Allow per network configuration file
on Dec 21, 2016
kallewoof commented at 2:13 AM on December 22, 2016: member

Since netconf values should override main config settings, you want to load netconf before conf, I think. See https://github.com/bitcoin/bitcoin/blame/master/src/util.cpp#L546
laanwj commented at 12:24 PM on December 22, 2016: member

Since netconf values should override main config settings, you want to load netconf before conf, I think

That's not possible, as conf can specify the network.
kallewoof commented at 10:38 PM on December 22, 2016: member

Edited: What I said doesn't really make a difference. It's still not possible as you say.
rebroad closed this on Dec 23, 2016
rebroad reopened this on Dec 24, 2016
rebroad force-pushed on Dec 24, 2016
rebroad force-pushed on Dec 28, 2016
rebroad force-pushed on Dec 28, 2016
rebroad force-pushed on Dec 28, 2016

Allow per network network.conf

The network.conf is processed in the same way as the bitcoin.conf and it
is looked for in the data directory, and processed after the
bitcoin.conf.

Fixes #9374

d4ff270b26

rebroad force-pushed on Dec 28, 2016
fanquake commented at 2:42 PM on January 12, 2017: member

Needs a rebase.
fanquake closed this on Apr 2, 2017
fanquake commented at 9:56 AM on April 2, 2017: member

Lack of feedback, and still needs a rebase. Feel free to reopen when it has been.
DrahtBot locked this on Sep 8, 2021

Contributors

Linked (view graph)

#9374 Per-network bitcoin.conf files #10996 Add per-network config file network.conf