Re: [PATCH v4 01/16] add util/search-path.{c, h} to test for executables in $PATH
[notmuch-archives.git] / 38 / 1f983ad9cc8af0fe90712336adc0eb0e9f8ac0
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
8 X-Spam-Flag: NO\r
9 X-Spam-Score: -2.3\r
10 X-Spam-Level: \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
18         [131.215.239.19])\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
40 MIME-Version: 1.0\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
45 Precedence: list\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
56 \r
57 --=-=-=\r
58 Content-Type: text/plain\r
59 \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
66 \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
73 ever.\r
74 \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
80 as well.\r
81 \r
82 jamie.\r
83 \r
84 --=-=-=\r
85 Content-Type: application/pgp-signature\r
86 \r
87 -----BEGIN PGP SIGNATURE-----\r
88 Version: GnuPG v1.4.12 (GNU/Linux)\r
89 \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
102 =AH8n\r
103 -----END PGP SIGNATURE-----\r
104 --=-=-=--\r