1 Return-Path: <jrollins@finestructure.net>
\r
2 X-Original-To: notmuch@notmuchmail.org
\r
3 Delivered-To: notmuch@notmuchmail.org
\r
4 Received: from localhost (localhost [127.0.0.1])
\r
5 by olra.theworths.org (Postfix) with ESMTP id E7F06431FB6
\r
6 for <notmuch@notmuchmail.org>; Wed, 6 Feb 2013 09:51:10 -0800 (PST)
\r
7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org
\r
11 X-Spam-Status: No, score=-2.3 tagged_above=-999 required=5
\r
12 tests=[RCVD_IN_DNSWL_MED=-2.3] autolearn=disabled
\r
13 Received: from olra.theworths.org ([127.0.0.1])
\r
14 by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)
\r
15 with ESMTP id ctryD9fRmJgu for <notmuch@notmuchmail.org>;
\r
16 Wed, 6 Feb 2013 09:51:07 -0800 (PST)
\r
17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu
\r
19 by olra.theworths.org (Postfix) with ESMTP id E44C1431FAF
\r
20 for <notmuch@notmuchmail.org>; Wed, 6 Feb 2013 09:51:06 -0800 (PST)
\r
21 Received: from earth-doxen.imss.caltech.edu (localhost [127.0.0.1])
\r
22 by earth-doxen-postvirus (Postfix) with ESMTP id 69B9166E01EB;
\r
23 Wed, 6 Feb 2013 09:51:06 -0800 (PST)
\r
24 X-Spam-Scanned: at Caltech-IMSS on earth-doxen by amavisd-new
\r
25 Received: from finestructure.net (adsl-76-240-184-246.dsl.irvnca.sbcglobal.net
\r
26 [76.240.184.246]) (Authenticated sender: jrollins)
\r
27 by earth-doxen-submit (Postfix) with ESMTP id 5FD2766E01C9;
\r
28 Wed, 6 Feb 2013 09:51:03 -0800 (PST)
\r
29 Received: by finestructure.net (Postfix, from userid 1000)
\r
30 id E9E60616F3; Wed, 6 Feb 2013 09:51:00 -0800 (PST)
\r
31 From: Jameson Graef Rollins <jrollins@finestructure.net>
\r
32 To: Jani Nikula <jani@nikula.org>, notmuch@notmuchmail.org
\r
33 Subject: Re: [PATCH 0/6] notmuch cli config changes
\r
34 In-Reply-To: <cover.1359495450.git.jani@nikula.org>
\r
35 References: <cover.1359495450.git.jani@nikula.org>
\r
36 User-Agent: Notmuch/0.15+8~gd4a7374 (http://notmuchmail.org) Emacs/24.2.1
\r
37 (x86_64-pc-linux-gnu)
\r
38 Date: Wed, 06 Feb 2013 09:50:58 -0800
\r
39 Message-ID: <871ucts54t.fsf@servo.finestructure.net>
\r
41 Content-Type: multipart/signed; boundary="=-=-=";
\r
42 micalg=pgp-sha256; protocol="application/pgp-signature"
\r
43 X-BeenThere: notmuch@notmuchmail.org
\r
44 X-Mailman-Version: 2.1.13
\r
46 List-Id: "Use and development of the notmuch mail system."
\r
47 <notmuch.notmuchmail.org>
\r
48 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,
\r
49 <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>
\r
50 List-Archive: <http://notmuchmail.org/pipermail/notmuch>
\r
51 List-Post: <mailto:notmuch@notmuchmail.org>
\r
52 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>
\r
53 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,
\r
54 <mailto:notmuch-request@notmuchmail.org?subject=subscribe>
\r
55 X-List-Received-Date: Wed, 06 Feb 2013 17:51:11 -0000
\r
58 Content-Type: text/plain
\r
60 On Tue, Jan 29 2013, Jani Nikula <jani@nikula.org> wrote:
\r
61 > Hi all, the goal here is to add support for --config=FILE option at the
\r
62 > notmuch top level (e.g. 'notmuch --config=FILE search foo'). In order to
\r
63 > achieve this neatly, I ended up moving config open/close to main() from
\r
64 > subcommands. This isn't a bad thing, because all notmuch commands opened
\r
65 > the config file anyway.
\r
67 Hi, Jani. I appreciate you've put a lot of work into this series, but
\r
68 I'll be the same devil's advocate that I was to David previously. Why
\r
69 do we need a command line option here when we already have an
\r
70 environment variable that handles just this? Is there some benefit to
\r
71 having a command line option for this that I don't see? I see this as
\r
72 another instance of an option that regular users will rarely use, if
\r
75 In general, I am a strong advocate of keeping the CLI slim. IMHO,
\r
76 adding more options makes the interface clunkier, and the manual harder
\r
77 to parse, and I'm against adding things that a normal user would likely
\r
78 never use. In retrospect, I should have had the same objection to the
\r
79 --format-version option, which I think could have just been an env var
\r
85 Content-Type: application/pgp-signature
\r
87 -----BEGIN PGP SIGNATURE-----
\r
88 Version: GnuPG v1.4.12 (GNU/Linux)
\r
90 iQIcBAEBCAAGBQJREpgCAAoJEO00zqvie6q82nIQAJ3+KgJwC9kXm10pMmcTS2jS
\r
91 tfKBMIeAYkFG/cAMEwzshiFtw0Nlr/+2NhitfOg8D+g9Tpa9luBr01IzdLSEA6ww
\r
92 YJXrWwzRBwMdUaNDibi1MWZdPuYDceqJBpLWXKsKx9B1BoM53HK2P9DG1m1NZ5kl
\r
93 7PotcbAHlD8L69oeym4DNkjHrNkib0c3LcbH0OqHosOXRqzRtxk9KzOp4cXCBJCU
\r
94 xsoSAKjcCZPwsV4hYVfRBYFnlSaJj5F6vdwAszMLef9fxk4s9q4HyqRHldG3ts06
\r
95 39ZQ+nKu0YQuLGRJGsev1+vticAjPhMkiD32BApyeFo57mfca/itRORwsNs1pBRn
\r
96 wqpQoXFQm8f/Smaj55rADdI9bCHFWUh+sm6BK/PQR7FqBlFUyH/CK6LZSG+426Sj
\r
97 Vvs4uUxVACyu+z4eNkL2aDV22yIhfDHm00P7vrms8zVb/UOxh5Y4qNNZJSh5chOX
\r
98 fuN5uANthFX5lHQaWZc/xxpb/Mthauqgz/PF+6DV80PuvGSf1cnAMt2WsQUCrBz7
\r
99 rOTyxlyh2UFiJDTp6ZqcA0ArOB4jzrdy43C1RsIWAyH76Yosy1Qz7PXaLb5h+a1n
\r
100 Z/klCPFCCf4iXK0qtqyBm2fJi4usAQHdYz49tfaGckcx4XZwg6L+57DG7F1KgyWc
\r
101 4xw7EoMgU+JRtPYJwlX6
\r
103 -----END PGP SIGNATURE-----
\r