d1/4652a230e91a33f8ef5c89c44a785bfb041973

   1 Return-Path: <bremner@unb.ca>\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 E73D9431FAF\r
   6         for <notmuch@notmuchmail.org>; Sun,  2 Sep 2012 06:54:29 -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\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         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 BVoDaKNVnHuE for <notmuch@notmuchmail.org>;\r
  16         Sun,  2 Sep 2012 06:54:28 -0700 (PDT)\r
  17 Received: from tesseract.cs.unb.ca (tesseract.cs.unb.ca [131.202.240.238])\r
  18         (using TLSv1 with cipher AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id C5DED431FB6\r
  21         for <notmuch@notmuchmail.org>; Sun,  2 Sep 2012 06:54:28 -0700 (PDT)\r
  22 Received: from fctnnbsc30w-156034089108.dhcp-dynamic.fibreop.nb.bellaliant.net\r
  23         ([156.34.89.108] helo=zancas.localnet)\r
  24         by tesseract.cs.unb.ca with esmtpsa\r
  25         (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.72)\r
  26         (envelope-from <bremner@unb.ca>)\r
  27         id 1T8Ad1-00085J-Hh; Sun, 02 Sep 2012 10:54:23 -0300\r
  28 Received: from bremner by zancas.localnet with local (Exim 4.80)\r
  29         (envelope-from <bremner@unb.ca>)\r
  30         id 1T8Acw-0005YK-29; Sun, 02 Sep 2012 10:54:14 -0300\r
  31 From: David Bremner <david@tethera.net>\r
  32 To: Tomi Ollila <tomi.ollila@iki.fi>,\r
  33  Mark Walters <markwalters1009@gmail.com>,      notmuch@notmuchmail.org\r
  34 Subject: Re: [PATCH 1/5] emacs: compile and load notmuch-pick.el if present.\r
  35 In-Reply-To: <m2ipdcqjwj.fsf@guru.guru-group.fi>\r
  36 References: <1343164911-31589-1-git-send-email-markwalters1009@gmail.com>\r
  37         <1343164911-31589-2-git-send-email-markwalters1009@gmail.com>\r
  38         <m2ipdcqjwj.fsf@guru.guru-group.fi>\r
  39 User-Agent: Notmuch/0.14 (http://notmuchmail.org) Emacs/24.1.1\r
  40         (x86_64-pc-linux-gnu)\r
  41 Date: Sun, 02 Sep 2012 10:54:13 -0300\r
  42 Message-ID: <87y5ks7coq.fsf@zancas.localnet>\r
  43 MIME-Version: 1.0\r
  44 Content-Type: text/plain\r
  45 X-Spam_bar: -\r
  46 X-BeenThere: notmuch@notmuchmail.org\r
  47 X-Mailman-Version: 2.1.13\r
  48 Precedence: list\r
  49 List-Id: "Use and development of the notmuch mail system."\r
  50         <notmuch.notmuchmail.org>\r
  51 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  52         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  53 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  54 List-Post: <mailto:notmuch@notmuchmail.org>\r
  55 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  56 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  57         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  58 X-List-Received-Date: Sun, 02 Sep 2012 13:54:30 -0000\r
  59 \r
  60 Tomi Ollila <tomi.ollila@iki.fi> writes:\r
  61 > I think it would be better to provide a shell script in \r
  62 > notmuch-pick directory which byte-compiles and installs notmuch-pick\r
  63 > in case user wants to install/update notmuch-pick. Whenever notmuch-pick\r
  64 > is good enough to be shipped inside $(dir) above then the aboce conditional\r
  65 > is not needed (at all).\r
  66 \r
  67 I just tried notmuch-pick for the first time, and at least on this\r
  68 relatively beefy machine, it runs fine uncompiled. \r
  69 \r
  70 I'm not sure if it is worth complicating the mainline install. Which\r
  71 means I guess I am agreeing with Tomi.  It could also be in independant\r
  72 Makefile rather than a script.\r
  73 \r
  74 I don't think it's crucial to install notmuch-pick into the main lisp\r
  75 context of (Debian) packaging I don't think that will work very well.\r
  76 The way org-mode works is that it ships a seperate contrib lisp\r
  77 directory; on Debian this is in /usr/share/org-mode/contrib. Then people\r
  78 who want to use a contrib piece of lisp would just add the following to\r
  79 their .emacs:\r
  80 \r
  81 (add-to-list 'load-path "/usr/share/notmuch/contrib/emacs-lisp")\r
  82 (require 'notmuch-pick)\r
  83 \r
  84 at least given the changes that Tomi proposes below.\r
  85 The path is obviously just an example.\r
  86 \r
  87 > (require 'notmuch-hello)\r
  88 > (require 'notmuch-show)\r
  89 > (require 'notmuch) ;; XXX ATM, as notmuch-search-mode-map is defined here\r
  90 >\r
  91 > And, at the end, before (provide 'notmuch-pick), execute the lines\r
  92 > what currenty are contained in (notmuch-pick-init).\r
  93 \r
  94 Yes, this sounds about right to me. require should also take care of the\r
  95 "only run this once" part.\r
  96 \r
  97 FWIW, I'm currently running pick from the source tree as:\r
  98 \r
  99 (add-to-list 'load-path\r
 100              (expand-file-name "~/blah"))\r
 101 (when (locate-library "notmuch-pick")\r
 102   (eval-after-load 'notmuch\r
 103     '(progn \r
 104        (require 'notmuch-pick)\r
 105        (notmuch-pick-init))))\r
 106 \r
 107 it's a bit overengineered, but this way it doesn't crap out on machines\r
 108 where notmuch-pick is not installed.\r