[PATCH 0/6] notmuch cli config changes

Jameson Graef Rollins jrollins at finestructure.net
Wed Feb 6 09:50:58 PST 2013


On Tue, Jan 29 2013, Jani Nikula <jani at nikula.org> wrote:
> Hi all, the goal here is to add support for --config=FILE option at the
> notmuch top level (e.g. 'notmuch --config=FILE search foo'). In order to
> achieve this neatly, I ended up moving config open/close to main() from
> subcommands. This isn't a bad thing, because all notmuch commands opened
> the config file anyway.

Hi, Jani.  I appreciate you've put a lot of work into this series, but
I'll be the same devil's advocate that I was to David previously.  Why
do we need a command line option here when we already have an
environment variable that handles just this?  Is there some benefit to
having a command line option for this that I don't see?  I see this as
another instance of an option that regular users will rarely use, if
ever.

In general, I am a strong advocate of keeping the CLI slim.  IMHO,
adding more options makes the interface clunkier, and the manual harder
to parse, and I'm against adding things that a normal user would likely
never use.  In retrospect, I should have had the same objection to the
--format-version option, which I think could have just been an env var
as well.

jamie.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20130206/19c57d67/attachment-0001.pgp>


More information about the notmuch mailing list