Re: [PATCH v4 16/16] add "notmuch reindex" subcommand
[notmuch-archives.git] / 5f / d14638201905cf2ed5e768057d0cef2f725032
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 7B294431FBF\r
6         for <notmuch@notmuchmail.org>; Mon,  7 Apr 2014 02:22:36 -0700 (PDT)\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 VmphGEC6OU6H for <notmuch@notmuchmail.org>;\r
16         Mon,  7 Apr 2014 02:22:26 -0700 (PDT)\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 C513B431FB6\r
19         for <notmuch@notmuchmail.org>; Mon,  7 Apr 2014 02:22:25 -0700 (PDT)\r
20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
21         by guru.guru-group.fi (Postfix) with ESMTP id 4910D100086;\r
22         Mon,  7 Apr 2014 12:22:17 +0300 (EEST)\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 v4 2/3] emacs: add notmuch-version.el.tmpl and create\r
26         notmuch-version.el from it\r
27 In-Reply-To: <87a9by47wh.fsf@maritornes.cs.unb.ca>\r
28 References: <1395261431-24668-1-git-send-email-tomi.ollila@iki.fi>\r
29         <1395261431-24668-2-git-send-email-tomi.ollila@iki.fi>\r
30         <87a9by47wh.fsf@maritornes.cs.unb.ca>\r
31 User-Agent: Notmuch/0.17+174~gef82849 (http://notmuchmail.org) Emacs/24.3.1\r
32         (x86_64-unknown-linux-gnu)\r
33 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
34         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
35         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
36 Date: Mon, 07 Apr 2014 12:22:17 +0300\r
37 Message-ID: <m261mlbik6.fsf@guru.guru-group.fi>\r
38 MIME-Version: 1.0\r
39 Content-Type: text/plain\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: Mon, 07 Apr 2014 09:22:36 -0000\r
53 \r
54 On Sun, Apr 06 2014, David Bremner <david@tethera.net> wrote:\r
55 \r
56 > Tomi Ollila <tomi.ollila@iki.fi> writes:\r
57 >\r
58 >> +;; You should have received a copy of the GNU General Public License\r
59 >> +;; along with Notmuch.  If not, see <http://www.gnu.org/licenses/>.\r
60 >\r
61 > Do you think we need a copyright header in a generated file?\r
62 \r
63 This is more due to the template-to-file conversion simplicity than need\r
64 and I thought there is no harm having the copyright there. At least the\r
65 file is consistent with others in that sense.\r
66 \r
67 >> +\r
68 >> +(defconst notmuch-emacs-version %VERSION%\r
69 >> +  "Version string of this version of Notmuch Emacs MUA.")\r
70 >> +\r
71 >\r
72 > At the risk of being very picky, how about less copies of Version, maybe\r
73 > "Version of Notmuch Emacs MUA"\r
74 \r
75 Actually that is a good point:\r
76 \r
77 What I put there was modeled after `emacs-version':\r
78 "Version numbers of this version of Emacs."\r
79 \r
80 Then, gnus has (lisp/gnus/gnus.el in emacs sources):\r
81 \r
82 (defconst gnus-version-number "5.13"\r
83   "Version number for this version of Gnus.")\r
84 \r
85 (defconst gnus-version (format "Gnus v%s" gnus-version-number)\r
86   "Version string for this version of Gnus.")\r
87 \r
88 All of these are of type string, and the documentation states whether \r
89 it is (at least close to) numeric or not within string. But there is\r
90 no tradition of what *-version should contain.\r
91 \r
92 Therefore if we say "Version of Notmuch Emacs MUA", it can contain\r
93 "0.17+163~gcedea23" "0.18" or "unknown"\r
94 \r
95 \r
96 > Other than that, the series looks OK, and works on Debian now.\r
97 >\r
98 > d\r
99 \r
100 Tomi\r