Re: Notmuch removing messages from the index, when calling `new` two times in a row
authorDavid Bremner <david@tethera.net>
Sun, 26 Apr 2015 23:45:03 +0000 (08:45 +0900)
committerW. Trevor King <wking@tremily.us>
Sat, 20 Aug 2016 21:48:52 +0000 (14:48 -0700)
d0/dddcbd83dfeb6ecbcf23e9e7ffd426ae65c0df [new file with mode: 0644]

diff --git a/d0/dddcbd83dfeb6ecbcf23e9e7ffd426ae65c0df b/d0/dddcbd83dfeb6ecbcf23e9e7ffd426ae65c0df
new file mode 100644 (file)
index 0000000..16979a2
--- /dev/null
@@ -0,0 +1,141 @@
+Return-Path: <david@tethera.net>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+ by arlo.cworth.org (Postfix) with ESMTP id 4EB2F6DE093A\r
+ for <notmuch@notmuchmail.org>; Sun, 26 Apr 2015 16:46:07 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at cworth.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: 0.332\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=0.332 tagged_above=-999 required=5 tests=[AWL=0.332]\r
+ autolearn=disabled\r
+Received: from arlo.cworth.org ([127.0.0.1])\r
+ by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
+ with ESMTP id ygA5r-bSThvQ for <notmuch@notmuchmail.org>;\r
+ Sun, 26 Apr 2015 16:46:05 -0700 (PDT)\r
+Received: from mx.xen14.node3324.gplhost.com (gitolite.debian.net\r
+ [87.98.215.224])\r
+ by arlo.cworth.org (Postfix) with ESMTPS id BA9146DE02D6\r
+ for <notmuch@notmuchmail.org>; Sun, 26 Apr 2015 16:46:04 -0700 (PDT)\r
+Received: from remotemail by mx.xen14.node3324.gplhost.com with local (Exim\r
+ 4.80) (envelope-from <david@tethera.net>)\r
+ id 1YmWEe-0002bb-E5; Sun, 26 Apr 2015 23:45:16 +0000\r
+Received: (nullmailer pid 21469 invoked by uid 1000); Sun, 26 Apr 2015\r
+ 23:45:03 -0000\r
+From: David Bremner <david@tethera.net>\r
+To: Mayeu <ml+notmuch@6x9.fr>\r
+Subject: Re: Notmuch removing messages from the index, when calling `new` two\r
+ times in a row\r
+In-Reply-To: <20150424112914.5ffef31c@azathoth>\r
+References: <87y4llpiom.fsf@azathoth.devlab.ch>\r
+ <87mw1z8pxu.fsf@maritornes.cs.unb.ca>\r
+ <87fv7ruk2j.fsf@azathoth.i-did-not-set--mail-host-address--so-tickle-me>\r
+ <87oame7a93.fsf@maritornes.cs.unb.ca> <20150424112914.5ffef31c@azathoth>\r
+User-Agent: Notmuch/0.19+96~g703c8f9 (http://notmuchmail.org) Emacs/24.4.1\r
+ (x86_64-pc-linux-gnu)\r
+Date: Mon, 27 Apr 2015 08:45:03 +0900\r
+Message-ID: <87k2wybgjk.fsf@maritornes.cs.unb.ca>\r
+MIME-Version: 1.0\r
+Content-Type: multipart/mixed; boundary="=-=-="\r
+Cc: notmuch@notmuchmail.org\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.18\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+ <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+ <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+ <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Sun, 26 Apr 2015 23:46:07 -0000\r
+\r
+--=-=-=\r
+Content-Type: text/plain\r
+\r
+Mayeu <ml+notmuch@6x9.fr> writes:\r
+\r
+> On Fri, 24 Apr 2015 07:25:28 +0900\r
+> David Bremner <david@tethera.net> wrote:\r
+>> OK. Please try running "notmuch new" (the second time) with --debug.\r
+>\r
+> The output is exactly the same, no other information. I also tried to\r
+> add --verbose but same output. Both are accepted as valid since I do\r
+> net get back any error, but the output is not changed.\r
+>\r
+\r
+I think there is just not enough debugging output in this part of the\r
+code. If you can compile from source, please try the attached patch\r
+(it's against master, but I guess it should apply to 0.19 as well).\r
+\r
+\r
+--=-=-=\r
+Content-Type: text/x-diff\r
+Content-Disposition: inline;\r
+ filename=0001-cli-new-add-more-debugging-output.patch\r
+\r
+>From 52975ecfdc754863e9ac9b465257a566845377c3 Mon Sep 17 00:00:00 2001\r
+From: David Bremner <david@tethera.net>\r
+Date: Mon, 27 Apr 2015 08:42:36 +0900\r
+Subject: [PATCH] cli/new: add more debugging output\r
+\r
+Try to narrow down what part of the code adds files and directories to\r
+the queue(s) to be deleted.\r
+---\r
+ notmuch-new.c | 14 ++++++++++++++\r
+ 1 file changed, 14 insertions(+)\r
+\r
+diff --git a/notmuch-new.c b/notmuch-new.c\r
+index e6c283e..39b0b9a 100644\r
+--- a/notmuch-new.c\r
++++ b/notmuch-new.c\r
+@@ -528,6 +528,10 @@ add_files (notmuch_database_t *notmuch,\r
+                                             "%s/%s", path,\r
+                                             notmuch_filenames_get (db_files));\r
\r
++          if (state->debug)\r
++              printf ("(D) add_files_recursive, pass 2: queuing passed file %s for deletion\n",\r
++                      absolute);\r
++\r
+           _filename_list_add (state->removed_files, absolute);\r
\r
+           notmuch_filenames_move_to_next (db_files);\r
+@@ -542,6 +546,9 @@ add_files (notmuch_database_t *notmuch,\r
+           {\r
+               char *absolute = talloc_asprintf (state->removed_directories,\r
+                                                 "%s/%s", path, filename);\r
++              if (state->debug)\r
++                  printf ("(D) add_files_recursive, pass 2: queuing passed directory %s for deletion\n",\r
++                      absolute);\r
\r
+               _filename_list_add (state->removed_directories, absolute);\r
+           }\r
+@@ -610,6 +617,9 @@ add_files (notmuch_database_t *notmuch,\r
+       char *absolute = talloc_asprintf (state->removed_files,\r
+                                         "%s/%s", path,\r
+                                         notmuch_filenames_get (db_files));\r
++      if (state->debug)\r
++          printf ("(D) add_files_recursive, pass 3: queuing leftover file %s for deletion\n",\r
++                  absolute);\r
\r
+       _filename_list_add (state->removed_files, absolute);\r
\r
+@@ -622,6 +632,10 @@ add_files (notmuch_database_t *notmuch,\r
+                                         "%s/%s", path,\r
+                                         notmuch_filenames_get (db_subdirs));\r
\r
++      if (state->debug)\r
++          printf ("(D) add_files_recursive, pass 3: queuing leftover directory %s for deletion\n",\r
++                  absolute);\r
++\r
+       _filename_list_add (state->removed_directories, absolute);\r
\r
+       notmuch_filenames_move_to_next (db_subdirs);\r
+-- \r
+2.1.4\r
+\r
+\r
+--=-=-=--\r