7b/3003034d01077f1e71836ac0d4dc4e2cc60584

   1 Return-Path: <bremner@tethera.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 BEDAF431FD0\r
   6         for <notmuch@notmuchmail.org>; Sat, 17 Dec 2011 19:17:07 -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: -2.3\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.3 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3] 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 59jHbzbZz8ci for <notmuch@notmuchmail.org>;\r
  16         Sat, 17 Dec 2011 19:17:07 -0800 (PST)\r
  17 Received: from tempo.its.unb.ca (tempo.its.unb.ca [131.202.1.21])\r
  18         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 2F6F9431FB6\r
  21         for <notmuch@notmuchmail.org>; Sat, 17 Dec 2011 19:17:07 -0800 (PST)\r
  22 Received: from zancas.localnet\r
  23         (fctnnbsc36w-156034079193.pppoe-dynamic.High-Speed.nb.bellaliant.net\r
  24         [156.34.79.193]) (authenticated bits=0)\r
  25         by tempo.its.unb.ca (8.13.8/8.13.8) with ESMTP id pBI3GuC0027762\r
  26         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NO);\r
  27         Sat, 17 Dec 2011 23:17:02 -0400\r
  28 Received: from bremner by zancas.localnet with local (Exim 4.77)\r
  29         (envelope-from <bremner@tethera.net>)\r
  30         id 1Rc7FA-00080L-7i; Sat, 17 Dec 2011 23:16:56 -0400\r
  31 From: David Bremner <david@tethera.net>\r
  32 To: notmuch@notmuchmail.org\r
  33 Subject: [PATCH] build-system: use a shell variable for TMPFILE in\r
  34         debian-snapshot\r
  35 Date: Sat, 17 Dec 2011 23:16:51 -0400\r
  36 Message-Id: <1324178211-30739-1-git-send-email-david@tethera.net>\r
  37 X-Mailer: git-send-email 1.7.7.3\r
  38 In-Reply-To: <1324136185-4509-1-git-send-email-aaronecay@gmail.com>\r
  39 References: <1324136185-4509-1-git-send-email-aaronecay@gmail.com>\r
  40 Cc: David Bremner <bremner@debian.org>\r
  41 X-BeenThere: notmuch@notmuchmail.org\r
  42 X-Mailman-Version: 2.1.13\r
  43 Precedence: list\r
  44 List-Id: "Use and development of the notmuch mail system."\r
  45         <notmuch.notmuchmail.org>\r
  46 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  47         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  48 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  49 List-Post: <mailto:notmuch@notmuchmail.org>\r
  50 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  51 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  52         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  53 X-List-Received-Date: Sun, 18 Dec 2011 03:17:08 -0000\r
  54 \r
  55 From: David Bremner <bremner@debian.org>\r
  56 \r
  57 Aaron Ecay points out in\r
  58 id:"1324136185-4509-1-git-send-email-aaronecay@gmail.com" that the\r
  59 mktemp in\r
  60 \r
  61      debian-snapshot: TMPFILE := $(shell mktemp)\r
  62 \r
  63 Is being evaluated for every target. As best I can tell, this is\r
  64 because make is evaluating the right hand side, even though it is not\r
  65 doing the assignment.\r
  66 \r
  67 Of course, it isn't quite as nice to edit with the line continuations,\r
  68 but it is ideomatic make.\r
  69 ---\r
  70  Makefile.local |   15 ++++++++-------\r
  71  1 files changed, 8 insertions(+), 7 deletions(-)\r
  72 \r
  73 diff --git a/Makefile.local b/Makefile.local\r
  74 index 5108a0c..97f397f 100644\r
  75 --- a/Makefile.local\r
  76 +++ b/Makefile.local\r
  77 @@ -139,15 +139,16 @@ pre-release:\r
  78         mv $(TAR_FILE) $(DEB_TAR_FILE) releases\r
  79  \r
  80  .PHONY: debian-snapshot\r
  81 -debian-snapshot: TMPFILE := $(shell mktemp)\r
  82  debian-snapshot:\r
  83         make VERSION=$(VERSION) clean\r
  84 -       cp debian/changelog $(TMPFILE)\r
  85 -       EDITOR=/bin/true dch -b -v $(VERSION)+1 -D UNRELEASED 'test build, not for upload'\r
  86 -       echo '3.0 (native)' > debian/source/format\r
  87 -       debuild -us -uc\r
  88 -       mv -f $(TMPFILE) debian/changelog\r
  89 -       echo '3.0 (quilt)' > debian/source/format\r
  90 +       TMPFILE=$$(mktemp /tmp/notmuch.XXXXXX);         \\r
  91 +         cp debian/changelog $${TMPFILE};              \\r
  92 +         EDITOR=/bin/true dch -b -v $(VERSION)+1       \\r
  93 +           -D UNRELEASED 'test build, not for upload'; \\r
  94 +         echo '3.0 (native)' > debian/source/format;   \\r
  95 +         debuild -us -uc;                              \\r
  96 +         mv -f $${TMPFILE} debian/changelog;           \\r
  97 +         echo '3.0 (quilt)' > debian/source/format\r
  98  \r
  99  .PHONY: release-message\r
 100  release-message:\r
 101 -- \r
 102 1.7.7.3\r
 103 \r