ab/d5a1d89b3ead4652f7d78636913c383be47acc

   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 7AEF5431FAF\r
   6         for <notmuch@notmuchmail.org>; Mon, 30 Apr 2012 12:54:30 -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 fUYWAnmNCNz4 for <notmuch@notmuchmail.org>;\r
  16         Mon, 30 Apr 2012 12:54:29 -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 20419431FAE\r
  21         for <notmuch@notmuchmail.org>; Mon, 30 Apr 2012 12:54:29 -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 B811B584BAB\r
  26         for <notmuch@notmuchmail.org>; Mon, 30 Apr 2012 21:54:24 +0200 (CEST)\r
  27 Received: by mail.jade-hamburg.de (Postfix, from userid 401)\r
  28         id 13941DF2A5; Mon, 30 Apr 2012 21:54:24 +0200 (CEST)\r
  29 Received: from thinkbox.jade-hamburg.de (unknown [10.1.1.153])\r
  30         (using TLSv1 with cipher AES256-SHA (256/256 bits))\r
  31         (No client certificate requested) (Authenticated sender: teythoon)\r
  32         by mail.jade-hamburg.de (Postfix) with ESMTPSA id 7767FDF2A1;\r
  33         Mon, 30 Apr 2012 21:54:22 +0200 (CEST)\r
  34 Received: from teythoon by thinkbox.jade-hamburg.de with local (Exim 4.77)\r
  35         (envelope-from <teythoon@thinkbox.jade-hamburg.de>)\r
  36         id 1SOwfs-0003vx-FK; Mon, 30 Apr 2012 21:54:20 +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 To: David Bremner <david@tethera.net>\r
  41 From: Justus Winter <4winter@informatik.uni-hamburg.de>\r
  42 In-Reply-To: <87lildph9w.fsf@zancas.localnet>\r
  43 References: <1335649518-1970-1-git-send-email-amdragon@mit.edu>\r
  44         <87lildph9w.fsf@zancas.localnet>\r
  45 Message-ID: <20120430195420.2528.48612@thinkbox.jade-hamburg.de>\r
  46 User-Agent: alot/0.3+\r
  47 Subject: Re: [PATCH] go: Update to the current notmuch_database_find_message\r
  48         API\r
  49 Date: Mon, 30 Apr 2012 21:54:20 +0200\r
  50 Cc: , notmuch mailing list <notmuch@notmuchmail.org>\r
  51 X-BeenThere: notmuch@notmuchmail.org\r
  52 X-Mailman-Version: 2.1.13\r
  53 Precedence: list\r
  54 List-Id: "Use and development of the notmuch mail system."\r
  55         <notmuch.notmuchmail.org>\r
  56 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  57         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  58 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  59 List-Post: <mailto:notmuch@notmuchmail.org>\r
  60 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  61 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  62         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  63 X-List-Received-Date: Mon, 30 Apr 2012 19:54:30 -0000\r
  64 \r
  65 Quoting David Bremner (2012-04-30 13:53:47)\r
  66 > Austin Clements <amdragon@MIT.EDU> writes:\r
  67 > =\r
  68 \r
  69 > > The signature of notmuch_database_find_message was changed in 02a30767\r
  70 > > to report errors and the Go bindings were never updated.  This brings\r
  71 > > the Go bindings in sync with that change and at least makes them\r
  72 > > compile with Go r60.3, the last release before Go 1.\r
  73 > =\r
  74 \r
  75 > I don't have any easy way to test this, since the current versions in =\r
  76 \r
  77 > Debian are all based on Go 1.  On the other hand I guess it doesn't make\r
  78 > things worse.  Anybody have any objections to this patch?\r
  79 \r
  80 lgtm.\r
  81 \r
  82 > In the long run we have to decide what to do about these bindings. Seb\r
  83 > (or anyone else) are you interested in porting them to Go 1 and\r
  84 > maintaining them?  Otherwise it may be time to deprecate them.\r
  85 \r
  86 I care enough for the go environment to keep the go bindings\r
  87 working. I've updated the bindings, the build infrastructure and the\r
  88 notmuch-addrlookup utility to go 1. I'll send a patch series as a\r
  89 followup that is meant to be applied upon Austins patch.\r
  90 \r
  91 The last patch in the series is a cleanup of the source code done with\r
  92 the gofmt utility. Go is somewhat strict in its coding conventions,\r
  93 but there is a utility that magically formats code. This makes the go\r
  94 code in the wild remarkably consistent. We should stick to the\r
  95 conventions too, hence this patch even if it breaks git blame\r
  96 somewhat.\r
  97 \r
  98 I haven't tested the bindings in depth, but the notmuch-addrlookup\r
  99 utility seems to be working fine.\r
 100 \r
 101 Cheers,\r
 102 Justus\r