c7/f83b672c9218e79d25f8c8452173080f22fdfd

   1 Return-Path: <david@tethera.net>\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 190EA431FB6\r
   6         for <notmuch@notmuchmail.org>; Fri, 26 Sep 2014 13:09:35 -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 T51JSThpj0WA for <notmuch@notmuchmail.org>;\r
  16         Fri, 26 Sep 2014 13:09:31 -0700 (PDT)\r
  17 Received: from yantan.tethera.net (yantan.tethera.net [199.188.72.155])\r
  18         (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 4F58C431FAF\r
  21         for <notmuch@notmuchmail.org>; Fri, 26 Sep 2014 13:09:31 -0700 (PDT)\r
  22 Received: from remotemail by yantan.tethera.net with local (Exim 4.80)\r
  23         (envelope-from <david@tethera.net>)\r
  24         id 1XXbpR-0000ZR-I9; Fri, 26 Sep 2014 17:09:21 -0300\r
  25 Received: (nullmailer pid 9379 invoked by uid 1000); Fri, 26 Sep 2014\r
  26         20:09:16 -0000\r
  27 From: David Bremner <david@tethera.net>\r
  28 To: Jani Nikula <jani@nikula.org>, notmuch@notmuchmail.org\r
  29 Subject: Re: [PATCH 09/11] cli/insert: add fail path to add_file_to_database\r
  30 In-Reply-To:\r
  31  <5089ce31afa0ba764520270af1e680a4ab50b4e6.1411379395.git.jani@nikula.org>\r
  32 References: <cover.1411379395.git.jani@nikula.org>\r
  33         <5089ce31afa0ba764520270af1e680a4ab50b4e6.1411379395.git.jani@nikula.org>\r
  34 User-Agent: Notmuch/0.18.1+98~gae27403 (http://notmuchmail.org) Emacs/24.3.1\r
  35         (x86_64-pc-linux-gnu)\r
  36 Date: Fri, 26 Sep 2014 22:09:16 +0200\r
  37 Message-ID: <87eguyf8k3.fsf@maritornes.cs.unb.ca>\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: Fri, 26 Sep 2014 20:09:35 -0000\r
  53 \r
  54 Jani Nikula <jani@nikula.org> writes:\r
  55 \r
  56 \r
  57 > +     status = notmuch_message_tags_to_maildir_flags (message);\r
  58 \r
  59 ...\r
  60 \r
  61 > +     /*\r
  62 > +      * Note: Unfortunately a failed maildir flag sync might\r
  63 > +      * already have renamed the file, in which case the cleanup\r
  64 > +      * path will fail.\r
  65 > +      */\r
  66 \r
  67 I'd like to be more explicit about what potentially bad outcomes there\r
  68 are here.  I guess this message file gets left on disk, unindexed,\r
  69 perhaps forever if the user never runs notmuch new? Would it make sense\r
  70 to suggest the user run notmuch new to "recover" these message files?\r
  71 \r