[PATCH] completion: add bash completion for "notmuch reindex"

[PATCH] completion: add bash completion for "notmuch reindex"

[Daniel Kahn Gillmor]

The main thing that notmuch reindex does is to use search terms, so we
can reuse a bunch of the existing completion framework.
---
 completion/notmuch-completion.bash | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/completion/notmuch-completion.bash b/completion/notmuch-completion.bash
index dd0ec275..440ee3cd 100644
--- a/completion/notmuch-completion.bash
+++ b/completion/notmuch-completion.bash
@@ -417,6 +417,24 @@ _notmuch_search()
     esac
 }
 
+_notmuch_reindex()
+{
+    local cur prev words cword split
+    _init_completion -s || return
+
+    ! $split &&
+    case "${cur}" in
+	-*)
+	    local options="${_notmuch_shared_options}"
+	    compopt -o nospace
+	    COMPREPLY=( $(compgen -W "$options" -- ${cur}) )
+	    ;;
+	*)
+	    _notmuch_search_terms
+	    ;;
+    esac
+}
+
 _notmuch_address()
 {
     local cur prev words cword split
@@ -532,7 +550,7 @@ _notmuch_tag()
 
 _notmuch()
 {
-    local _notmuch_commands="compact config count dump help insert new reply restore search address setup show tag emacs-mua"
+    local _notmuch_commands="compact config count dump help insert new reply restore reindex search address setup show tag emacs-mua"
     local arg cur prev words cword split
 
     # require bash-completion with _init_completion
-- 
2.13.2

Re: [PATCH] completion: add bash completion for "notmuch reindex"

[David Bremner]

Daniel Kahn Gillmor <dkg@fifthhorseman.net> writes:

> The main thing that notmuch reindex does is to use search terms, so we
> can reuse a bunch of the existing completion framework.

pushed to master

d