c0/5ce16800532767a791c604cfffb420c9daeefc

   1 Return-Path: <jrollins@finestructure.net>\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 4C252431FAF\r
   6         for <notmuch@notmuchmail.org>; Thu,  9 Aug 2012 00:55:04 -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.29\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.29 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3, 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 3VwNPTi3SDBp for <notmuch@notmuchmail.org>;\r
  16         Thu,  9 Aug 2012 00:55:03 -0700 (PDT)\r
  17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
  18         [131.215.239.19])\r
  19         by olra.theworths.org (Postfix) with ESMTP id C88F5431FAE\r
  20         for <notmuch@notmuchmail.org>; Thu,  9 Aug 2012 00:55:03 -0700 (PDT)\r
  21 Received: from earth-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by earth-doxen-postvirus (Postfix) with ESMTP id 60EC766E0123;\r
  23         Thu,  9 Aug 2012 00:55:01 -0700 (PDT)\r
  24 X-Spam-Scanned: at Caltech-IMSS on earth-doxen by amavisd-new\r
  25 Received: from finestructure.net (unknown [76.89.192.57])\r
  26         (Authenticated sender: jrollins)\r
  27         by earth-doxen-submit (Postfix) with ESMTP id 4C8E666E0121;\r
  28         Thu,  9 Aug 2012 00:54:58 -0700 (PDT)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id DD46A73E; Thu,  9 Aug 2012 00:54:57 -0700 (PDT)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: Austin Clements <amdragon@MIT.EDU>, notmuch@notmuchmail.org\r
  33 Subject: Re: [PATCH] emacs: Fix "not defined at runtime" warning\r
  34 In-Reply-To: <1344462010-5349-1-git-send-email-amdragon@mit.edu>\r
  35 References: <1344462010-5349-1-git-send-email-amdragon@mit.edu>\r
  36 User-Agent: Notmuch/0.13.2+137~ge34ed71 (http://notmuchmail.org) Emacs/23.4.1\r
  37         (x86_64-pc-linux-gnu)\r
  38 Date: Thu, 09 Aug 2012 00:54:55 -0700\r
  39 Message-ID: <87628sa4a8.fsf@servo.finestructure.net>\r
  40 MIME-Version: 1.0\r
  41 Content-Type: multipart/signed; boundary="=-=-=";\r
  42         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  43 X-BeenThere: notmuch@notmuchmail.org\r
  44 X-Mailman-Version: 2.1.13\r
  45 Precedence: list\r
  46 List-Id: "Use and development of the notmuch mail system."\r
  47         <notmuch.notmuchmail.org>\r
  48 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  50 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  51 List-Post: <mailto:notmuch@notmuchmail.org>\r
  52 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  53 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  54         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  55 X-List-Received-Date: Thu, 09 Aug 2012 07:55:04 -0000\r
  56 \r
  57 --=-=-=\r
  58 Content-Transfer-Encoding: quoted-printable\r
  59 \r
  60 On Wed, Aug 08 2012, Austin Clements <amdragon@MIT.EDU> wrote:\r
  61 > Previously, the Emacs byte compiler produced the warning\r
  62 >\r
  63 >     the function `remove-if-not' might not be defined at runtime.\r
  64 >\r
  65 > because we only required cl at compile-time (not runtime).  This fixes\r
  66 > this warning by requiring cl at runtime, ensuring that the definition\r
  67 > of remove-if-not is available.\r
  68 > ---\r
  69 >  emacs/notmuch-lib.el |    2 +-\r
  70 >  1 file changed, 1 insertion(+), 1 deletion(-)\r
  71 >\r
  72 > diff --git a/emacs/notmuch-lib.el b/emacs/notmuch-lib.el\r
  73 > index 30db58f..900235b 100644\r
  74 > --- a/emacs/notmuch-lib.el\r
  75 > +++ b/emacs/notmuch-lib.el\r
  76 > @@ -24,7 +24,7 @@\r
  77 >  (require 'mm-view)\r
  78 >  (require 'mm-decode)\r
  79 >  (require 'json)\r
  80 > -(eval-when-compile (require 'cl))\r
  81 > +(require 'cl)\r
  82 >=20=20\r
  83 >  (defvar notmuch-command "notmuch"\r
  84 >    "Command to run the notmuch binary.")\r
  85 \r
  86 LGTM.  I think it would be good to finally just give in a require cl, so\r
  87 we can finally put to rest the long standing schizophrenia we've had in\r
  88 regards to using it, and maybe finally kill the compile warnings that\r
  89 keep popping up.\r
  90 \r
  91 jamie.\r
  92 \r
  93 --=-=-=\r
  94 Content-Type: application/pgp-signature\r
  95 \r
  96 -----BEGIN PGP SIGNATURE-----\r
  97 Version: GnuPG v1.4.12 (GNU/Linux)\r
  98 \r
  99 iQIcBAEBCAAGBQJQI2zPAAoJEO00zqvie6q8WvsP/03xPiW6ed/n232244uU5LkZ\r
 100 K1+N0Iz7VvcwCp/3IL/gbYW7iQTg0hxLcfcRIBzVvY/eHM6ejB7FQjzzUSLqBL49\r
 101 qzepIvXuxC+EbIi5gINhNh51ATbV26243d09a+ulVyO7sBjyqUNA5yGCworjyLBX\r
 102 a5uE2cLZe6LbcbVTooXiZqOLo5o0rvzxJoxCCfk4qCm1v5DYnl7kkKa6zmqdESrx\r
 103 3vb+sVoU+VBVjIdeCmw6PzYXlM/Aq0MfWuNOxkk9XxNEDuJJeKHGJK99/A7yL/vP\r
 104 gsgxwionPySlpT2Kg7lx5VHNUP9tIAR/O2CWkxF937ku4j1PuT/tUl/LNjaqp8py\r
 105 Y7Td/KEGwaY41MxoWV8LiKFOghrkYCwwp2bc0LqzJbzGidpD56I0FAGZkKtkISsh\r
 106 T6lEYAk5V1yHMptkutNB7moVD/xGa8GO4V8wsE9EjNMoFtZZ9BZLQXszH//U2A2K\r
 107 7YMWF9+Ora94D5Tj09c//S8LLCnjWOLJRCm7N4Re4VLioZI43eW+98P19fvABQSQ\r
 108 7G0NjZygBnjwUrwOGxXRpQsEcDzg8OxFqGX/adC+Wx26EdA2supvRorK5k0lejRp\r
 109 IJ18fwBH/AZ8kE+sEcKN+OX91qGWro49ISF5aAdNgAoHW/xD0yqjpYjcDlG8O88e\r
 110 u/CAoEDeVFtUwUu9VLXh\r
 111 =ZVfl\r
 112 -----END PGP SIGNATURE-----\r
 113 --=-=-=--\r