Re: [PATCH v2 1/7] cli: allow query to come from stdin
[notmuch-archives.git] / 20 / 37df366cf034b9a8fbe30fd39dbb91671e54b9
1 Return-Path: <ccx@webprojekty.cz>\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 D372B431FAF\r
6         for <notmuch@notmuchmail.org>; Tue, 17 Jul 2012 03:18:15 -0700 (PDT)\r
7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
8 X-Spam-Flag: NO\r
9 X-Spam-Score: 1.274\r
10 X-Spam-Level: *\r
11 X-Spam-Status: No, score=1.274 tagged_above=-999 required=5\r
12         tests=[RDNS_NONE=1.274] 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 KtrnN6SfUTiB for <notmuch@notmuchmail.org>;\r
16         Tue, 17 Jul 2012 03:18:12 -0700 (PDT)\r
17 Received: from new.webprojekty.cz (unknown [178.238.36.176])\r
18         by olra.theworths.org (Postfix) with ESMTP id 477A3431FAE\r
19         for <notmuch@notmuchmail.org>; Tue, 17 Jul 2012 03:18:12 -0700 (PDT)\r
20 Received: from shell.webprojekty.cz (shell.vs [192.168.100.30])\r
21         by new.webprojekty.cz (Postfix) with SMTP id B73AEB8C5E2;\r
22         Tue, 17 Jul 2012 12:18:08 +0200 (CEST)\r
23 Received: by shell.webprojekty.cz (sSMTP sendmail emulation);\r
24         Tue, 17 Jul 2012 12:18:08 +0200\r
25 Date: Tue, 17 Jul 2012 12:18:08 +0200\r
26 From: ccx@webprojekty.cz\r
27 To: notmuch@notmuchmail.org\r
28 Subject: Re: Notmuch scripts (again), now with more usenet\r
29 Message-ID: <20120717101808.GA30479@dorje.inet.te2000>\r
30 References: <20120714050156.GA11178@dorje.inet.te2000>\r
31         <87ipdp9e1l.fsf@maritornes.cs.unb.ca>\r
32 MIME-Version: 1.0\r
33 Content-Type: multipart/signed; micalg=pgp-sha1;\r
34         protocol="application/pgp-signature"; boundary="wRRV7LY7NUeQGEoC"\r
35 Content-Disposition: inline\r
36 In-Reply-To: <87ipdp9e1l.fsf@maritornes.cs.unb.ca>\r
37 User-Agent: Mutt/1.5.21 (2010-09-15)\r
38 X-BeenThere: notmuch@notmuchmail.org\r
39 X-Mailman-Version: 2.1.13\r
40 Precedence: list\r
41 List-Id: "Use and development of the notmuch mail system."\r
42         <notmuch.notmuchmail.org>\r
43 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
44         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
45 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
46 List-Post: <mailto:notmuch@notmuchmail.org>\r
47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
48 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
49         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
50 X-List-Received-Date: Tue, 17 Jul 2012 10:18:16 -0000\r
51 \r
52 \r
53 --wRRV7LY7NUeQGEoC\r
54 Content-Type: text/plain; charset=us-ascii\r
55 Content-Disposition: inline\r
56 Content-Transfer-Encoding: quoted-printable\r
57 \r
58 On Sun, Jul 15, 2012 at 10:36:54AM -0600, David Bremner wrote:\r
59 > ccx@webprojekty.cz writes:\r
60 >=20\r
61 > > I hope it's now in the form acceptable for inclusion to contrib.\r
62 >=20\r
63 > Dear Jan;\r
64 >=20\r
65 > Without looking too much at the details I think this is reasonable to\r
66 > include in contrib, as a set of examples if nothing else. Still, I'm\r
67 > curious about a few things. Are there actual users (other than the\r
68 > author)?  What would we do about bug reports? Will you follow the list,\r
69 > or would we explicitely tell people to contact you directly?\r
70 \r
71 I might not always have time to read the entirety of the ML, so it would\r
72 be great if people CC me when posting there.\r
73 \r
74 > I guess some of the python experts might want to discuss the coding\r
75 > style of notmuch-new.py. pylint goes ballistic, for whatever that is\r
76 > worth.\r
77 \r
78 That's mainly because of tabs used for indentation, which used to be\r
79 considered a viable alternative, but is now considered obsolete. I\r
80 reformatted the code to conform to pep8 guidelines.\r
81 \r
82 > For the shell stuff, I noticed lots of unquoted parameter expansion;\r
83 > also at some point you make a temporary directory using $$\r
84 > directly rather than using e.g. mktemp.\r
85 \r
86 Zsh does not word-split parameter expansion by default, this is\r
87 intentional. The temporary directories are created in xdg cache dir,\r
88 which should not be susceptible to symlink attacks etc. and the pid\r
89 actually helps to correlate the directories to processes, eg. when you\r
90 want to autoclean after searches that got killed for some reason.\r
91 \r
92 > What did you have in mind for a license? since there is no linking here,\r
93 > it does not have to be GPL3+, but some standard Free license is needed I\r
94 > think; since I'm also maintaining the Debian package, I don't want to\r
95 > have to remove your stuff before uploading to Debian.\r
96 \r
97 You can consider it public domain. I guess I should add wtfpl text to my\r
98 repo. :-)\r
99 \r
100 > d\r
101 \r
102 Thanks very much for your rewiev.\r
103 \r
104 --wRRV7LY7NUeQGEoC\r
105 Content-Type: application/pgp-signature; name="signature.asc"\r
106 Content-Description: Digital signature\r
107 \r
108 -----BEGIN PGP SIGNATURE-----\r
109 Version: GnuPG v2.0.19 (GNU/Linux)\r
110 \r
111 iEYEARECAAYFAlAFO+AACgkQq2Q0xnrW+pnsKACfcM66QoShwYevk+JmkRkmaDNH\r
112 UmgAnAhtwyxr2mA7DcYtqdB32NtyWxDd\r
113 =ke0R\r
114 -----END PGP SIGNATURE-----\r
115 \r
116 --wRRV7LY7NUeQGEoC--\r