notmuch insert --keep fails in parallel w/ notmuch new
authorMaarten Aertsen <sagi-notmuch@rtsn.nl>
Sun, 24 Jan 2016 15:48:00 +0000 (16:48 +0100)
committerW. Trevor King <wking@tremily.us>
Sat, 20 Aug 2016 23:20:56 +0000 (16:20 -0700)
ca/521e66ddbb9a3c96a7ca9d6b6e62e48d0d7909 [new file with mode: 0644]

diff --git a/ca/521e66ddbb9a3c96a7ca9d6b6e62e48d0d7909 b/ca/521e66ddbb9a3c96a7ca9d6b6e62e48d0d7909
new file mode 100644 (file)
index 0000000..76c4381
--- /dev/null
@@ -0,0 +1,98 @@
+Return-Path: <sagi@rtsn.nl>\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 849AB6DE02DA\r
+ for <notmuch@notmuchmail.org>; Sun, 24 Jan 2016 07:55:15 -0800 (PST)\r
+X-Virus-Scanned: Debian amavisd-new at cworth.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.552\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.552 tagged_above=-999 required=5\r
+ tests=[RP_MATCHES_RCVD=-0.55, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001]\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 y2NJmVcKTC9K for <notmuch@notmuchmail.org>;\r
+ Sun, 24 Jan 2016 07:55:13 -0800 (PST)\r
+X-Greylist: delayed 427 seconds by postgrey-1.35 at arlo;\r
+ Sun, 24 Jan 2016 07:55:13 PST\r
+Received: from kardo2.rtsn.nl (rtsn.nl [37.97.129.244])\r
+ by arlo.cworth.org (Postfix) with ESMTPS id 179926DE0217\r
+ for <notmuch@notmuchmail.org>; Sun, 24 Jan 2016 07:55:13 -0800 (PST)\r
+Received: by kardo2.rtsn.nl (Postfix, from userid 1000)\r
+ id D3C691FF55F; Sun, 24 Jan 2016 16:48:00 +0100 (CET)\r
+Content-Type: text/plain; charset="utf-8"\r
+MIME-Version: 1.0\r
+Content-Transfer-Encoding: quoted-printable\r
+To: Notmuch Mail <notmuch@notmuchmail.org>\r
+Message-ID: <20160124154800.19103.24730@kardo2.rtsn.nl>\r
+From: Maarten Aertsen <sagi-notmuch@rtsn.nl>\r
+User-Agent: alot/0.3.6\r
+Subject: notmuch insert --keep fails in parallel w/ notmuch new\r
+Date: Sun, 24 Jan 2016 16:48:00 +0100\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.20\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+ <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <https://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: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
+ <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Sun, 24 Jan 2016 15:55:15 -0000\r
+\r
+Hello list,\r
+\r
+I'd like to report a bug against notmuch 0.21, packaged for Debian =\r
+\r
+jessie-backports as 0.21-3~bpo8+1.\r
+\r
+# Steps to reproduce:\r
+  1. start a notmuch new mail w/ lots of stuff to process (e.g. a =\r
+\r
+     migration)\r
+  2. try notmuch insert --keep\r
+\r
+# Expected result:\r
+Given the current text in the manpage, I'd expect notmuch insert --keep =\r
+\r
+to save the message to the Maildir even if no db interaction is possible:\r
+\r
+"--keep Keep the message file if indexing fails, and keep the message =\r
+\r
+       indexed if applying tags or maildir flag synchronization fails.  =\r
+\r
+       Ignore these  errors and return exit status 0 to indicate =\r
+\r
+        succesful mail delivery."\r
+\r
+That way, I can cron notmuch new to pick up the occasional message that =\r
+\r
+can't get indexed right away.\r
+\r
+# Actual result:\r
+notmuch insert --keep fails with exit code 1 due to the inability to get =\r
+\r
+db access.\r
+\r
+# Context:\r
+I've switched to a setup with postfix, where notmuch insert --keep is =\r
+\r
+called by /local/, postfix's local delivery daemon, via a pipe in =\r
+\r
+.forward. Local treats a non-zero exit code as a 5.3.x status code and =\r
+\r
+sends a bounce to the sender.\r
+\r
+I had a short chat with j4ni on #notmuch, where he said:\r
+< j4ni> Sagi: oh bummer, we try to open the db first, and bail out =\r
+\r
+early. we should change the order of that (at least with --keep).\r
+\r
+Thanks for developing notmuch.\r
+\r
+best regards, Maarten Aertsen\r