08/daf1038658bd9d37cc60eb6786ae6a96e90233

   1 Return-Path: <m.walters@qmul.ac.uk>\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 1E712431FB6\r
   6         for <notmuch@notmuchmail.org>; Sat, 22 Feb 2014 00:49:34 -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: -1.098\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-1.098 tagged_above=-999 required=5\r
  12         tests=[DKIM_ADSP_CUSTOM_MED=0.001, FREEMAIL_FROM=0.001,\r
  13         NML_ADSP_CUSTOM_MED=1.2, RCVD_IN_DNSWL_MED=-2.3] autolearn=disabled\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id ZyI7eCiADLr4 for <notmuch@notmuchmail.org>;\r
  17         Sat, 22 Feb 2014 00:49:26 -0800 (PST)\r
  18 Received: from mail2.qmul.ac.uk (mail2.qmul.ac.uk [138.37.6.6])\r
  19         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id D56D3431FBD\r
  22         for <notmuch@notmuchmail.org>; Sat, 22 Feb 2014 00:49:25 -0800 (PST)\r
  23 Received: from smtp.qmul.ac.uk ([138.37.6.40])\r
  24         by mail2.qmul.ac.uk with esmtp (Exim 4.71)\r
  25         (envelope-from <m.walters@qmul.ac.uk>)\r
  26         id 1WH8Gf-0004yy-Jp; Sat, 22 Feb 2014 08:49:21 +0000\r
  27 Received: from 93-97-24-31.zone5.bethere.co.uk ([93.97.24.31] helo=localhost)\r
  28         by smtp.qmul.ac.uk with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.71)\r
  29         (envelope-from <m.walters@qmul.ac.uk>)\r
  30         id 1WH8Gf-0006vR-2x; Sat, 22 Feb 2014 08:49:05 +0000\r
  31 From: Mark Walters <markwalters1009@gmail.com>\r
  32 To: Tomi Ollila <tomi.ollila@iki.fi>, notmuch@notmuchmail.org\r
  33 Subject: Re: [PATCH 1/3] build: write version.stamp file containing $(VERSION)\r
  34         string\r
  35 In-Reply-To: <1393024837-30394-1-git-send-email-tomi.ollila@iki.fi>\r
  36 References: <1393024837-30394-1-git-send-email-tomi.ollila@iki.fi>\r
  37 User-Agent: Notmuch/0.15.2+484~gfb59956 (http://notmuchmail.org) Emacs/23.4.1\r
  38         (x86_64-pc-linux-gnu)\r
  39 Date: Sat, 22 Feb 2014 08:49:04 +0000\r
  40 Message-ID: <8761o7mt1b.fsf@qmul.ac.uk>\r
  41 MIME-Version: 1.0\r
  42 Content-Type: text/plain; charset=us-ascii\r
  43 X-Sender-Host-Address: 93.97.24.31\r
  44 X-QM-Geographic: According to ripencc,\r
  45         this message was delivered by a machine in Britain (UK) (GB).\r
  46 X-QM-SPAM-Info: Sender has good ham record.  :)\r
  47 X-QM-Body-MD5: 69668abbfd8a8890ebf51833d97cd925 (of first 20000 bytes)\r
  48 X-SpamAssassin-Score: 0.0\r
  49 X-SpamAssassin-SpamBar: /\r
  50 X-SpamAssassin-Report: The QM spam filters have analysed this message to\r
  51         determine if it is\r
  52         spam. We require at least 5.0 points to mark a message as spam.\r
  53         This message scored 0.0 points. Summary of the scoring: \r
  54         * 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\r
  55         provider *      (markwalters1009[at]gmail.com)\r
  56         *  0.0 AWL AWL: From: address is in the auto white-list\r
  57 X-QM-Scan-Virus: ClamAV says the message is clean\r
  58 Cc: tomi.ollila@iki.fi\r
  59 X-BeenThere: notmuch@notmuchmail.org\r
  60 X-Mailman-Version: 2.1.13\r
  61 Precedence: list\r
  62 List-Id: "Use and development of the notmuch mail system."\r
  63         <notmuch.notmuchmail.org>\r
  64 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  65         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  66 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  67 List-Post: <mailto:notmuch@notmuchmail.org>\r
  68 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  69 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  70         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  71 X-List-Received-Date: Sat, 22 Feb 2014 08:49:34 -0000\r
  72 \r
  73 \r
  74 Patches 2 and 3 look fine +1.\r
  75 \r
  76 I don't know any Make and haven't really being following the discussion\r
  77 between Austin and Tomi so can't comment on Patch 1. It all seems to\r
  78 work well in my testing.\r
  79 \r
  80 Best wishes\r
  81 \r
  82 Mark\r
  83 \r
  84 \r
  85 \r
  86 On Fri, 21 Feb 2014, Tomi Ollila <tomi.ollila@iki.fi> wrote:\r
  87 > This version file will be as prerequisite to the target files\r
  88 > that use the version info for some purpose, like printing\r
  89 > it for the user to examine. The contents of the version.stamp\r
  90 > file is seldom read by the build system itself as the $(VERSION)\r
  91 > variable has the same information.\r
  92 >\r
  93 > Thanks to Trevor, David and Mark for their contributions.\r
  94 > ---\r
  95 >\r
  96 > This is version 6 of\r
  97 >\r
  98 >      id:1391956711-21231-1-git-send-email-tomi.ollila@iki.fi\r
  99 >\r
 100 >   The only change is $(VERSTAMP) -> version.stamp -- $(VERSTAMP) does not\r
 101 >   get expanded for prerequisities if those are not already defined,\r
 102 >   and as emacs/Makefile.local is included before Makefile.local $(VERSTAMP)\r
 103 >   is useless in emacs/... (expands to nothing). And my original idea for\r
 104 >   $(VERSTAMP) was for typo-checking -- I just did not realize that make does\r
 105 >   not tell me that the variable is undefined so the typo-check I thought of\r
 106 >   just did't happen.\r
 107 >\r
 108 >\r
 109 > And the following 2 patches version 2 of\r
 110 >\r
 111 >         id:1390135903-28167-1-git-send-email-tomi.ollila@iki.fi\r
 112 >\r
 113 >   Now utilizes version.stamp to know when notmuch.el is to be\r
 114 >   re-bytecompiled.\r
 115 >\r
 116 >  Makefile.local | 14 +++++++++++++-\r
 117 >  1 file changed, 13 insertions(+), 1 deletion(-)\r
 118 >\r
 119 > diff --git a/Makefile.local b/Makefile.local\r
 120 > index 174506c..3a56c06 100644\r
 121 > --- a/Makefile.local\r
 122 > +++ b/Makefile.local\r
 123 > @@ -22,6 +22,11 @@ VERSION:=$(shell cat ${srcdir}/version)\r
 124 >  ifeq ($(filter release release-message pre-release update-versions,$(MAKECMDGOALS)),)\r
 125 >  ifeq ($(IS_GIT),yes)\r
 126 >  VERSION:=$(shell git describe --match '[0-9.]*'|sed -e s/_/~/ -e s/-/+/ -e s/-/~/)\r
 127 > +# Write the file 'version.stamp' in case its contents differ from $(VERSION)\r
 128 > +FILE_VERSION:=$(shell test -f version.stamp && read vs < version.stamp || vs=; echo $$vs)\r
 129 > +ifneq ($(FILE_VERSION),$(VERSION))\r
 130 > +       $(shell echo "$(VERSION)" > version.stamp)\r
 131 > +endif\r
 132 >  endif\r
 133 >  endif\r
 134 >  \r
 135 > @@ -69,6 +74,11 @@ ifeq ($(shell cat .first-build-message 2>/dev/null),)\r
 136 >  endif\r
 137 >  endif\r
 138 >  \r
 139 > +# Depend (also) on the file 'version'. In case of ifeq ($(IS_GIT),yes)\r
 140 > +# this file may already have been updated.\r
 141 > +version.stamp: version\r
 142 > +     echo $(VERSION) > $@\r
 143 > +\r
 144 >  $(TAR_FILE):\r
 145 >       if git tag -v $(VERSION) >/dev/null 2>&1; then \\r
 146 >             ref=$(VERSION); \\r
 147 > @@ -280,6 +290,8 @@ notmuch_client_srcs =             \\r
 148 >  \r
 149 >  notmuch_client_modules = $(notmuch_client_srcs:.c=.o)\r
 150 >  \r
 151 > +notmuch.o: version.stamp\r
 152 > +\r
 153 >  notmuch: $(notmuch_client_modules) lib/libnotmuch.a util/libutil.a parse-time-string/libparse-time-string.a\r
 154 >       $(call quiet,CXX $(CFLAGS)) $^ $(FINAL_LIBNOTMUCH_LDFLAGS) -o $@\r
 155 >  \r
 156 > @@ -318,7 +330,7 @@ install-desktop:\r
 157 >       desktop-file-install --mode 0644 --dir "$(DESTDIR)$(desktop_dir)" notmuch.desktop\r
 158 >  \r
 159 >  SRCS  := $(SRCS) $(notmuch_client_srcs)\r
 160 > -CLEAN := $(CLEAN) notmuch notmuch-shared $(notmuch_client_modules) notmuch.elc\r
 161 > +CLEAN := $(CLEAN) notmuch notmuch-shared $(notmuch_client_modules) version.stamp\r
 162 >  \r
 163 >  DISTCLEAN := $(DISTCLEAN) .first-build-message Makefile.config\r
 164 >  \r
 165 > -- \r
 166 > 1.8.0\r
 167 >\r
 168 > _______________________________________________\r
 169 > notmuch mailing list\r
 170 > notmuch@notmuchmail.org\r
 171 > http://notmuchmail.org/mailman/listinfo/notmuch\r