a9/b60ebc77e647be1a75d890c866a478f9a5b3fc

   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 C8B81431FB6\r
   6         for <notmuch@notmuchmail.org>; Tue, 17 Apr 2012 02:33:53 -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 sNsCmSPJGD4b for <notmuch@notmuchmail.org>;\r
  17         Tue, 17 Apr 2012 02:33:50 -0700 (PDT)\r
  18 Received: from mail-lb0-f181.google.com (mail-lb0-f181.google.com\r
  19         [209.85.217.181]) (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 BAA4A431FAE\r
  22         for <notmuch@notmuchmail.org>; Tue, 17 Apr 2012 02:33:49 -0700 (PDT)\r
  23 Received: by lbdb5 with SMTP id b5so1776964lbd.26\r
  24         for <notmuch@notmuchmail.org>; Tue, 17 Apr 2012 02:33:48 -0700 (PDT)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113;\r
  26         h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
  27         :mime-version:content-type;\r
  28         bh=MXEupgwvwdp0h8hrPQI3T77WxadoY5RIl1op0ccB7eg=;\r
  29         b=qWYROo/t/qzlPALDjDJzZG3hPdpfl4oRtZnaecINB0ABwQWkezmsiGVKOOJSy1XYKh\r
  30         Qv+S/nDaS4JxUToJK6krUSz7vNrVwQulgDET9aDRp8mGJjFagpuAFxuKxKzrqU2sAeNT\r
  31         iF5lyahXuT8lCQ/vo1dHmgkhpsLEI18bv3Ss4n80k7mmwfgFP8jcZVSCJGllnyNSd5GU\r
  32         qpFgSRN2Z1n+LjqckZVOqtbgqfcCOw++2VGWkuJWnFY7awdPR1xx9aJqo6ldorSHEovu\r
  33         1ncM9fDiYbWNDXrlWd5r/Er9PRRTAKNHygJ50ARn58ziwn3NwpG6QOGXF78BXV4c9cTz\r
  34         Whfw==\r
  35 Received: by 10.152.103.239 with SMTP id fz15mr13370977lab.42.1334655228113;\r
  36         Tue, 17 Apr 2012 02:33:48 -0700 (PDT)\r
  37 Received: from localhost ([91.144.186.21])\r
  38         by mx.google.com with ESMTPS id ay4sm28181169lbb.2.2012.04.17.02.33.46\r
  39         (version=TLSv1/SSLv3 cipher=OTHER);\r
  40         Tue, 17 Apr 2012 02:33:47 -0700 (PDT)\r
  41 From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>\r
  42 To: Jani Nikula <jani@nikula.org>, notmuch@notmuchmail.org\r
  43 Subject: Re: [RFC PATCH 2/4] emacs: add no-display arg to\r
  44         notmuch-hello-refresh-hook\r
  45 In-Reply-To: <87hawivh73.fsf@nikula.org>\r
  46 References:\r
  47  <bcfdc0d1969997e89e5abe0b320d77ee2109796a.1334651669.git.jani@nikula.org>\r
  48         <21f129a3426c58d44fed7150b32c4fec7021bda0.1334651669.git.jani@nikula.org>\r
  49         <877gxeohh6.fsf@gmail.com> <87hawivh73.fsf@nikula.org>\r
  50 User-Agent: Notmuch/0.12+104~gf9ad237 (http://notmuchmail.org) Emacs/23.4.1\r
  51         (x86_64-pc-linux-gnu)\r
  52 Date: Tue, 17 Apr 2012 13:31:57 +0400\r
  53 Message-ID: <87y5pun1pu.fsf@gmail.com>\r
  54 MIME-Version: 1.0\r
  55 Content-Type: text/plain; charset=us-ascii\r
  56 X-BeenThere: notmuch@notmuchmail.org\r
  57 X-Mailman-Version: 2.1.13\r
  58 Precedence: list\r
  59 List-Id: "Use and development of the notmuch mail system."\r
  60         <notmuch.notmuchmail.org>\r
  61 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  62         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  63 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  64 List-Post: <mailto:notmuch@notmuchmail.org>\r
  65 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  66 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  67         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  68 X-List-Received-Date: Tue, 17 Apr 2012 09:33:53 -0000\r
  69 \r
  70 Jani Nikula <jani@nikula.org> writes:\r
  71 \r
  72 > On Tue, 17 Apr 2012 13:06:13 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:\r
  73 >> Can you please give some explanaition why is this needed?  Would this\r
  74 >> change break custom hooks that do not expect any arguments?\r
  75 >\r
  76 > For patch 3/4. We don't want to display a message if someone calls\r
  77 > (notmuch-hello-update t) from some script, and notmuch-hello buffer\r
  78 > isn't even visible.\r
  79 >\r
  80 > Yes, it would break custom hooks. And a bunch of tests. But I think it\r
  81 > would be useful for custom hooks too, for the same reason as above.\r
  82 >\r
  83 \r
  84 Makes sense.  I think it should be mentioned in the commit message.\r
  85 Especially the fact that it breaks existing hooks.  We should mention it\r
  86 in the NEWS later as well.\r
  87 \r
  88 Regards,\r
  89   Dmitry\r
  90 \r
  91 > Jani.\r
  92 >\r
  93 >\r
  94 >\r
  95 >> \r
  96 >> Regards,\r
  97 >>   Dmitry\r
  98 >> \r
  99 >> Jani Nikula <jani@nikula.org> writes:\r
 100 >> \r
 101 >> > ---\r
 102 >> >  emacs/notmuch-hello.el |    2 +-\r
 103 >> >  1 files changed, 1 insertions(+), 1 deletions(-)\r
 104 >> >\r
 105 >> > diff --git a/emacs/notmuch-hello.el b/emacs/notmuch-hello.el\r
 106 >> > index 9cd907a..0596bbe 100644\r
 107 >> > --- a/emacs/notmuch-hello.el\r
 108 >> > +++ b/emacs/notmuch-hello.el\r
 109 >> > @@ -776,7 +776,7 @@ following:\r
 110 >> >      (widget-setup)\r
 111 >> >  \r
 112 >> >      (goto-char final-target-pos))\r
 113 >> > -  (run-hooks 'notmuch-hello-refresh-hook)\r
 114 >> > +  (run-hook-with-args 'notmuch-hello-refresh-hook no-display)\r
 115 >> >    (setq notmuch-hello-first-run nil))\r
 116 >> >  \r
 117 >> >  (defun notmuch-folder ()\r
 118 >> > -- \r
 119 >> > 1.7.1\r
 120 >> >\r
 121 >> > _______________________________________________\r
 122 >> > notmuch mailing list\r
 123 >> > notmuch@notmuchmail.org\r
 124 >> > http://notmuchmail.org/mailman/listinfo/notmuch\r