[PATCH v4 3/3] cli/reply: make --decrypt take a keyword
Daniel Kahn Gillmor
dkg at fifthhorseman.net
Fri Dec 29 15:04:55 PST 2017
On Fri 2017-12-29 10:30:00 -0400, David Bremner wrote:
> Daniel Kahn Gillmor <dkg at fifthhorseman.net> writes:
>
>> No, we should not support --decrypt=nostash for show or reply. The
>> semantics of the display commands (show, reply) are such that they
>> *never* modify the index or stash anything in there. The equivalent for
>> the indexing (new, insert, reindex) commands' "--decrypt=nostash" in the
>> display commands is simply "--decrypt=true".
>
> I'm not sure I completely agree, but its a trivial matter to add nostash
> later if desired. And it's always easier to add API / command options
> than to take them away.
yes, true that we can always expand out, and it's more parsimonious to
start small. :)
It sounds like you were suggesting "--decrypt=nostash" as a synonym for
"--decrypt=true" on show/reply, which i confess i didn't fully
understand when i wrote my response. If it really makes you feel better
to add the alias/synonym, i wouldn't block such a change.
But I think the documentation is tricky to write (and trickier to read
and trickier still to understand!) if "--decrypt=nostash" means the
same thing as "--decrypt=true" in one context, but they mean different
things in a different context.
I'm still trying to aim for that sweet spot where the smallest possible
API guides the user to sensible decisions while still understanding
what's going on. :)
--dkg
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20171229/38af40fd/attachment.sig>
More information about the notmuch
mailing list