Skip to content

Conversation

@unsystemizer
Copy link
Contributor

Added "conf, pid, and wallet accept relative paths which are interpreted as relative to the data directory. wallet only supports relative paths."

Added "conf, pid, and wallet accept relative paths which are interpreted as relative to the data directory. wallet only supports relative paths."
@laanwj
Copy link
Member

laanwj commented Aug 4, 2015

Looks good to me.

@laanwj laanwj added the Docs label Aug 4, 2015
@laanwj
Copy link
Member

laanwj commented Aug 4, 2015

But an easier-to-find place may be to mention it in the --help messages for the appropriate options in init.cpp itself.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not related to this PR, but this is no longer the case, right? It will generate a cookie instead.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct. It was changed in #6388

@laanwj laanwj merged commit fc25a87 into bitcoin:master Aug 10, 2015
laanwj added a commit that referenced this pull request Aug 10, 2015
fc25a87 Add note on relative paths, improve formatting (unsystemizer)
@unsystemizer unsystemizer deleted the patch-1 branch August 11, 2015 12:55
luke-jr pushed a commit to luke-jr/bitcoin that referenced this pull request Jan 9, 2016
Added "conf, pid, and wallet accept relative paths which are interpreted as relative to the data directory. wallet only supports relative paths."
Github-Pull: bitcoin#6512
Rebased-From: fc25a87
luke-jr pushed a commit to luke-jr/bitcoin that referenced this pull request Jan 10, 2016
Added "conf, pid, and wallet accept relative paths which are interpreted as relative to the data directory. wallet only supports relative paths."
Github-Pull: bitcoin#6512
Rebased-From: fc25a87
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants