bash-completion: Adapt for 0.12 and 0.13#8289
Conversation
roques
commented
Jun 29, 2016
- separate completion for bitcoind and bitcoin-cli
- remove RPC support from bitcoind completion
- add completion for bitcoin-tx and bitcoin-qt
|
Nice! Thanks. |
contrib/bitcoin-tx.bash-completion
Outdated
There was a problem hiding this comment.
Are the first two chars desired?
There was a problem hiding this comment.
Definitely not. Somehow I too often hit «1» instead of «esc» these days.
|
Concept ACK. Would you mind switching to the standard project copyright header while you're at it? |
|
@luke-jr, I switched to a copyright year range instead of an enumeration, as it is more popular here. |
|
The "Copyright Bitcoin Core developers" is essentially saying each author retains copyright to their own contributions; there is no centralised/formal/legal entity. Maybe we should clarify this somewhere. (To be clear, I don't consider this a blocker to your PR, just a "nice to have".) |
|
If you are the sole author of the file, it is fine to just put your name and not "Bitcoin Core developers". However, if someone modifies the file, we just put "Bitcoin Core developers" in there to not clutter the headers. The detailed list of authors could then be fetched via |
|
I did some more digging through the Copyright headers. Below |
|
Travis failure unrelated in tests.exe: |
|
The phrase "Bitcoin Core developers" in the copyright notice just means "developers of Bitcoin Core" - which is anyone at all who has contributed anything. There's a good writeup on this subject that here, which takes into account how distributed revision control systems like git do a good job of recording exact authorship for legal purposes: https://softwarefreedom.org/resources/2012/ManagingCopyrightInformation.html Concept ACK |
You cannot 'accidentally' assign your copyright to someone else, only by signing an explicit contract. Some projects (such as leveldb) have a separate contributor license agreement that transfers copyright to a company or organization (to Google in that case). Bitcoin Core does not have that. |
|
Anyone knows what is the proper way to test this on fedora with just
self-compiled Bitcoin Core?
|
My guess would be typing this into your shell: export PATH=$PATH:$BUILDDIR/src
source contrib/bitcoin-tx.bash-completion
source contrib/bitcoind.bash-completionThe commands need to be in your path. The completion script are simply bash scripts that define a function and install it using |
* separate completion for bitcoind and bitcoin-cli * remove RPC support from bitcoind completion * add completion for bitcoin-tx and bitcoin-qt * rely on autoloading of completions
|
I've looked into testing the completions without installing them and found that the use of |
|
Appears to be working: Though, I couldn't get it work with the cli: shows nothing |
|
@MarcoFalke Seems to work here, do you have the daemon running? I think it queries the remote for the command list: |
|
ACK 1ba3db6 |
|
I am not running the daemon. This is good for merge. |
1ba3db6 bash-completion: Adapt for 0.12 and 0.13 (Christian von Roques)
Bash completion This PR pulls in bitcoin/bitcoin#8289, updates the bash completion files for use with Zcash, and bundles them into the Debian package.
1ba3db6 bash-completion: Adapt for 0.12 and 0.13 (Christian von Roques)
1ba3db6 bash-completion: Adapt for 0.12 and 0.13 (Christian von Roques)