[PATCH 2/2] completion: generate notmuch-completion.zsh using doc/conf.py

Tomi Ollila tomi.ollila at iki.fi
Sat Jan 3 05:49:36 PST 2015


On Sat, Jan 03 2015, David Bremner <david at tethera.net> wrote:

> David Bremner <david at tethera.net> writes:
>
>> I noticed that the (out of date) list of subcommands was simply
>> repeating the summary lines from the manpage. Since those summary
>> lines are already kept in machine readable form in doc/conf.py,
>> it's easy to compute that part of the completion file.
>
> I meant to annotate this, but lost a fight with git-send-email.  I'm not
> sure if the change to notmuch-completions.zsh is worth the trouble (how
> often will we add subcommands?), but the configuration change should
> probably go in anyway, at least one person already mentioned this in a
> bug report.

I agree that 2/2 may be left out (I rather skip checking it for now ;)

The changes in 1/2 looks good, but you could have included the new python
definition in line ~454 in (new) configure, which originally is:

    if hash sphinx-build > /dev/null 2>&1 && python -m sphinx.writers.manpage >/dev/null 2>&1 ; then

(actually, this could be like:

    if command -v sphinx-build >/dev/null && "$python" -m sphinx.writers.manpage >/dev/null 2>&1 ; then

    (add that 2>&1 to be consistent with other code, I did not include it
    as I think it is wrong to do so ;))


Tomi


More information about the notmuch mailing list