46/0b3ec673d984c904fb7d3a497b492b5d92273b

   1 Return-Path: <cworth@cworth.org>\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 868834196F0\r
   6         for <notmuch@notmuchmail.org>; Sat,  3 Apr 2010 12:37:59 -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: -2.89\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01]\r
  13         autolearn=ham\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 swaBBBunp24C; Sat,  3 Apr 2010 12:37:58 -0700 (PDT)\r
  17 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 8C402431FC1;\r
  19         Sat,  3 Apr 2010 12:37:58 -0700 (PDT)\r
  20 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  21         id 3D7E8254390; Sat,  3 Apr 2010 12:37:58 -0700 (PDT)\r
  22 From: Carl Worth <cworth@cworth.org>\r
  23 To: David Edmondson <dme@dme.org>, notmuch <notmuch@notmuchmail.org>\r
  24 In-Reply-To: <87k4spxcz7.fsf@ut.hh.sledj.net>\r
  25 References: <87oci344n4.fsf@ut.hh.sledj.net>\r
  26         <87bpe2j2vu.fsf@yoom.home.cworth.org> <87oci2xmkv.fsf@ut.hh.sledj.net>\r
  27         <87d3yhihzl.fsf@yoom.home.cworth.org> <87k4spxcz7.fsf@ut.hh.sledj.net>\r
  28 Date: Sat, 03 Apr 2010 12:37:46 -0700\r
  29 Message-ID: <87tyrsz5rp.fsf@yoom.home.cworth.org>\r
  30 MIME-Version: 1.0\r
  31 Content-Type: multipart/signed; boundary="=-=-=";\r
  32         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  33 Subject: Re: [notmuch] pull request [was Re:  pull request]\r
  34 X-BeenThere: notmuch@notmuchmail.org\r
  35 X-Mailman-Version: 2.1.13\r
  36 Precedence: list\r
  37 List-Id: "Use and development of the notmuch mail system."\r
  38         <notmuch.notmuchmail.org>\r
  39 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  40         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  41 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  42 List-Post: <mailto:notmuch@notmuchmail.org>\r
  43 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  44 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  46 X-List-Received-Date: Sat, 03 Apr 2010 19:37:59 -0000\r
  47 \r
  48 --=-=-=\r
  49 Content-Transfer-Encoding: quoted-printable\r
  50 \r
  51 On Sat, 03 Apr 2010 07:32:44 +0100, David Edmondson <dme@dme.org> wrote:\r
  52 > * commit a9590dfb4efc2c05a35948ef4522c362eb788c10\r
  53 > | Author: David Edmondson <dme@dme.org>\r
  54 > | Date:   Thu Apr 1 11:38:30 2010 +0100\r
  55 > |=20\r
  56 > |     Makefile: Add the emacs directory to load-path when compiling\r
  57 \r
  58 That's a nice one-line summary of the commit that says "what" the patch\r
  59 does just fine. But the commit is missing the rest of the commit message\r
  60 that should give the "why".\r
  61 \r
  62 What's the motivation of the change? Is something perhaps broken without\r
  63 this? Or is this a preparation for something else that will be coming\r
  64 along in a future commit?\r
  65 \r
  66 From=20looking at the next commit, I assume this is to enable the\r
  67 "(require 'notmuch-show)" that is being added subsequently. So I've just\r
  68 noted that.\r
  69 \r
  70 > * commit 4de9f3f09e998d7312be2a1c08526e59bbf135a9\r
  71 > | Author: David Edmondson <dme@dme.org>\r
  72 > | Date:   Sun Mar 21 09:54:08 2010 +0000\r
  73 > |=20\r
  74 > |     emacs/Makefile.local: Use makefile mode\r
  75 \r
  76 I added similar treatment to the other instances of files named\r
  77 Makefile.local.\r
  78 \r
  79 > * commit 94893f25d36aaf43487e111fbfba4f7dae808dd2\r
  80 > | Author: David Edmondson <dme@dme.org>\r
  81 > | Date:   Tue Mar 23 07:04:34 2010 +0000\r
  82 > |=20\r
  83 > |     emacs/notmuch.el: Improve tag highlighting in search mode\r
  84 > |=20=20=20=20=20\r
  85 > |     Assume that tags never include an opening bracket, and hence improve\r
  86 > |     the regular expression used to highlight them. This avoids false\r
  87 > |     matches where the 'from' address of a thread participant includes an\r
  88 > |     opening bracket.\r
  89 \r
  90 Thanks. That's a good fix.\r
  91 \r
  92 The above are all pushed.\r
  93 \r
  94 > * commit f7ecad654fd8d0274fc75833d92117c8e496bcea\r
  95 > | Author: David Edmondson <dme@dme.org>\r
  96 > | Date:   Thu Apr 1 18:36:21 2010 +0100\r
  97 > |=20\r
  98 > |     emacs: Move notmuch-show functionality to notmuch-show.el\r
  99 > |=20=20=20=20=20\r
 100 > |     To ease the transition to a JSON based implementation of\r
 101 > |     `notmuch-show', move the current implementation into a separate fil=\r
 102 e.\r
 103 \r
 104 This is definitely a nice improvement in modularization. But there are\r
 105 some aspects of doing multiple-file emacs code that I'm unclear on.\r
 106 \r
 107 If I apply this patch as is, then when compiling the notmuch-show.el I\r
 108 get the following warnings:\r
 109 \r
 110   In notmuch-show:\r
 111   notmuch-show.el:969:34:Warning: reference to free variable `notmuch-comma=\r
 112 nd'\r
 113 \r
 114   In end of data:\r
 115   notmuch-show.el:983:1:Warning: the following functions are not known to be\r
 116       defined: point-invisible-p, mail-header-extract-no-properties,\r
 117       notmuch-select-tag-with-completion, union, intersection, set-differen=\r
 118 ce,\r
 119       notmuch-search-show-thread, mm-display-parts, mm-dissect-buffer,\r
 120       notmuch-save-attachments, notmuch-count-attachments, notmuch-reply,\r
 121       mm-handle-type, mm-display-part, notmuch-fontify-headers\r
 122 \r
 123 I can eliminate a few of these by copying the various require calls from\r
 124 notmuch.el to notmuch-show.el, but that still leaves problems for all of\r
 125 the functionality defined in notmuch.el and referenced in\r
 126 notmuch-show.el:\r
 127 \r
 128   In notmuch-show:\r
 129   notmuch-show.el:973:34:Warning: reference to free variable `notmuch-comma=\r
 130 nd'\r
 131 \r
 132   In end of data:\r
 133   notmuch-show.el:987:1:Warning: the following functions are not known to be\r
 134       defined: point-invisible-p, notmuch-select-tag-with-completion,\r
 135       notmuch-search-show-thread, notmuch-save-attachments,\r
 136       notmuch-count-attachments, notmuch-reply, notmuch-fontify-headers\r
 137 \r
 138 Does anyone know the right way to fix this? I'd like to get the output\r
 139 clean as I plan to move the compilation of the emacs code from "make\r
 140 install-emacs" to "make", (made conditional on a check for the presence\r
 141 of emacs by configure).\r
 142 \r
 143 So I haven't merged this commit yet.\r
 144 \r
 145 =2DCarl\r
 146 \r
 147 --=-=-=\r
 148 Content-Type: application/pgp-signature\r
 149 \r
 150 -----BEGIN PGP SIGNATURE-----\r
 151 Version: GnuPG v1.4.10 (GNU/Linux)\r
 152 \r
 153 iD8DBQFLt5kK6JDdNq8qSWgRAvqJAJ9h13lHmLZHLtByPYDht8lXBLpMbACfRGZn\r
 154 PKUWxSZ+S0Vxk2cDJbG3m/U=\r
 155 =NB8r\r
 156 -----END PGP SIGNATURE-----\r
 157 --=-=-=--\r