7c/852fd1d6fc657a7afd966964766b549af344b3

   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 EBF62431FB6\r
   6         for <notmuch@notmuchmail.org>; Sun, 13 May 2012 14:55:42 -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 nKP-KO2Z-0ep for <notmuch@notmuchmail.org>;\r
  16         Sun, 13 May 2012 14:55:42 -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 7AA7C431FAE\r
  21         for <notmuch@notmuchmail.org>; Sun, 13 May 2012 14:55:42 -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 F3C61587DD8\r
  26         for <notmuch@notmuchmail.org>; Sun, 13 May 2012 23:55:40 +0200 (CEST)\r
  27 Received: by mail.jade-hamburg.de (Postfix, from userid 401)\r
  28         id 5BCE2DF2A3; Sun, 13 May 2012 23:55:40 +0200 (CEST)\r
  29 Received: from thinkbox.jade-hamburg.de (thinkbox.jadE-Hamburg.de\r
  30  [10.1.1.153])  (using TLSv1 with cipher AES256-SHA (256/256 bits))     (No client\r
  31  certificate requested) (Authenticated sender: teythoon)        by\r
  32  mail.jade-hamburg.de (Postfix) with ESMTPSA id 0D7CFDF2A0;     Sun, 13 May 2012\r
  33  23:55:39 +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 1STglO-0004K0-I1; Sun, 13 May 2012 23:55:38 +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+\r
  42 To: Tomi Ollila <tomi.ollila@iki.fi>, Austin Clements <amdragon@MIT.EDU>,\r
  43         notmuch@notmuchmail.org\r
  44 References: <1336939029-13969-1-git-send-email-amdragon@mit.edu>\r
  45         <m2txzjlsax.fsf@guru.guru-group.fi>\r
  46 In-Reply-To: <m2txzjlsax.fsf@guru.guru-group.fi>\r
  47 Message-ID: <20120513215538.15106.43740@thinkbox.jade-hamburg.de>\r
  48 Subject: Re: [PATCH 0/5] Fix notmuch_database_get_directory API\r
  49 Date: Sun, 13 May 2012 23:55:38 +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: Sun, 13 May 2012 21:55:43 -0000\r
  63 \r
  64 Quoting Tomi Ollila (2012-05-13 22:49:58)\r
  65 > On Sun, May 13 2012, Austin Clements <amdragon@MIT.EDU> wrote:\r
  66 > =\r
  67 \r
  68 > > This is a proposed last-minute change for 0.13.  It fixes the\r
  69 > > notmuch_database_get_directory API in the same way we're fixing\r
  70 > > notmuch_database_open, etc in this release.  Since this is a\r
  71 > > backwards-incompatible change, it would be nice to lump it with the\r
  72 > > other API-breaking changes.\r
  73 > >\r
  74 > > To keep the patch simple, this does not change the behavior of\r
  75 > > notmuch_database_get_directory, but it puts us in a good position to\r
  76 > > fix it in the future.\r
  77 > =\r
  78 \r
  79 > Looks good to me (and applied in my current environment). The c/c++ =\r
  80 \r
  81 > changes were easy to understand and the python/go/ruby binding\r
  82 > changes looks like the old changes -- but those who understand more\r
  83 > (and actually uses those bindings) could do better review.\r
  84 \r
  85 I'd say the changes to the python and go bindings are fine.\r
  86 \r
  87 > It would be nice to get those in so that we may have chance not\r
  88 > updating SONAME for notmuch 0.14.\r
  89 \r
  90 Yes, I'd love to see this in 0.13 as well, but I don't really see the\r
  91 problem with breaking the api if this fixes design issues. After all,\r
  92 notmuch is young and so are all the projects building uppon it.\r
  93 \r
  94 Justus\r