a8/18631c06218cc02f5a57ff5735a54b0b06b19b

   1 Return-Path: <tomi.ollila@iki.fi>\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 D03D8431FD7\r
   6         for <notmuch@notmuchmail.org>; Wed,  4 Dec 2013 11:27:45 -0800 (PST)\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 rYKgK2s4g5Oy for <notmuch@notmuchmail.org>;\r
  16         Wed,  4 Dec 2013 11:27:36 -0800 (PST)\r
  17 Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 9099D431FD0\r
  19         for <notmuch@notmuchmail.org>; Wed,  4 Dec 2013 11:27:36 -0800 (PST)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21         by guru.guru-group.fi (Postfix) with ESMTP id 406E3100086;\r
  22         Wed,  4 Dec 2013 21:27:30 +0200 (EET)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: David Bremner <david@tethera.net>, notmuch@notmuchmail.org\r
  25 Subject: Re: [PATCH 1/1] emacs: Makefile.local: HAVE_EMACS usage fixes\r
  26 In-Reply-To: <87ob523snl.fsf@zancas.localnet>\r
  27 References: <1377630047-27756-1-git-send-email-tomi.ollila@iki.fi>\r
  28         <87ob523snl.fsf@zancas.localnet>\r
  29 User-Agent: Notmuch/0.17~rc1+17~ga2e1a2f (http://notmuchmail.org) Emacs/24.3.1\r
  30         (x86_64-unknown-linux-gnu)\r
  31 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  32         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  33         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  34 Date: Wed, 04 Dec 2013 21:27:29 +0200\r
  35 Message-ID: <m2mwkgmn6m.fsf@guru.guru-group.fi>\r
  36 MIME-Version: 1.0\r
  37 Content-Type: text/plain\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: Wed, 04 Dec 2013 19:27:46 -0000\r
  51 \r
  52 On Sat, Nov 30 2013, David Bremner <david@tethera.net> wrote:\r
  53 \r
  54 > Tomi Ollila <tomi.ollila@iki.fi> writes:\r
  55 >\r
  56 >>  \r
  57 >>  ifeq ($(WITH_EMACS),1)\r
  58 >>  ifeq ($(HAVE_EMACS),1)\r
  59 >>  all: $(emacs_bytecode)\r
  60 >> +install-emacs: $(emacs_bytecode)\r
  61 >>  endif\r
  62 >>  \r
  63 >\r
  64 > It's a bit confusing that we have both HAVE_EMACS and WITH_EMACS. Should\r
  65 > this be documented somewhere? Otherwise the patch seems sensible.\r
  66 >\r
  67 > See also the discussion at id:528B455A.70200@iki.fi.\r
  68 \r
  69 Eventually it's probably good idea to merge HAVE_EMACS to WITH_EMACS but\r
  70 doing in a good way requires quite a few ./configure changes (and while\r
  71 doing that one have to check that some consistency is kept).\r
  72 Eventually it would probably be nice that in case *_EMACS is not set (is 0)\r
  73 the clean rules stay (provided that those do not require emacs) but\r
  74 build rules are ifdeffed out...\r
  75 \r
  76 In the meantime my patch here maintains status quo where things can be\r
  77 done in case HAVE_EMACS is set (1) -- emacs is available in the system,\r
  78 while keeping the cleanup rules in case *_EMACS is not set... and\r
  79 this patch also fixes the compilation whenever there is no emacs in system.\r
  80 \r
  81 I could add short notes of what HAVE_EMACS & WITH_EMACS mean but although\r
  82 it is bit confusing that we currently have both, those are nevertheless\r
  83 quite descriptive themselves.\r
  84 \r
  85 >\r
  86 > d\r
  87 \r
  88 Tomi\r
  89 \r
  90 > _______________________________________________\r
  91 > notmuch mailing list\r
  92 > notmuch@notmuchmail.org\r
  93 > http://notmuchmail.org/mailman/listinfo/notmuch\r