cf/2a7f4dd9abd4406e7e7296d6a29f3081234607

   1 Return-Path: <sojkam1@fel.cvut.cz>\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 BFC4F431FAF\r
   6         for <notmuch@notmuchmail.org>; Wed, 19 Dec 2012 13:39:27 -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: -2.3\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.3 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3] 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 5rJoA43xMx9h for <notmuch@notmuchmail.org>;\r
  16         Wed, 19 Dec 2012 13:39:25 -0800 (PST)\r
  17 Received: from max.feld.cvut.cz (max.feld.cvut.cz [147.32.192.36])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 70901431FAE\r
  19         for <notmuch@notmuchmail.org>; Wed, 19 Dec 2012 13:39:25 -0800 (PST)\r
  20 Received: from localhost (unknown [192.168.200.7])\r
  21         by max.feld.cvut.cz (Postfix) with ESMTP id E0D683CFE88;\r
  22         Wed, 19 Dec 2012 22:39:24 +0100 (CET)\r
  23 X-Virus-Scanned: IMAP STYX AMAVIS\r
  24 Received: from max.feld.cvut.cz ([192.168.200.1])\r
  25         by localhost (styx.feld.cvut.cz [192.168.200.7]) (amavisd-new,\r
  26         port 10044)\r
  27         with ESMTP id yi_hg2MnMMbp; Wed, 19 Dec 2012 22:39:23 +0100 (CET)\r
  28 Received: from imap.feld.cvut.cz (imap.feld.cvut.cz [147.32.192.34])\r
  29         by max.feld.cvut.cz (Postfix) with ESMTP id 3716719F30DA;\r
  30         Wed, 19 Dec 2012 22:39:23 +0100 (CET)\r
  31 Received: from steelpick.2x.cz (unknown [213.29.198.144])\r
  32         (Authenticated sender: sojkam1)\r
  33         by imap.feld.cvut.cz (Postfix) with ESMTPSA id 080D8660904;\r
  34         Wed, 19 Dec 2012 22:39:23 +0100 (CET)\r
  35 Received: from wsh by steelpick.2x.cz with local (Exim 4.80)\r
  36         (envelope-from <sojkam1@fel.cvut.cz>)\r
  37         id 1TlRMD-00079N-KE; Wed, 19 Dec 2012 22:39:17 +0100\r
  38 From: Michal Sojka <sojka@os.inf.tu-dresden.de>\r
  39 To: Jani Nikula <jani@nikula.org>, david@tethera.net, notmuch@notmuchmail.org\r
  40 Subject: Re: [PATCH] test: Adding non-maildir tags does not move message from\r
  41         new to cur\r
  42 In-Reply-To: <87vcccp4y3.fsf@nikula.org>\r
  43 References: <1316039001-32602-5-git-send-email-l.rilling@av7.net>\r
  44         <1354994132-866-1-git-send-email-david@tethera.net>\r
  45         <87vcccp4y3.fsf@nikula.org>\r
  46 User-Agent: Notmuch/0.14+107~g342038e (http://notmuchmail.org) Emacs/24.2.1\r
  47         (x86_64-pc-linux-gnu)\r
  48 Date: Wed, 19 Dec 2012 22:39:17 +0100\r
  49 Message-ID: <87mwx9g1ru.fsf@steelpick.2x.cz>\r
  50 MIME-Version: 1.0\r
  51 Content-Type: text/plain\r
  52 X-BeenThere: notmuch@notmuchmail.org\r
  53 X-Mailman-Version: 2.1.13\r
  54 Precedence: list\r
  55 List-Id: "Use and development of the notmuch mail system."\r
  56         <notmuch.notmuchmail.org>\r
  57 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  58         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  59 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  60 List-Post: <mailto:notmuch@notmuchmail.org>\r
  61 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  62 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  63         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  64 X-List-Received-Date: Wed, 19 Dec 2012 21:39:27 -0000\r
  65 \r
  66 Hello Jani,\r
  67 \r
  68 On Sat, Dec 08 2012, Jani Nikula wrote:\r
  69 > On Sat, 08 Dec 2012, david@tethera.net wrote:\r
  70 >> From: Michal Sojka <sojka@os.inf.tu-dresden.de>\r
  71 >>\r
  72 >> Some MUA's like mutt show the difference between "new" emails living in maildir\r
  73 >> directory new/, and "old" emails living in maildir directory cur/. However\r
  74 >> notmuch tag unconditionally moves selected messages from new/ to cur/, even if\r
  75 >> no maildir synchronized tag is changed.\r
  76 >>\r
  77 >> While maildir specification forbids messages with tags living in new/, there is\r
  78 >> no need to move messages to cur/ when no maildir synchronized tag is changed.\r
  79 >> Thus notmuch can remain transparent with respect to other MUA's.\r
  80 >>\r
  81 >> [ Edited commit log to better describe the intended changes, and tag the\r
  82 >>   test as broken until the actual changes are implemented -- Louis Rilling ]\r
  83 >>\r
  84 >> Signed-off-by: Louis Rilling <l.rilling@av7.net>\r
  85 >>\r
  86 >> [ Converted to use test_subtest_known_broken, David Bremner ]\r
  87 >> ---\r
  88 >>\r
  89 >> Do we agree that the behaviour of moving messages to ./cur on tagging\r
  90 >> is broken? If so, maybe it's worth tidying up and applying this.  The\r
  91 >> use of cd and ls strikes me as slightly suspect, but I welcome other\r
  92 >> opinions.\r
  93 >\r
  94 > I think I would narrow down the special case a bit: I think messages in\r
  95 > ./new that have no maildir flags, and have no ":2," in the end of the\r
  96 > filename, and and the tag change(s) will not affect maildir flags,\r
  97 > should stay in ./new. Files in ./new should not have ":2," or maildir\r
  98 > flags, and I see no reason to support having them there.\r
  99 >\r
 100 > Thus any messages in ./new that do have maildir flags, or have ":2," in\r
 101 > the end of the filename should probably be moved to ./cur, even if the\r
 102 > tag change(s) do not affect maildir flags. The patch in this thread\r
 103 > fails here. It also changes the behaviour for messages in ./cur by not\r
 104 > appending ":2," to them.\r
 105 \r
 106 I agree with you. In\r
 107 id:1355952747-27350-1-git-send-email-sojkam1@fel.cvut.cz I sent the\r
 108 tests for the cases descried above as well as the updated patch for tag\r
 109 to maildir synchronization.\r
 110 \r
 111 > As to the test, I think it should do something along the lines of (based\r
 112 > on search-output test):\r
 113 >\r
 114 > notmuch search --output=files subject:"Message to stay in new" | sed -e "s,$MAIL_DIR,MAIL_DIR," >OUTPUT\r
 115 > cat <<EOF >EXPECTED\r
 116 > MAIL_DIR/new/message-to-stay-in-new\r
 117 > EOF\r
 118 > test_expect_equal_file OUTPUT EXPECTED\r
 119 \r
 120 With this you test what notmuch thinks about the file names of messages,\r
 121 not whether the files have actually been renamed. For this reason I kept\r
 122 the previous way of testing in the new patches.\r
 123 \r
 124 Cheers,\r
 125 -Michal\r