summaryrefslogtreecommitdiff
path: root/tests/TODO.txt
diff options
context:
space:
mode:
authorMarc Cornellà <marc.cornella@live.com>2014-07-28 08:04:49 +0200
committerJason A. Donenfeld <Jason@zx2c4.com>2014-07-29 19:19:36 +0200
commit5a6df91e49376c701381b99d89d61af698c98463 (patch)
tree40987de285df8c3cae05ea9fa47d1d57bb004d1e /tests/TODO.txt
parent50895b559eb252323b989efb47a32cce7af7551a (diff)
downloadpass-5a6df91e49376c701381b99d89d61af698c98463.tar.gz
pass-5a6df91e49376c701381b99d89d61af698c98463.tar.bz2
pass-5a6df91e49376c701381b99d89d61af698c98463.zip
Fix pass zsh completion and autoloading
When autocompleting from `pass <TAB>', sometimes the following errors appear: _values:compvalues:10: not enough arguments find: `/home/user/.password-store': No such file or directory _values:compvalues:10: not enough arguments find: `/home/user/.password-store': No such file or directory The `_values' error happens when there is no password-store folder *or* there are no passwords in pass; the `find' error only when there is no password-store folder. We can trace it back to line 108, which contains the only `_values' statement that is executed when we autocomplete from pass. We confirm this by following the trail of execution, which is _pass -> _pass_cmd_show -> _pass_complete_entries -> -> _pass_complete_entries_helper If we try running the command inside `$()' on line 104, we see that it returns nothing and the output is blank. This means that `_values' only receives 1 of its 2 mandatory parameters, therefore the above error is triggered (not enough arguments). That is unless we don't have a password-store folder, in which case the `find: [...] no such file or directory' error is *also* triggered. We solve the first error by supplying a default value of "" if the command outputs nothing, using the zsh construct ${var:-else}. We solve the second error by redirecting the find command's stderr output to /dev/null, so the error is effectively suppressed. * * * * This patch also fixes the first tab completion, which currently only loads the completion function definition. We do this by adding a `_pass' statement at the end of the file, which runs the `_pass' completion function after loading its definition. This is the standard way an autoloaded function works; for other examples look at zsh's official completion files.
Diffstat (limited to 'tests/TODO.txt')
0 files changed, 0 insertions, 0 deletions
.git/commit/.config/dunst/dunstrc?id=b0429d05f562448e9f4366bf4602e05789634302&follow=1'>dunst: Revert to Monospace 8René 'Necoro' Neumann1-1/+1 2013-06-18Remove subtle configRené 'Necoro' Neumann1-876/+0 2013-06-18dunst: Use DroidSansMonoRené 'Necoro' Neumann1-1/+1 2013-06-18i3: workspaces.py -- small cosmetic changesRené 'Necoro' Neumann1-3/+3 2013-06-18i3: Use i3-msg 'exec …' for processes, not sh.CommandRené 'Necoro' Neumann2-3/+8 2013-06-17i3: workspaces.py -- python errorRené 'Necoro' Neumann1-0/+1 2013-06-17i3: workspaces.py -- python errorRené 'Necoro' Neumann1-1/+1 2013-06-17i3 changesRené 'Necoro' Neumann2-2/+2 2013-06-14i3: New 'open new workspace with a terminal on'René 'Necoro' Neumann2-15/+31 2013-06-14Vimperator: Do not open popups in a new window -- sucks with tiling WMsRené 'Necoro' Neumann1-1/+1 2013-06-08Cleanup X startup filesRené 'Necoro' Neumann3-19/+42 2013-06-07i3: better renameRené 'Necoro' Neumann2-3/+17 2013-06-07i3: Some keybinding changes.René 'Necoro' Neumann1-5/+6 2013-06-07i3: Overhaul workspaces.pyRené 'Necoro' Neumann1-27/+19 2013-06-06i3: use $mod+t for tabbed instead of $mod+wRené 'Necoro' Neumann1-1/+1 2013-06-06i3: unify scripts into oneRené 'Necoro' Neumann6-49/+1771 2013-06-06i3: new temporary workspaceRené Neumann2-1/+27 2013-06-06Allow 'klass' in i3.py descriptionsRené Neumann1-0/+4 2013-06-04i3: rename workspaceRené Neumann1-0/+3 2013-06-03Some more greek chars / math product and sumRené 'Necoro' Neumann1-0/+6 2013-06-02Fix rules in i3-config. Somehow 'set' expands variables twice :-/René Neumann1-14/+11 2013-06-01Overhaul xlock.shRené 'Necoro' Neumann1-4/+8 2013-06-01Better workspace chooserRené 'Necoro' Neumann3-2/+581 2013-06-01FF/TB/Pidgin stuffRené 'Necoro' Neumann1-26/+45 2013-06-01More i3René 'Necoro' Neumann1-2/+11 2013-06-01FixesRené 'Necoro' Neumann2-2/+2