slaesvuo/hledger
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bugfix: Must use array here because of word splitting

Browse Source
This commit is contained in:
Jakob Schöttl 2019-01-11 23:41:45 +01:00 committed by Simon Michael
parent e3826be379
commit 1cd979ae18
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
shell-completion/hledger-completion.bash.m4
View File

@ -1,4 +1,4 @@
#!/bin/bash
# Completion script for hledger. # Completion script for hledger.
# Created using a Makefile and real hledger. # Created using a Makefile and real hledger.
@ -52,7 +52,12 @@ hledgerCompletionFunction() {
if [[ -n $completeFiles ]]; then if [[ -n $completeFiles ]]; then
#COMP_WORDBREAKS='= ' #COMP_WORDBREAKS='= '
COMPREPLY+=( $(compgen -df | grep "^$filenameSoFar") ) declare -a files
# This does not work because assignment to 'files' in the "pipe
# subshell" has no effect!
#compgen -df | grep "^$filenameSoFar" | readarray -t files
readarray -t files < <(compgen -df | grep "^$filenameSoFar")
COMPREPLY=( "${files[@]}" )
else else
COMPREPLY+=( $(grep -h "^$wordToComplete" -- "$HLEDGER_COMPLETION_TEMPDIR/commands.txt" "$HLEDGER_COMPLETION_TEMPDIR/generic-options.txt") ) COMPREPLY+=( $(grep -h "^$wordToComplete" -- "$HLEDGER_COMPLETION_TEMPDIR/commands.txt" "$HLEDGER_COMPLETION_TEMPDIR/generic-options.txt") )