5f/ad1f0b83238a792bb0cb05ad15963e2532b28a

   1 Return-Path: <david@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 4223E431FBD\r
   6         for <notmuch@notmuchmail.org>; Mon, 27 Jan 2014 17:56:40 -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 kf-OfOsfWP+Y for <notmuch@notmuchmail.org>;\r
  16         Mon, 27 Jan 2014 17:56:32 -0800 (PST)\r
  17 Received: from yantan.tethera.net (yantan.tethera.net [199.188.72.155])\r
  18         (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 57011431FBC\r
  21         for <notmuch@notmuchmail.org>; Mon, 27 Jan 2014 17:56:32 -0800 (PST)\r
  22 Received: from remotemail by yantan.tethera.net with local (Exim 4.80)\r
  23         (envelope-from <david@tethera.net>)\r
  24         id 1W7xub-00005m-17; Mon, 27 Jan 2014 21:56:25 -0400\r
  25 Received: (nullmailer pid 10238 invoked by uid 1000); Tue, 28 Jan 2014\r
  26         01:56:21 -0000\r
  27 From: David Bremner <david@tethera.net>\r
  28 To: Tomi Ollila <tomi.ollila@iki.fi>, notmuch@notmuchmail.org\r
  29 Subject: Re: [PATCH v3] build: write VERSION file containing $(VERSION) string\r
  30 In-Reply-To: <1390859290-10112-1-git-send-email-tomi.ollila@iki.fi>\r
  31 References: <1390858998-9846-1-git-send-email-tomi.ollila@iki.fi>\r
  32         <1390859290-10112-1-git-send-email-tomi.ollila@iki.fi>\r
  33 User-Agent: Notmuch/0.17+53~g3e1d7f6 (http://notmuchmail.org) Emacs/24.3.1\r
  34         (x86_64-pc-linux-gnu)\r
  35 Date: Mon, 27 Jan 2014 21:56:21 -0400\r
  36 Message-ID: <87zjmgkgxm.fsf@zancas.localnet>\r
  37 MIME-Version: 1.0\r
  38 Content-Type: text/plain\r
  39 Cc: tomi.ollila@iki.fi\r
  40 X-BeenThere: notmuch@notmuchmail.org\r
  41 X-Mailman-Version: 2.1.13\r
  42 Precedence: list\r
  43 List-Id: "Use and development of the notmuch mail system."\r
  44         <notmuch.notmuchmail.org>\r
  45 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  46         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  47 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  48 List-Post: <mailto:notmuch@notmuchmail.org>\r
  49 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  50 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  51         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  52 X-List-Received-Date: Tue, 28 Jan 2014 01:56:40 -0000\r
  53 \r
  54 Tomi Ollila <tomi.ollila@iki.fi> writes:\r
  55 \r
  56 > +VERSION:=$(shell gv=`exec git describe --match '[0-9.]*'|sed -e s/_/~/ -e s/-/+/ -e s/-/~/`; test -f VERSION || touch VERSION; read fv < VERSION; test x"$$fv" = x"$$gv" || echo "$$gv" > VERSION; echo "$$gv")\r
  57 \r
  58 does this need to be one line long? It's a bit scary.\r
  59 \r
  60 > +# this file may already have been updated.\r
  61 > +VERSION: version\r
  62 > +     echo $(VERSION) > version\r
  63 \r
  64 I'd prefer the build process does not change any version controlled\r
  65 file. It should be source, or generated, but not both. did  you mean to\r
  66 write to VERSION here?\r
  67 \r
  68 d\r
  69 \r