Re: [PATCH 7/9] add a gpg_path value for notmuch_database

Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t

Subject: Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t

Date: Fri, 11 Dec 2015 23:10:19 -0500

Cc:

On Fri 2015-12-11 17:35:58 -0500, J. Lewis Muir wrote:
> I guess I still don't get it.  Why even have a _find_in_path function?
> Why not just expect the gpg executable path to have already been
> specified somehow (e.g. Notmuch configuration file, build-time constant,
> or environment variable)?

This is happening in the library, which doesn't read the config file,
and doesn't depend on the environment.  if a user tells the library "use
this as your gpg executable", it seems nice to fail early for them.

It's also nice if we want to have a default that says something like
"use gpg2 if it's available, but gpg otherwise".

     --dkg

Previous message (by thread): Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t

Thread:

Daniel Kahn Gillmor—allow indexing cleartext of encrypted messages [inbox, unread]
- Daniel Kahn Gillmor—[PATCH 1/9] reorganize indexing of multipart/signed and multipart/encrypted [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Tomi Ollila—Re: [PATCH 1/9] reorganize indexing of multipart/signed and multipart/encrypted [inbox, unread]
- Daniel Kahn Gillmor—[PATCH 2/9] Add a lazily-initialized crypto context to notmuch_database_t [inbox, notmuch::obsolete, notmuch::patch, unread]
  - David Bremner—Re: [PATCH 2/9] Add a lazily-initialized crypto context to notmuch_database_t [inbox, unread]
    - Daniel Kahn Gillmor—Re: [PATCH 2/9] Add a lazily-initialized crypto context to notmuch_database_t [inbox, unread]
  - Tomi Ollila—Re: [PATCH 2/9] Add a lazily-initialized crypto context to notmuch_database_t [inbox, unread]
- Daniel Kahn Gillmor—[PATCH 3/9] index encrypted parts when the message is flagged appropriately [inbox, notmuch::obsolete, notmuch::patch, unread]
- Daniel Kahn Gillmor—[PATCH 4/9] Add new n_d_add_message_try_decrypt (analogous to to n_d_add_message) [inbox, notmuch::obsolete, notmuch::patch, unread]
- Daniel Kahn Gillmor—[PATCH 5/9] Enable cleartext indexing in python bindings [inbox, notmuch::obsolete, notmuch::patch, unread]
- Daniel Kahn Gillmor—[PATCH 6/9] search for a reasonable gpg implementation [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Tomi Ollila—Re: [PATCH 6/9] search for a reasonable gpg implementation [inbox, unread]
    - J. Lewis Muir—Re: [PATCH 6/9] search for a reasonable gpg implementation [inbox, unread]
      - Daniel Kahn Gillmor—Re: [PATCH 6/9] search for a reasonable gpg implementation [inbox, unread]
- Daniel Kahn Gillmor—[PATCH 7/9] add a gpg_path value for notmuch_database_t [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Tomi Ollila—Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t [inbox, unread]
    - Daniel Kahn Gillmor—Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t [inbox, unread]
      - Tomi Ollila—Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t [inbox, unread]
      - David Bremner—Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t [inbox, unread]
        Tomi Ollila—Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t [inbox, unread]
        Tomi Ollila—Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t [inbox, unread]
        Daniel Kahn Gillmor—Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t [inbox, unread]
  - J. Lewis Muir—Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t [inbox, unread]
    - Daniel Kahn Gillmor—Re: [PATCH 7/9] add a gpg_path value for notmuch_database_t [inbox, unread]
- Daniel Kahn Gillmor—[PATCH 8/9] add --try-decrypt to notmuch insert [inbox, notmuch::obsolete, notmuch::patch, unread]
- Daniel Kahn Gillmor—[PATCH 9/9] add --try-decrypt to notmuch new [inbox, notmuch::obsolete, notmuch::patch, unread]
- Daniel Kahn Gillmor—Re: allow indexing cleartext of encrypted messages [inbox, unread]
  - Tomi Ollila—Re: allow indexing cleartext of encrypted messages [inbox, unread]