Re: [PATCH v4 01/16] add util/search-path.{c, h} to test for executables in $PATH
[notmuch-archives.git] / 5b / 237e2bb0c2d4c0f58b773de5e322f60913798c
1 Return-Path: <dkg@fifthhorseman.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 arlo.cworth.org (Postfix) with ESMTP id 396C06DE0B39\r
6  for <notmuch@notmuchmail.org>; Fri, 12 Aug 2016 12:25:50 -0700 (PDT)\r
7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
8 X-Spam-Flag: NO\r
9 X-Spam-Score: -0.051\r
10 X-Spam-Level: \r
11 X-Spam-Status: No, score=-0.051 tagged_above=-999 required=5\r
12  tests=[AWL=-0.051] autolearn=disabled\r
13 Received: from arlo.cworth.org ([127.0.0.1])\r
14  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
15  with ESMTP id jJHyLM2WiwXq for <notmuch@notmuchmail.org>;\r
16  Fri, 12 Aug 2016 12:25:42 -0700 (PDT)\r
17 Received: from che.mayfirst.org (che.mayfirst.org [162.247.75.118])\r
18  by arlo.cworth.org (Postfix) with ESMTP id CF44B6DE0B12\r
19  for <notmuch@notmuchmail.org>; Fri, 12 Aug 2016 12:25:41 -0700 (PDT)\r
20 Received: from fifthhorseman.net (unknown [38.109.115.130])\r
21  by che.mayfirst.org (Postfix) with ESMTPSA id 95514F98B;\r
22  Fri, 12 Aug 2016 15:25:39 -0400 (EDT)\r
23 Received: by fifthhorseman.net (Postfix, from userid 1000)\r
24  id D046820248; Fri, 12 Aug 2016 14:46:38 -0400 (EDT)\r
25 From: Daniel Kahn Gillmor <dkg@fifthhorseman.net>\r
26 To: David Bremner <david@tethera.net>, Notmuch Mail <notmuch@notmuchmail.org>\r
27 Subject: Re: [PATCH v4 01/16] add util/search-path.{c,\r
28  h} to test for executables in $PATH\r
29 In-Reply-To: <87y442phbm.fsf@maritornes.cs.unb.ca>\r
30 References: <1467970047-8013-1-git-send-email-dkg@fifthhorseman.net>\r
31  <1467970047-8013-2-git-send-email-dkg@fifthhorseman.net>\r
32  <878tw2r0vf.fsf@maritornes.cs.unb.ca>\r
33  <87k2fmse5h.fsf@alice.fifthhorseman.net>\r
34  <87y442phbm.fsf@maritornes.cs.unb.ca>\r
35 User-Agent: Notmuch/0.22.1+88~g8d09e96 (https://notmuchmail.org) Emacs/24.5.1\r
36  (x86_64-pc-linux-gnu)\r
37 Date: Fri, 12 Aug 2016 14:46:38 -0400\r
38 Message-ID: <87mvkhrfjl.fsf@alice.fifthhorseman.net>\r
39 MIME-Version: 1.0\r
40 Content-Type: text/plain\r
41 X-BeenThere: notmuch@notmuchmail.org\r
42 X-Mailman-Version: 2.1.20\r
43 Precedence: list\r
44 List-Id: "Use and development of the notmuch mail system."\r
45  <notmuch.notmuchmail.org>\r
46 List-Unsubscribe: <https://notmuchmail.org/mailman/options/notmuch>,\r
47  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
48 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
49 List-Post: <mailto:notmuch@notmuchmail.org>\r
50 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
51 List-Subscribe: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
52  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
53 X-List-Received-Date: Fri, 12 Aug 2016 19:25:50 -0000\r
54 \r
55 On Fri 2016-08-12 03:38:53 -0400, David Bremner wrote:\r
56 > Daniel Kahn Gillmor <dkg@fifthhorseman.net> writes:\r
57 >\r
58 >>> Should we distinguish between relative and absolute paths here?  I can't\r
59 >>> think of any security implications, but I'm wondering if a relative path\r
60 >>> is likely just a user error.\r
61 >>\r
62 >> I don't think a relative path is necessarily a user error.  I certainly\r
63 >> use relative paths myself from time to time.\r
64 >\r
65 > As configuration values? That seems quite fragile, since it depends on\r
66 > the current working directory when notmuch is run.\r
67 \r
68 rarely!  but sometimes i do it because i'm testing things in strange\r
69 ways, and it can be a bit frustrating to have a tool second-guess me\r
70 when it seems like i ought to be able to drop the same string i'm using\r
71 on the command line into the configuration.\r
72 \r
73 I don't feel strongly, though.  if you want to say that bare words found\r
74 in the $PATH and absolute filenames (starting with /) are fine in the\r
75 notmuch config but relative paths are not, i'd be ok with that :)\r
76 \r
77         --dkg\r