03/faf2d21b25230a0a3264eddf7343e7f2080a53

   1 Return-Path: <dottedmag@dottedmag.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 5FDDB431FBC\r
   6         for <notmuch@notmuchmail.org>; Sun, 22 Nov 2009 17:13:41 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 Received: from olra.theworths.org ([127.0.0.1])\r
   9         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  10         with ESMTP id etT6r1jbggiX for <notmuch@notmuchmail.org>;\r
  11         Sun, 22 Nov 2009 17:13:40 -0800 (PST)\r
  12 Received: from dottedmag.net (burger.dottedmag.net [212.75.37.82])\r
  13         by olra.theworths.org (Postfix) with ESMTP id DC293431FAE\r
  14         for <notmuch@notmuchmail.org>; Sun, 22 Nov 2009 17:13:39 -0800 (PST)\r
  15 Received: from vertex.dottedmag (unknown [91.197.127.125])\r
  16         by dottedmag.net (Postfix) with ESMTPSA id D82028C979\r
  17         for <notmuch@notmuchmail.org>; Mon, 23 Nov 2009 02:13:36 +0100 (CET)\r
  18 Received: from dottedmag by vertex.dottedmag with local (Exim 4.69)\r
  19         (envelope-from <dottedmag@dottedmag.net>) id 1NCNUb-00049x-MJ\r
  20         for notmuch@notmuchmail.org; Mon, 23 Nov 2009 07:13:25 +0600\r
  21 From: Mikhail Gusarov <dottedmag@dottedmag.net>\r
  22 To: notmuch@notmuchmail.org\r
  23 Date: Mon, 23 Nov 2009 07:13:25 +0600\r
  24 Message-Id: <1258938805-15961-1-git-send-email-dottedmag@dottedmag.net>\r
  25 X-Mailer: git-send-email 1.6.3.3\r
  26 Subject: [notmuch] [PATCH] Handle message renames in mail spool\r
  27 X-BeenThere: notmuch@notmuchmail.org\r
  28 X-Mailman-Version: 2.1.12\r
  29 Precedence: list\r
  30 List-Id: "Use and development of the notmuch mail system."\r
  31         <notmuch.notmuchmail.org>\r
  32 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  33         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  34 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  35 List-Post: <mailto:notmuch@notmuchmail.org>\r
  36 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  37 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  38         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  39 X-List-Received-Date: Mon, 23 Nov 2009 01:13:41 -0000\r
  40 \r
  41 In order to handle message renames the following changes were deemed necessary:\r
  42 \r
  43 * Mtime check on individual files was disabled. As files may be moved around\r
  44 without changing their mtime, it's necessary to parse them even if they appear\r
  45 old in case old message was moved. mtime check on directories was kept as moving\r
  46 files changes mtime of directory.\r
  47 \r
  48 * If message being parsed is already found in database under different path,\r
  49 then this message is considered to be moved, path is updated in database and\r
  50 this file does not undergo further processing.\r
  51 \r
  52 Note that after applying this patch notmuch still does not handle copying files\r
  53 (which is harmless, database will point to the last copy of message found during\r
  54 'notmuch new') and deleting files (which is more serious, as dangling entries\r
  55 will show up in searches).\r
  56 \r
  57 Signed-off-by: Mikhail Gusarov <dottedmag@dottedmag.net>\r
  58 ---\r
  59  lib/database.cc |   21 +++++++-----\r
  60  notmuch-new.c   |   92 ++++++++++++++++++++++++++----------------------------\r
  61  2 files changed, 56 insertions(+), 57 deletions(-)\r
  62 \r
  63 diff --git a/lib/database.cc b/lib/database.cc\r
  64 index 2c90019..d88efbe 100644\r
  65 --- a/lib/database.cc\r
  66 +++ b/lib/database.cc\r
  67 @@ -990,20 +990,23 @@ notmuch_database_add_message (notmuch_database_t *notmuch,\r
  68         if (private_status == NOTMUCH_PRIVATE_STATUS_NO_DOCUMENT_FOUND) {\r
  69             _notmuch_message_set_filename (message, filename);\r
  70             _notmuch_message_add_term (message, "type", "mail");\r
  71 +\r
  72 +           ret = _notmuch_database_link_message (notmuch, message, message_file);\r
  73 +           if (ret)\r
  74 +               goto DONE;\r
  75 +\r
  76 +           date = notmuch_message_file_get_header (message_file, "date");\r
  77 +           _notmuch_message_set_date (message, date);\r
  78 +\r
  79 +           _notmuch_message_index_file (message, filename);\r
  80 +       } else if (strcmp(notmuch_message_get_filename(message), filename)) {\r
  81 +           /* Message file has been moved/renamed */\r
  82 +           _notmuch_message_set_filename (message, filename);\r
  83         } else {\r
  84             ret = NOTMUCH_STATUS_DUPLICATE_MESSAGE_ID;\r
  85             goto DONE;\r
  86         }\r
  87  \r
  88 -       ret = _notmuch_database_link_message (notmuch, message, message_file);\r
  89 -       if (ret)\r
  90 -           goto DONE;\r
  91 -\r
  92 -       date = notmuch_message_file_get_header (message_file, "date");\r
  93 -       _notmuch_message_set_date (message, date);\r
  94 -\r
  95 -       _notmuch_message_index_file (message, filename);\r
  96 -\r
  97         _notmuch_message_sync (message);\r
  98      } catch (const Xapian::Error &error) {\r
  99         fprintf (stderr, "A Xapian exception occurred adding message: %s.\n",\r
 100 diff --git a/notmuch-new.c b/notmuch-new.c\r
 101 index 0dd2784..d16679c 100644\r
 102 --- a/notmuch-new.c\r
 103 +++ b/notmuch-new.c\r
 104 @@ -174,54 +174,50 @@ add_files_recursive (notmuch_database_t *notmuch,\r
 105         }\r
 106  \r
 107         if (S_ISREG (st->st_mode)) {\r
 108 -           /* If the file hasn't been modified since the last\r
 109 -            * add_files, then we need not look at it. */\r
 110 -           if (path_dbtime == 0 || st->st_mtime > path_dbtime) {\r
 111 -               state->processed_files++;\r
 112 -\r
 113 -               status = notmuch_database_add_message (notmuch, next, &message);\r
 114 -               switch (status) {\r
 115 -                   /* success */\r
 116 -                   case NOTMUCH_STATUS_SUCCESS:\r
 117 -                       state->added_messages++;\r
 118 -                       tag_inbox_and_unread (message);\r
 119 -                       break;\r
 120 -                   /* Non-fatal issues (go on to next file) */\r
 121 -                   case NOTMUCH_STATUS_DUPLICATE_MESSAGE_ID:\r
 122 -                       /* Stay silent on this one. */\r
 123 -                       break;\r
 124 -                   case NOTMUCH_STATUS_FILE_NOT_EMAIL:\r
 125 -                       fprintf (stderr, "Note: Ignoring non-mail file: %s\n",\r
 126 -                                next);\r
 127 -                       break;\r
 128 -                   /* Fatal issues. Don't process anymore. */\r
 129 -                   case NOTMUCH_STATUS_READONLY_DATABASE:\r
 130 -                   case NOTMUCH_STATUS_XAPIAN_EXCEPTION:\r
 131 -                   case NOTMUCH_STATUS_OUT_OF_MEMORY:\r
 132 -                       fprintf (stderr, "Error: %s. Halting processing.\n",\r
 133 -                                notmuch_status_to_string (status));\r
 134 -                       ret = status;\r
 135 -                       goto DONE;\r
 136 -                   default:\r
 137 -                   case NOTMUCH_STATUS_FILE_ERROR:\r
 138 -                   case NOTMUCH_STATUS_NULL_POINTER:\r
 139 -                   case NOTMUCH_STATUS_TAG_TOO_LONG:\r
 140 -                   case NOTMUCH_STATUS_UNBALANCED_FREEZE_THAW:\r
 141 -                   case NOTMUCH_STATUS_LAST_STATUS:\r
 142 -                       INTERNAL_ERROR ("add_message returned unexpected value: %d",  status);\r
 143 -                       goto DONE;\r
 144 -               }\r
 145 -\r
 146 -               if (message) {\r
 147 -                   notmuch_message_destroy (message);\r
 148 -                   message = NULL;\r
 149 -               }\r
 150 -\r
 151 -               if (do_add_files_print_progress) {\r
 152 -                   do_add_files_print_progress = 0;\r
 153 -                   add_files_print_progress (state);\r
 154 -               }\r
 155 -           }\r
 156 +            state->processed_files++;\r
 157 +\r
 158 +            status = notmuch_database_add_message (notmuch, next, &message);\r
 159 +            switch (status) {\r
 160 +                /* success */\r
 161 +                case NOTMUCH_STATUS_SUCCESS:\r
 162 +                    state->added_messages++;\r
 163 +                    tag_inbox_and_unread (message);\r
 164 +                    break;\r
 165 +               /* Non-fatal issues (go on to next file) */\r
 166 +                case NOTMUCH_STATUS_DUPLICATE_MESSAGE_ID:\r
 167 +                    /* Stay silent on this one. */\r
 168 +                    break;\r
 169 +                case NOTMUCH_STATUS_FILE_NOT_EMAIL:\r
 170 +                    fprintf (stderr, "Note: Ignoring non-mail file: %s\n",\r
 171 +                             next);\r
 172 +                    break;\r
 173 +               /* Fatal issues. Don't process anymore. */\r
 174 +                case NOTMUCH_STATUS_READONLY_DATABASE:\r
 175 +                case NOTMUCH_STATUS_XAPIAN_EXCEPTION:\r
 176 +                case NOTMUCH_STATUS_OUT_OF_MEMORY:\r
 177 +                    fprintf (stderr, "Error: %s. Halting processing.\n",\r
 178 +                             notmuch_status_to_string (status));\r
 179 +                    ret = status;\r
 180 +                    goto DONE;\r
 181 +                default:\r
 182 +                case NOTMUCH_STATUS_FILE_ERROR:\r
 183 +                case NOTMUCH_STATUS_NULL_POINTER:\r
 184 +                case NOTMUCH_STATUS_TAG_TOO_LONG:\r
 185 +                case NOTMUCH_STATUS_UNBALANCED_FREEZE_THAW:\r
 186 +                case NOTMUCH_STATUS_LAST_STATUS:\r
 187 +                    INTERNAL_ERROR ("add_message returned unexpected value: %d",  status);\r
 188 +                    goto DONE;\r
 189 +            }\r
 190 +\r
 191 +            if (message) {\r
 192 +                notmuch_message_destroy (message);\r
 193 +                message = NULL;\r
 194 +            }\r
 195 +\r
 196 +            if (do_add_files_print_progress) {\r
 197 +                do_add_files_print_progress = 0;\r
 198 +                add_files_print_progress (state);\r
 199 +            }\r
 200         } else if (S_ISDIR (st->st_mode)) {\r
 201             status = add_files_recursive (notmuch, next, st, state);\r
 202             if (status && ret == NOTMUCH_STATUS_SUCCESS)\r
 203 -- \r
 204 1.6.3.3\r
 205 \r