24/b3af8fe16573addb37d0205cda30ecdeba4a9b

   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 24242431FD0\r
   6         for <notmuch@notmuchmail.org>; Fri, 24 Jun 2011 10:52:01 -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.01\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.01 tagged_above=-999 required=5\r
  12         tests=[T_MIME_NO_TEXT=0.01] 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 R-r6bLRvcb+C for <notmuch@notmuchmail.org>;\r
  16         Fri, 24 Jun 2011 10:52:00 -0700 (PDT)\r
  17 Received: from arlo.cworth.org (arlo.cworth.org [50.43.72.2])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 5F6E8431FB6\r
  19         for <notmuch@notmuchmail.org>; Fri, 24 Jun 2011 10:52:00 -0700 (PDT)\r
  20 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  21         by arlo.cworth.org (Postfix) with ESMTP id E4CC029A608;\r
  22         Fri, 24 Jun 2011 10:51:58 -0700 (PDT)\r
  23 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  24         id D0705254157; Fri, 24 Jun 2011 10:51:58 -0700 (PDT)\r
  25 From: Carl Worth <cworth@cworth.org>\r
  26 To: david@tethera.net, notmuch@notmuchmail.org\r
  27 Subject: Re: [PATCH] libnotmuch: fix typo in CLEAN setting, add file\r
  28 In-Reply-To: <1308912824-1709-1-git-send-email-david@tethera.net>\r
  29 References: <1308912824-1709-1-git-send-email-david@tethera.net>\r
  30 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.3.1\r
  31         (i486-pc-linux-gnu)\r
  32 Date: Fri, 24 Jun 2011 10:51:51 -0700\r
  33 Message-ID: <87aad7ce2w.fsf@yoom.home.cworth.org>\r
  34 MIME-Version: 1.0\r
  35 Content-Type: multipart/signed; boundary="=-=-=";\r
  36         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  37 Cc: David Bremner <bremner@debian.org>\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: Fri, 24 Jun 2011 17:52:01 -0000\r
  51 \r
  52 --=-=-=\r
  53 Content-Transfer-Encoding: quoted-printable\r
  54 \r
  55 On Fri, 24 Jun 2011 07:53:44 -0300, david@tethera.net wrote:\r
  56 > From: David Bremner <bremner@debian.org>\r
  57 \r
  58 Hi David,\r
  59 \r
  60 Thanks for the fix. A couple of nit-picky comments on the commit itself\r
  61 first:\r
  62 \r
  63 > - c0961e6 introduced a missing slash and\r
  64 > - cdf1c70a created a file and neglected to add it to clean\r
  65 \r
  66 I'd put a little more detail in the commit message here:\r
  67 \r
  68 =2D c0961e6 introduced a missing slash $(dir)$(LIBNAME)\r
  69 =2D cdf1c70a created $(dir)/notmuch.h.gch and neglected to add it to clean\r
  70 \r
  71 > The former seems to have been harmless, so maybe someone (Carl?) can\r
  72 > check if $(dir)/$(LIBNAME) really needs to be removed?\r
  73 \r
  74 This auxiliary text should be below the "---" line in the email so that\r
  75 it won't be in the commit message after I do "git am". (As is, I'd have\r
  76 to do extra work to "git commit --amend" this text away).\r
  77 \r
  78 As for the actual question, yes, $(dir)/$(LIBNAME) is supposed to be\r
  79 cleaned and is not getting cleaned. I've got libnotmuch.so.1.{1,2,3,4}.0\r
  80 all sitting around never cleaned.\r
  81 \r
  82 > -CLEAN :=3D $(CLEAN) $(libnotmuch_modules) $(dir)/$(SONAME) $(dir)/$(LINK=\r
  83 ER_NAME) $(dir)$(LIBNAME) libnotmuch.a notmuch.aux notmuch.sym\r
  84 > +CLEAN +=3D $(libnotmuch_modules) $(dir)/$(SONAME) $(dir)/$(LINKER_NAME)\r
  85 > +CLEAN +=3D $(dir)/$(LIBNAME) libnotmuch.a notmuch.aux notmuch.sym=20\r
  86 \r
  87 Meanwhile, I notice now that "libnotmuch.a" is also wrong, (should be\r
  88 "$(dir)/libnotmuch.a).\r
  89 \r
  90 Clearly we could use some testing of our "make clean" target to ensure\r
  91 it actually works. Does the Debian stuff not test anything like this?\r
  92 (Maybe I'm thinking of typical testing done by autoconf/automake's "make\r
  93 distcheck" that we haven't replicated yet.)\r
  94 \r
  95 =2DCarl\r
  96 \r
  97 =2D-=20\r
  98 carl.d.worth@intel.com\r
  99 \r
 100 --=-=-=\r
 101 Content-Type: application/pgp-signature\r
 102 \r
 103 -----BEGIN PGP SIGNATURE-----\r
 104 Version: GnuPG v1.4.11 (GNU/Linux)\r
 105 \r
 106 iEYEARECAAYFAk4EzrcACgkQ6JDdNq8qSWgK7ACfU/seIsV4PHvqjyhyxk0jB3AM\r
 107 75sAnjKkjX+Ccn+Z0Nydp0LaMcziXh2N\r
 108 =Vmdn\r
 109 -----END PGP SIGNATURE-----\r
 110 --=-=-=--\r