5e/c3c60291a8a0caeab0cc0c5c4c4d3b912f6f9d

   1 Return-Path: <bremner@unb.ca>\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 48307431FB6\r
   6         for <notmuch@notmuchmail.org>; Wed, 26 Dec 2012 19:24:02 -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 hQUe00uJARxk for <notmuch@notmuchmail.org>;\r
  16         Wed, 26 Dec 2012 19:24:01 -0800 (PST)\r
  17 Received: from tesseract.cs.unb.ca (tesseract.cs.unb.ca [131.202.240.238])\r
  18         (using TLSv1 with cipher AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 7816B431FAF\r
  21         for <notmuch@notmuchmail.org>; Wed, 26 Dec 2012 19:24:01 -0800 (PST)\r
  22 Received: from fctnnbsc30w-156034082078.dhcp-dynamic.fibreop.nb.bellaliant.net\r
  23         ([156.34.82.78] helo=zancas.localnet)\r
  24         by tesseract.cs.unb.ca with esmtpsa\r
  25         (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.72)\r
  26         (envelope-from <bremner@unb.ca>)\r
  27         id 1To44a-0003F7-Ge; Wed, 26 Dec 2012 23:23:56 -0400\r
  28 Received: from bremner by zancas.localnet with local (Exim 4.80)\r
  29         (envelope-from <bremner@unb.ca>)\r
  30         id 1To44U-0006cb-Qc; Wed, 26 Dec 2012 23:23:50 -0400\r
  31 From: David Bremner <david@tethera.net>\r
  32 To: Michael Forney <mforney@mforney.org>, Adrien Bustany <adrien@bustany.org>,\r
  33         notmuch@notmuchmail.org\r
  34 Subject: Re: [PATCH v2 0/2] Add flush/reopen methods to notmuch_database_t\r
  35 In-Reply-To: <1355276420-ner-1510@novus>\r
  36 References: <1342723401-26103-1-git-send-email-adrien@bustany.org>\r
  37         <1350510742-13447-1-git-send-email-adrien@bustany.org>\r
  38         <1355276420-ner-1510@novus>\r
  39 User-Agent: Notmuch/0.14+213~g4af1ac6 (http://notmuchmail.org) Emacs/24.2.1\r
  40         (x86_64-pc-linux-gnu)\r
  41 Date: Wed, 26 Dec 2012 23:23:50 -0400\r
  42 Message-ID: <878v8kqit5.fsf@zancas.localnet>\r
  43 MIME-Version: 1.0\r
  44 Content-Type: text/plain\r
  45 X-Spam_bar: -\r
  46 X-BeenThere: notmuch@notmuchmail.org\r
  47 X-Mailman-Version: 2.1.13\r
  48 Precedence: list\r
  49 List-Id: "Use and development of the notmuch mail system."\r
  50         <notmuch.notmuchmail.org>\r
  51 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  52         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  53 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  54 List-Post: <mailto:notmuch@notmuchmail.org>\r
  55 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  56 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  57         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  58 X-List-Received-Date: Thu, 27 Dec 2012 03:24:02 -0000\r
  59 \r
  60 Michael Forney <mforney@mforney.org> writes:\r
  61 \r
  62 > On Thu, 18 Oct 2012 00:52:20 +0300, Adrien Bustany <adrien@bustany.org> wrote:\r
  63 >> The code of the patches in unchanged, but the formatting issues are now\r
  64 >> hopefully fixed.\r
  65 >\r
  66 > I would like to bump this patch set. I also need these features from\r
  67 > libnotmuch. Currently there is no way to recover from Xapian errors,\r
  68 > which is pretty limiting.\r
  69 >\r
  70 > If it is the flush/commit that is the issue, I would be happy to make an\r
  71 > updated patch set.\r
  72 \r
  73 Hi Michael;\r
  74 \r
  75 Right, we should use the current name of 'commit'. We also need some way\r
  76 to check the xapian version in the configure script, and to make sure\r
  77 the right name is supported.\r
  78 \r
  79 Jani refers to \r
  80 \r
  81      id:1350487737-32058-2-git-send-email-bgamari.foss@gmail.com\r
  82 \r
  83 farther up the thread as having some relevant code/ideas.\r
  84 \r
  85 I think it's reasonable to require Xapian version at least 1.2.0, so\r
  86 that might simplify the configuration check. \r
  87 \r
  88 The other thing I wondered is if the error handling in these routines is\r
  89 as good as it could be. I guess it isn't worse than what is there, but\r
  90 especially since you are explicitly wanting to handle error conditions,\r
  91 it would be good to think if we can do something better than adding yet\r
  92 another printf to the library.\r
  93 \r
  94 d\r