Skip to content

Conversation

@hebasto
Copy link
Member

@hebasto hebasto commented Aug 4, 2018

Currently help2man produces some unexpected commands in the manual page file bitcoin-cli.1:

  • .TP + .IP for 'Usage' in DESCRIPTION section. Should be all .TP;
  • .TP + .IP for '-stdinrpcpass' in OPTIONS section. Should be only an .IP

Currently help2man produces some unexpected commands in manual page file bitcoin-cli.1:
1) .TP + .IP for 'Usage' in DESCRIPTION section. Should be all .TP;
2) .TP + .IP for '-stdinrpcpass' in OPTIONS section. Should be only an .IP
@DrahtBot
Copy link
Contributor

DrahtBot commented Aug 5, 2018

Note to reviewers: This pull request conflicts with the following ones:

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

strUsage += "\nUsage:\n"
" bitcoin-cli [options] <command> [params] " + strprintf("Send command to %s", PACKAGE_NAME) + "\n" +
" bitcoin-cli [options] -named <command> [name=value] ... " + strprintf("Send command to %s (with named arguments)", PACKAGE_NAME) + "\n" +
" bitcoin-cli [options] -named <command> [name=value]... " + strprintf("Send command to %s (with named arguments)", PACKAGE_NAME) + "\n" +
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the solution in #13872 is better for help2man.

Copy link
Member Author

Choose a reason for hiding this comment

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

#13872 introduces SYNOPSIS section to the all bitcoind, bitcoin-qt and utilities manual pages. I think that is a good move. And, as a side effect, #13872 fixes one of two bugs in bitcoin-cli --help output.

#13879 just fixes all found bugs in bitcoin-cli --help output and is safe for the whole code.

@laanwj
Copy link
Member

laanwj commented Aug 7, 2018

needs rebase (if still necessary) after #13872

@DrahtBot
Copy link
Contributor

DrahtBot commented Aug 7, 2018

Needs rebase

@hebasto
Copy link
Member Author

hebasto commented Aug 7, 2018

Pull request after rebase: #13905

@hebasto hebasto closed this Aug 7, 2018
@bitcoin bitcoin locked and limited conversation to collaborators Aug 8, 2018
@hebasto hebasto deleted the bitcoin-cli-help2man branch August 13, 2018 13:26
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.

5 participants