3f/1f45e0c731f516a5b4c39661cd35b77aa403e5

   1 Return-Path: <jani@nikula.org>\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 arlo.cworth.org (Postfix) with ESMTP id A934D6DE0C3A\r
   6  for <notmuch@notmuchmail.org>; Fri, 25 Sep 2015 13:49:21 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Amavis-Alert: BAD HEADER SECTION, Duplicate header field: "References"\r
   9 X-Spam-Flag: NO\r
  10 X-Spam-Score: -0.156\r
  11 X-Spam-Level: \r
  12 X-Spam-Status: No, score=-0.156 tagged_above=-999 required=5\r
  13  tests=[AWL=-0.180, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H3=-0.01,\r
  14  RCVD_IN_MSPIKE_WL=-0.01, URIBL_SBL=0.644, URIBL_SBL_A=0.1]\r
  15  autolearn=disabled\r
  16 Received: from arlo.cworth.org ([127.0.0.1])\r
  17  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  18  with ESMTP id ydWVKpyAWAh9 for <notmuch@notmuchmail.org>;\r
  19  Fri, 25 Sep 2015 13:49:20 -0700 (PDT)\r
  20 Received: from mail-wi0-f181.google.com (mail-wi0-f181.google.com\r
  21  [209.85.212.181])\r
  22  by arlo.cworth.org (Postfix) with ESMTPS id 28D7A6DE1216\r
  23  for <notmuch@notmuchmail.org>; Fri, 25 Sep 2015 13:49:12 -0700 (PDT)\r
  24 Received: by wicge5 with SMTP id ge5so36252877wic.0\r
  25  for <notmuch@notmuchmail.org>; Fri, 25 Sep 2015 13:49:10 -0700 (PDT)\r
  26 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  27  d=1e100.net; s=20130820;\r
  28  h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\r
  29  :references:in-reply-to:references;\r
  30  bh=ACYfOSYsYDvgwOI+5WUrSlr9NJzVF1jHSbab53uAx4o=;\r
  31  b=QrUWqJ7cuvnzJBC6A1HT2KxNzC0vSa7f29KgYAvv1dplG3hZVFJ6Bf5LAI/MJzznJT\r
  32  0EcNzAzPXfMmONcPFzzSlZLowtcBjGWMi6vk2a2qBamp4uZYoerCe9N5BCrUzlfLIR7m\r
  33  8zDfZkkj6mzg4oPwaAhlUH2bbhfOrCxmJ390S1dim739d72wEa0R2xUJvmPpm3w191Oa\r
  34  CgEBek5X5fq6/D7+1IKyIzA7QIiz2/eIEAVC34dvqfrhpRFVOex/OwCbFFiPSLJ6C9t8\r
  35  mPAIsKts23Kc4/5SOhAsH5bu9ICumaDf6ytk2o8Gqg1Os/ghVpsTI6pU6MoVFDv2rDUd\r
  36  Ooxw==\r
  37 X-Gm-Message-State:\r
  38  ALoCoQmGFchM6k212xkGNsDuQwVr5fjZr/8wWqe7QVj5zqkJw3Hf808xzdUJzQ+gmAMhfYHxuITF\r
  39 X-Received: by 10.180.102.226 with SMTP id fr2mr5876914wib.3.1443214150663;\r
  40  Fri, 25 Sep 2015 13:49:10 -0700 (PDT)\r
  41 Received: from localhost (mobile-access-bcee63-221.dhcp.inet.fi.\r
  42  [188.238.99.221])\r
  43  by smtp.gmail.com with ESMTPSA id lj12sm4997002wic.0.2015.09.25.13.49.09\r
  44  (version=TLSv1/SSLv3 cipher=OTHER);\r
  45  Fri, 25 Sep 2015 13:49:10 -0700 (PDT)\r
  46 From: Jani Nikula <jani@nikula.org>\r
  47 To: notmuch@notmuchmail.org\r
  48 Subject: [PATCH 3/3] cli: delete directory documents on directory removal\r
  49 Date: Fri, 25 Sep 2015 23:48:46 +0300\r
  50 Message-Id:\r
  51  <591f5298ac3f2c79f727cac8bad1251830b28a3c.1443213654.git.jani@nikula.org>\r
  52 X-Mailer: git-send-email 2.1.4\r
  53 In-Reply-To: <cover.1443213654.git.jani@nikula.org>\r
  54 References: <cover.1443213654.git.jani@nikula.org>\r
  55 In-Reply-To: <cover.1443213654.git.jani@nikula.org>\r
  56 References: <cover.1443213654.git.jani@nikula.org>\r
  57 X-BeenThere: notmuch@notmuchmail.org\r
  58 X-Mailman-Version: 2.1.18\r
  59 Precedence: list\r
  60 List-Id: "Use and development of the notmuch mail system."\r
  61  <notmuch.notmuchmail.org>\r
  62 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  63  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  64 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  65 List-Post: <mailto:notmuch@notmuchmail.org>\r
  66 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  67 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  68  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  69 X-List-Received-Date: Fri, 25 Sep 2015 20:49:21 -0000\r
  70 \r
  71 There was a problem with the directory documents being left behind\r
  72 when the filesystem directory was removed. This was worked around in\r
  73 \r
  74 commit acd66cdec075312944e527febd46382e54d99367\r
  75 Author: Jani Nikula <jani@nikula.org>\r
  76 Date:   Sat Sep 5 12:35:31 2015 +0300\r
  77 \r
  78     cli: reset db directory mtime upon directory removal\r
  79 \r
  80 However, that ignored the fact that the directory documents are also\r
  81 still listed by notmuch_directory_get_child_directories() leading to\r
  82 confusing results when running notmuch new. The directory documents\r
  83 are found and queued for removal over and over again.\r
  84 \r
  85 Fix the problem for real by removing the directory documents. This\r
  86 fixes the tests flagged as broken in\r
  87 \r
  88 commit ed9ceda623d3f22fb66365b33db63c5c982067d3\r
  89 Author: David Bremner <david@tethera.net>\r
  90 Date:   Tue Aug 4 08:48:34 2015 +0200\r
  91 \r
  92     test: add debugging output to notmuch-new tests, mark 5 as broken\r
  93 \r
  94 The (non-deterministic) hack test from [1] also still passes with this\r
  95 change.\r
  96 \r
  97 [1] id:1441445731-4362-1-git-send-email-jani@nikula.org\r
  98 ---\r
  99  notmuch-new.c | 12 +++---------\r
 100  1 file changed, 3 insertions(+), 9 deletions(-)\r
 101 \r
 102 diff --git a/notmuch-new.c b/notmuch-new.c\r
 103 index 33645349cd5f..8bfed37fef96 100644\r
 104 --- a/notmuch-new.c\r
 105 +++ b/notmuch-new.c\r
 106 @@ -878,17 +878,11 @@ _remove_directory (void *ctx,\r
 107             goto DONE;\r
 108      }\r
 109  \r
 110 -    /*\r
 111 -     * XXX: The library does not have a function to remove a directory\r
 112 -     * document for a path. Usually this doesn't matter except for a\r
 113 -     * slight waste of space. However, if the directory gets added to\r
 114 -     * the filesystem again, the old directory document is found with\r
 115 -     * the old mtime. Reset the directory mtime to avoid problems.\r
 116 -     */\r
 117 -    notmuch_directory_set_mtime (directory, 0);\r
 118 +    status = notmuch_directory_delete (directory);\r
 119  \r
 120    DONE:\r
 121 -    notmuch_directory_destroy (directory);\r
 122 +    if (status)\r
 123 +       notmuch_directory_destroy (directory);\r
 124      return status;\r
 125  }\r
 126  \r
 127 -- \r
 128 2.1.4\r
 129 \r