[PATCH 1/2] fix compilation without emacs
[notmuch-archives.git] / 8c / 90e91e0ae9244c91e5c02c0c5d31436ee36c47
1 Return-Path: <teythoon@jade-hamburg.de>\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 38A0C431FBC\r
6         for <notmuch@notmuchmail.org>; Wed, 24 Apr 2013 02:42:19 -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 vnmOE3wvtIL3 for <notmuch@notmuchmail.org>;\r
16         Wed, 24 Apr 2013 02:42:18 -0700 (PDT)\r
17 Received: from mail.cryptobitch.de (cryptobitch.de [88.198.7.68])\r
18         (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits))\r
19         (No client certificate requested)\r
20         by olra.theworths.org (Postfix) with ESMTPS id 5F155431FB6\r
21         for <notmuch@notmuchmail.org>; Wed, 24 Apr 2013 02:42:18 -0700 (PDT)\r
22 Received: from mail.jade-hamburg.de (mail.jade-hamburg.de [85.183.11.228])\r
23         (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits))\r
24         (No client certificate requested)\r
25         by mail.cryptobitch.de (Postfix) with ESMTPSA id 57D8E605F2B\r
26         for <notmuch@notmuchmail.org>; Wed, 24 Apr 2013 11:42:09 +0200 (CEST)\r
27 Received: by mail.jade-hamburg.de (Postfix, from userid 401)\r
28         id B2F08DF2A5; Wed, 24 Apr 2013 11:42:08 +0200 (CEST)\r
29 Received: from thinkbox.jade-hamburg.de (thinkbox.jadE-Hamburg.de\r
30  [10.1.1.109])  (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))     (No\r
31  client certificate requested) (Authenticated sender: teythoon) by\r
32  mail.jade-hamburg.de (Postfix) with ESMTPSA id A96DDDF29F;     Wed, 24 Apr 2013\r
33  11:42:06 +0200 (CEST)\r
34 Received: from teythoon by thinkbox.jade-hamburg.de with local (Exim 4.80)\r
35         (envelope-from <teythoon@thinkbox.jade-hamburg.de>)\r
36         id 1UUwCE-00073a-Uh; Wed, 24 Apr 2013 11:41:02 +0200\r
37 Content-Type: text/plain; charset="utf-8"\r
38 MIME-Version: 1.0\r
39 Content-Transfer-Encoding: quoted-printable\r
40 From: Justus Winter <4winter@informatik.uni-hamburg.de>\r
41 User-Agent: alot/0.3.3+\r
42 To: David Bremner <david@tethera.net>,  notmuch@notmuchmail.org\r
43 References:\r
44  <1366729987-16925-1-git-send-email-4winter@informatik.uni-hamburg.de>\r
45         <87d2tkoo80.fsf@zancas.localnet>\r
46 In-Reply-To: <87d2tkoo80.fsf@zancas.localnet>\r
47 Message-ID: <20130424094102.7753.16880@thinkbox.jade-hamburg.de>\r
48 Subject: Re: [PATCH] go: use a different goconfig package\r
49 Date: Wed, 24 Apr 2013 11:41:02 +0200\r
50 X-BeenThere: notmuch@notmuchmail.org\r
51 X-Mailman-Version: 2.1.13\r
52 Precedence: list\r
53 List-Id: "Use and development of the notmuch mail system."\r
54         <notmuch.notmuchmail.org>\r
55 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
56         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
57 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
58 List-Post: <mailto:notmuch@notmuchmail.org>\r
59 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
60 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
61         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
62 X-List-Received-Date: Wed, 24 Apr 2013 09:42:19 -0000\r
63 \r
64 Quoting David Bremner (2013-04-24 00:51:11)\r
65 > Justus Winter <4winter@informatik.uni-hamburg.de> writes:\r
66 > =\r
67 \r
68 > > The notmuch-addrlookup utility uses a third party library to read the\r
69 > > notmuch configuration file. The previously used implementation at\r
70 > > "github.com/kless/goconfig" vanished, so this patch switches to the\r
71 > > implementation at "github.com/msbranco/goconfig". As the\r
72 > > implementations differ at the API level, the code is updated\r
73 > > accordingly.\r
74 > =\r
75 \r
76 > It's great that you fixed this, but the whole idea of something failing\r
77 > to build depending on the status of a website makes me pretty\r
78 > uncomfortable.\r
79 > =\r
80 \r
81 > d\r
82 \r
83 Ah, I pondered whether I should write more about the lib change. Maybe\r
84 I should have.\r
85 \r
86 So there are two libs, kless and msbranco for short. kless is a fork\r
87 of msbranco, and it seems like the author abandoned his fork. So we\r
88 are now using the original lib, msbranco. msbranco seems to be in\r
89 "maintenance" mode, but has recently received updates to fix some\r
90 minor issues wrt. go 1.1. There are a lot of forks of msbranco on\r
91 github, so even if msbranco vanishes as well, we can pick the next\r
92 most promising fork.\r
93 \r
94 So yeah, I feel the same way, it's unfortunately that I had to patch\r
95 this, but we're better of now (it builds again and we moved to a more\r
96 "vanilla" goconfig API (assuming the other forks do not change the api\r
97 as much as kless did)).\r
98 \r
99 And I think you are being a bit harsh with the "status of the website"\r
100 comment. In case you are not familiar with go,\r
101 "github.com/kless/goconfig" is a package path, not a website. It just\r
102 happens to also be the host+resource part of a URL that points to the\r
103 github project page. So what happened here is that the VCS repository\r
104 specified by the package path vanished, not some web site (well, that\r
105 vanished too, but that didn't cause the build failure). If you happen\r
106 to have golang installed, type "go help remote" for more information.\r
107 \r
108 Cheers,\r
109 Justus\r