Page MenuHomePhabricator

New -includeconf argument for including external configuration files

Authored by Fabien on May 13 2019, 14:01.


  • adds -includeconf=<path>, where <path> is relative to datadir or to

the path of the file being read, if in a file

  • protects against circular includes
  • updates help docs

Backport of core PR10267

Includes a fix from PR13126

Completes T541
Progresses towards T652

Test Plan
make check
./test/functional/ feature_includeconf

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Fabien created this revision.May 13 2019, 14:01
Herald added a reviewer: Restricted Project. ยท View Herald TranscriptMay 13 2019, 14:01
Herald added a reviewer: deadalnix. ยท View Herald Transcript

Which one should we do? ( D3026 ) ๐Ÿ˜‚

Fabien abandoned this revision.May 13 2019, 14:38

You were first to submit, you get the priority

Ah I see the extra clause is the fix you mention.

7 โ†—(On Diff #8631)

I like your release notes more than mine :)

391 โ†—(On Diff #8631)

should alphabetically sort after -finalizationdelay

818 โ†—(On Diff #8631)

hmm mine lacks this clause

108 โ†—(On Diff #8631)

Interesting, didn't know about this file!

Fabien reclaimed this revision.May 13 2019, 14:53
Fabien updated this revision to Diff 8632.May 13 2019, 14:54

Update help ordering and add missing newline in release notes.

Fabien edited the summary of this revision. (Show Details)May 13 2019, 14:55
markblundeberg accepted this revision.May 13 2019, 14:58
This revision is now accepted and ready to land.May 13 2019, 14:58
Fabien updated this revision to Diff 8685.May 16 2019, 07:25


Fabien updated this revision to Diff 8773.May 21 2019, 09:24


This revision was automatically updated to reflect the committed changes.