Re: [PATCH 1/2] Add Google Inc. to AUTHORS as a contributor.
[notmuch-archives.git] / c5 / d1d6954c793732c255b7f23f5cec8d6291a5dd
1 Return-Path: <dmitry.kurochkin@gmail.com>\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 B83B2429E41\r
6         for <notmuch@notmuchmail.org>; Mon, 27 Jun 2011 21:44:58 -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: -0.799\r
10 X-Spam-Level: \r
11 X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
13         FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
14 Received: from olra.theworths.org ([127.0.0.1])\r
15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
16         with ESMTP id gzkRqktszTlv for <notmuch@notmuchmail.org>;\r
17         Mon, 27 Jun 2011 21:44:58 -0700 (PDT)\r
18 Received: from mail-bw0-f53.google.com (mail-bw0-f53.google.com\r
19         [209.85.214.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
20         (No client certificate requested)\r
21         by olra.theworths.org (Postfix) with ESMTPS id B9B33429E32\r
22         for <notmuch@notmuchmail.org>; Mon, 27 Jun 2011 21:44:57 -0700 (PDT)\r
23 Received: by bwg12 with SMTP id 12so4889410bwg.26\r
24         for <notmuch@notmuchmail.org>; Mon, 27 Jun 2011 21:44:56 -0700 (PDT)\r
25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma;\r
26         h=domainkey-signature:from:to:cc:subject:in-reply-to:references\r
27         :user-agent:date:message-id:mime-version:content-type;\r
28         bh=5dzt1K6xFewaNwgKecUVdMCADHen/A9PhxqP3oZXJrE=;\r
29         b=FNw+JreqkorzWqDNP2DZYeOoDL1IpaKmuHlvJWMhWox+NRDl1H2WCU4LY0Ofxt8r72\r
30         zlpge2v366BiB6p+xVfDS/Y+fJ2y3UMhQeZu6bEM/rzbS7K7UFBDiQs18FKo3vud/E75\r
31         5pWc7qfyOuxMG2+soIODUHSeyRIIw4ILW75lU=\r
32 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma;\r
33         h=from:to:cc:subject:in-reply-to:references:user-agent:date\r
34         :message-id:mime-version:content-type;\r
35         b=Yqjv18Ki0sxUsawYuJGvYjsj4p+/MQ2URT1kICemrV9RcaDqKEkOvaf7PeCuvDOs1I\r
36         PaAd8sGNZxmL8nDS8DX5OuCDJekUydAb+iuqlcKQ4hgXPomkzCFBAhlyZVYfqp6HoywU\r
37         Q/cktAqyVH+abHpdH8wvboKpSFHjxP1pdXLdQ=\r
38 Received: by 10.204.83.33 with SMTP id d33mr4893503bkl.55.1309236296160;\r
39         Mon, 27 Jun 2011 21:44:56 -0700 (PDT)\r
40 Received: from localhost ([91.144.186.21])\r
41         by mx.google.com with ESMTPS id n3sm4741982bka.16.2011.06.27.21.44.54\r
42         (version=TLSv1/SSLv3 cipher=OTHER);\r
43         Mon, 27 Jun 2011 21:44:55 -0700 (PDT)\r
44 From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>\r
45 To: Austin Clements <amdragon@MIT.EDU>\r
46 Subject: Re: [PATCH 2/2] test: use emacsclient(1) for Emacs tests\r
47 In-Reply-To: <20110628041742.GC4120@mit.edu>\r
48 References: <1309132379-24089-1-git-send-email-dmitry.kurochkin@gmail.com>\r
49         <1309146846-30991-1-git-send-email-dmitry.kurochkin@gmail.com>\r
50         <1309146846-30991-2-git-send-email-dmitry.kurochkin@gmail.com>\r
51         <BANLkTi=nwHRndz367K5oh4=tPaU-KNeZ8Q@mail.gmail.com>\r
52         <87hb7byqge.fsf@gmail.com> <87d3hyzs1c.fsf@gmail.com>\r
53         <20110628034937.GB4120@mit.edu> <87aad2zjuz.fsf@gmail.com>\r
54         <20110628041742.GC4120@mit.edu>\r
55 User-Agent: Notmuch/0.5-275-g75c95e5 (http://notmuchmail.org) Emacs/23.3.1\r
56         (x86_64-pc-linux-gnu)\r
57 Date: Tue, 28 Jun 2011 08:44:51 +0400\r
58 Message-ID: <877h86zhrw.fsf@gmail.com>\r
59 MIME-Version: 1.0\r
60 Content-Type: text/plain; charset=us-ascii\r
61 Cc: notmuch@notmuchmail.org\r
62 X-BeenThere: notmuch@notmuchmail.org\r
63 X-Mailman-Version: 2.1.13\r
64 Precedence: list\r
65 List-Id: "Use and development of the notmuch mail system."\r
66         <notmuch.notmuchmail.org>\r
67 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
68         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
69 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
70 List-Post: <mailto:notmuch@notmuchmail.org>\r
71 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
72 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
73         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
74 X-List-Received-Date: Tue, 28 Jun 2011 04:44:58 -0000\r
75 \r
76 On Tue, 28 Jun 2011 00:17:42 -0400, Austin Clements <amdragon@MIT.EDU> wrote:\r
77 > Quoth Dmitry Kurochkin on Jun 28 at  7:59 am:\r
78 > > I am sure that would work, but I do not like the complexity.  How about\r
79 > > getting back to standard emacsclient and running a watchdog in the\r
80 > > emacs?  Like:\r
81 > > \r
82 > > (defun orphan-watchdog (pid)\r
83 > >   "Periodically check that the process with id PID is still\r
84 > > running, quit if it terminated."\r
85 > >   (if (not (process-attributes pid))\r
86 > >       (kill-emacs)\r
87 > >     (run-at-time "1 min" nil orphan-watchdog pid)))\r
88 > > \r
89 > > This function (or my other changes) do not work (by yet unknown reason\r
90 > > :)), but you get the idea.\r
91\r
92 > I would consider this more complex than a few file descriptors. ]:--8)\r
93 \r
94 More shell code and more elisp code.  I do not think can be considered\r
95 simpler :)\r
96 \r
97 > Though, I'm automatically distrustful of anything that relies on\r
98 > polling (why poll when you can be notified instantly?).\r
99\r
100 \r
101 I agree that polling is not as elegant as an instant notification.  But\r
102 IMO reinventing emacsclient just kills all the beauty of this solution.\r
103 I liked it when it was a simple read loop, but now it is too complex to\r
104 my taste.  Besides, I am a bit worried that we will face new problems in\r
105 the future that would force us to add more "features" to our\r
106 not-so-poor-man's Emacs server implementation.\r
107 \r
108 > It also has some problems.  For example, PID's are easily reused, so\r
109 > if another process happens to take up that PID, the emacs could still\r
110 > hang around for a long time.\r
111 \r
112 Indeed.  We may add a more complex process detection, but IMO it is an\r
113 overkill.\r
114 \r
115 Anyway, I am done with reworking the patch series and will post it now.\r
116 Perhaps others would voice their opinion on this one.\r
117 \r
118 Regards,\r
119   Dmitry\r