[Trivial] bitcoind: alias -h for -help #6846

Merged
merged 1 commit into from Oct 22, 2015

Conversation

Projects
None yet
6 participants
@dcousens
Contributor

dcousens commented Oct 18, 2015

Not sure if more than this is required.

Resolves #6838

@dcousens dcousens changed the title from bitcoind: alias -h for --help to [Trivial] bitcoind: alias -h for --help Oct 18, 2015

@btcdrak

This comment has been minimized.

Show comment
Hide comment
@btcdrak

btcdrak Oct 18, 2015

Member

utACK

Member

btcdrak commented Oct 18, 2015

utACK

@MarcoFalke

This comment has been minimized.

Show comment
Hide comment
@MarcoFalke

MarcoFalke Oct 18, 2015

Member

Any reason not to do this for the other bitcoin binaries?

Member

MarcoFalke commented Oct 18, 2015

Any reason not to do this for the other bitcoin binaries?

@paveljanik

This comment has been minimized.

Show comment
Hide comment
@paveljanik

paveljanik Oct 18, 2015

Contributor

Is it -help (as in the commit) or --help (as in the PR title)? Or both? ;-)

Contributor

paveljanik commented Oct 18, 2015

Is it -help (as in the commit) or --help (as in the PR title)? Or both? ;-)

@dcousens

This comment has been minimized.

Show comment
Hide comment
@dcousens

dcousens Oct 18, 2015

Contributor

@paveljanik seems it is -help, but --help also works [from the CLI].

Contributor

dcousens commented Oct 18, 2015

@paveljanik seems it is -help, but --help also works [from the CLI].

@dcousens dcousens changed the title from [Trivial] bitcoind: alias -h for --help to [Trivial] bitcoind: alias -h for -help Oct 18, 2015

@laanwj

This comment has been minimized.

Show comment
Hide comment
@laanwj

laanwj Oct 19, 2015

Member

Any reason not to do this for the other bitcoin binaries?

Yes, if you do this, please do it everywhere where -help is accepted.

Member

laanwj commented Oct 19, 2015

Any reason not to do this for the other bitcoin binaries?

Yes, if you do this, please do it everywhere where -help is accepted.

@laanwj laanwj added the Docs label Oct 19, 2015

@dcousens

This comment has been minimized.

Show comment
Hide comment
@dcousens

dcousens Oct 21, 2015

Contributor

@laanwj done.

Contributor

dcousens commented Oct 21, 2015

@laanwj done.

@MarcoFalke

This comment has been minimized.

Show comment
Hide comment
Member

MarcoFalke commented Oct 21, 2015

utACK 3cb56f3

@paveljanik

This comment has been minimized.

Show comment
Hide comment
@paveljanik

paveljanik Oct 21, 2015

Contributor

tested ACK. Now all bitcoind, bitcoin-cli and bitcoin-tx accept -h as an argument.

Contributor

paveljanik commented Oct 21, 2015

tested ACK. Now all bitcoind, bitcoin-cli and bitcoin-tx accept -h as an argument.

@jmcorgan

This comment has been minimized.

Show comment
Hide comment
@jmcorgan

jmcorgan Oct 21, 2015

Contributor

tested ACK

Contributor

jmcorgan commented Oct 21, 2015

tested ACK

@laanwj

This comment has been minimized.

Show comment
Hide comment
@laanwj

laanwj Oct 22, 2015

Member

ACK

Member

laanwj commented Oct 22, 2015

ACK

@laanwj laanwj merged commit 3cb56f3 into bitcoin:master Oct 22, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

laanwj added a commit that referenced this pull request Oct 22, 2015

Merge pull request #6846
3cb56f3 *: alias -h for --help (Daniel Cousens)

@dcousens dcousens deleted the dcousens:aliash branch Oct 22, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment