d5/af8e437963122ef51ef2a29d2f8b0dd6af3385

   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 1B51D431FBC\r
   6         for <notmuch@notmuchmail.org>; Tue,  6 Nov 2012 08:51: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 sieaIR7mnlKH for <notmuch@notmuchmail.org>;\r
  16         Tue,  6 Nov 2012 08:51:44 -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 45BF2431FB6\r
  19         for <notmuch@notmuchmail.org>; Tue,  6 Nov 2012 08:51:44 -0800 (PST)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21         by guru.guru-group.fi (Postfix) with ESMTP id 8ADB5100094;\r
  22         Tue,  6 Nov 2012 18:51:43 +0200 (EET)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: Blake Jones <blakej@foo.net>, notmuch@notmuchmail.org\r
  25 Subject: Re: [PATCH v2 05/10] install: check for non-SysV version (Solaris\r
  26         support)\r
  27 In-Reply-To: <1352142123-18286-6-git-send-email-blakej@foo.net>\r
  28 References: <1352142123-18286-1-git-send-email-blakej@foo.net>\r
  29         <1352142123-18286-6-git-send-email-blakej@foo.net>\r
  30 User-Agent: Notmuch/0.14+81~gb8371cd (http://notmuchmail.org) Emacs/24.2.1\r
  31         (x86_64-unknown-linux-gnu)\r
  32 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  33         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  34         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  35 Date: Tue, 06 Nov 2012 18:51:43 +0200\r
  36 Message-ID: <m2vcdiznc0.fsf@guru.guru-group.fi>\r
  37 MIME-Version: 1.0\r
  38 Content-Type: text/plain\r
  39 X-BeenThere: notmuch@notmuchmail.org\r
  40 X-Mailman-Version: 2.1.13\r
  41 Precedence: list\r
  42 List-Id: "Use and development of the notmuch mail system."\r
  43         <notmuch.notmuchmail.org>\r
  44 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  46 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  47 List-Post: <mailto:notmuch@notmuchmail.org>\r
  48 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  49 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  51 X-List-Received-Date: Tue, 06 Nov 2012 16:51:45 -0000\r
  52 \r
  53 On Mon, Nov 05 2012, Blake Jones <blakej@foo.net> wrote:\r
  54 \r
  55 Some quick comments, haven't got time to test yet.\r
  56 \r
  57 // stuff deleted //\r
  58 \r
  59 > diff --git a/vim/Makefile b/vim/Makefile\r
  60 > index f17bebf..7ceba7a 100644\r
  61 > --- a/vim/Makefile\r
  62 > +++ b/vim/Makefile\r
  63 > @@ -5,8 +5,6 @@ files = plugin/notmuch.vim \\r
  64 >  prefix = $(HOME)/.vim\r
  65 >  destdir = $(prefix)/plugin\r
  66 >  \r
  67 > -INSTALL = install -D -m644\r
  68 > -\r
  69 >  all: help\r
  70 >  \r
  71 >  help:\r
  72 > @@ -17,7 +15,7 @@ help:\r
  73 >       @echo "    make symlink     - create symlinks in ~/.vim (useful for development)"\r
  74 >  \r
  75 >  install:\r
  76 > -     @for x in $(files); do $(INSTALL) $(CURDIR)/$$x $(prefix)/$$x; done\r
  77 > +     @for x in $(files); do $(INSTALL) -D -m644 $(CURDIR)/$$x $(prefix)/$$x; done\r
  78 >  \r
  79 > -link symlink: INSTALL = ln -fs\r
  80 >  link symlink: install\r
  81 > +     @for x in $(files); do ln -fs $(CURDIR)/$$x $(prefix)/$$x; done\r
  82 > -- \r
  83 \r
  84 Here you'd need to remove the 'install' dependency as it would first\r
  85 do it and then overwriting the results with dependency..\r
  86 \r
  87 another option is to use other variable like \r
  88 \r
  89 __INSTALL = $(INSTALL) -D -m644\r
  90 \r
  91 and then replace other uses of INSTALL with __INSTALL\r
  92 \r
  93 (this doesn't look too good but that was what I could come\r
  94 quickly)\r
  95 \r
  96 > 1.7.3.2\r
  97 \r
  98 Tomi\r