a2/0d53828ebfdb9544095410a2c8391209dcdd37

   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 olra.theworths.org (Postfix) with ESMTP id 1263D431FC2\r
   6         for <notmuch@notmuchmail.org>; Sun, 23 Nov 2014 03:15:23 -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: -0.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 4untd6rGMfhi for <notmuch@notmuchmail.org>;\r
  16         Sun, 23 Nov 2014 03:15:19 -0800 (PST)\r
  17 Received: from mail-wi0-f172.google.com (mail-wi0-f172.google.com\r
  18         [209.85.212.172]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 07A8C431FBC\r
  21         for <notmuch@notmuchmail.org>; Sun, 23 Nov 2014 03:15:19 -0800 (PST)\r
  22 Received: by mail-wi0-f172.google.com with SMTP id n3so3090444wiv.17\r
  23         for <notmuch@notmuchmail.org>; Sun, 23 Nov 2014 03:15:18 -0800 (PST)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=1e100.net; s=20130820;\r
  26         h=x-gm-message-state:from:to:cc:subject:date:message-id;\r
  27         bh=+clLSqP6UFD8Fm4ZmdLFUrDNhCLc+ZChqX2iIYq4TOY=;\r
  28         b=kH/gT1X8v8mZChzFcnw7Yayy7pVr+uikStxMbRzyVEprrf7ig6578OZEbzVPArVENW\r
  29         azdoEF6Z/hNlrmS5Rz1SWQ0tx9FNX2Dw1Yh2CdqXYM95lh0HPobdRXs5mB6MzoF+J37m\r
  30         p2YjyDfzFdPrezeHzRVf0/G/im69tMxfEM6v9ZX17op6NyVKvBLG5E9PevtK3QTQLqA/\r
  31         3clTyqEkW+KyXjfVm1922Cd32HVfJMmcgEV1q0O4TMwVFkI/arKSZWh/UUNT8iiaT1lh\r
  32         DL+n2stVjPCFKXhCgNzkmSlULnOWRLpS2DwSqg7rfY0pusz2SIWCEquX+KideBXVHZ4H\r
  33         1s2A==\r
  34 X-Gm-Message-State:\r
  35  ALoCoQn26z281h8Nd8fRXMOdUMH5fyJAjPDdxwTb0xzGKONxPFIGKcjqDIT2Shvw2x8iW4g9NP3O\r
  36 X-Received: by 10.180.92.234 with SMTP id cp10mr12734733wib.16.1416741317798; \r
  37         Sun, 23 Nov 2014 03:15:17 -0800 (PST)\r
  38 Received: from localhost (dsl-hkibrasgw2-58c362-144.dhcp.inet.fi.\r
  39         [88.195.98.144])\r
  40         by mx.google.com with ESMTPSA id pl6sm6928995wic.22.2014.11.23.03.15.16\r
  41         for <multiple recipients>\r
  42         (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\r
  43         Sun, 23 Nov 2014 03:15:17 -0800 (PST)\r
  44 From: Jani Nikula <jani@nikula.org>\r
  45 To: notmuch@notmuchmail.org\r
  46 Subject: [PATCH] lib: drop the deprecation message for single-message mbox\r
  47         files\r
  48 Date: Sun, 23 Nov 2014 13:15:12 +0200\r
  49 Message-Id: <1416741312-26332-1-git-send-email-jani@nikula.org>\r
  50 X-Mailer: git-send-email 2.1.3\r
  51 X-BeenThere: notmuch@notmuchmail.org\r
  52 X-Mailman-Version: 2.1.13\r
  53 Precedence: list\r
  54 List-Id: "Use and development of the notmuch mail system."\r
  55         <notmuch.notmuchmail.org>\r
  56 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  57         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  58 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  59 List-Post: <mailto:notmuch@notmuchmail.org>\r
  60 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  61 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  62         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  63 X-List-Received-Date: Sun, 23 Nov 2014 11:15:23 -0000\r
  64 \r
  65 We generally do not support mbox files, but for historical reasons\r
  66 we've supported single-message mbox files, with a deprecation\r
  67 message. We've tried dropping the support altogether, but backed out\r
  68 of it because we'd need to stop indexing them, while keeping support\r
  69 for previously indexed files. This would be more complicated than\r
  70 simply supporting single-message mbox files. Therefore, drop the\r
  71 deprecation message, and just silently accept single-message mboxes.\r
  72 ---\r
  73  lib/message-file.c | 21 ++++-----------------\r
  74  test/T050-new.sh   |  6 +-----\r
  75  2 files changed, 5 insertions(+), 22 deletions(-)\r
  76 \r
  77 diff --git a/lib/message-file.c b/lib/message-file.c\r
  78 index eda1b748e202..a41d9ad2a719 100644\r
  79 --- a/lib/message-file.c\r
  80 +++ b/lib/message-file.c\r
  81 @@ -170,25 +170,12 @@ _notmuch_message_file_parse (notmuch_message_file_t *message)\r
  82         goto DONE;\r
  83      }\r
  84  \r
  85 -    if (is_mbox) {\r
  86 -       if (! g_mime_parser_eos (parser)) {\r
  87 -           /* This is a multi-message mbox. */\r
  88 -           status = NOTMUCH_STATUS_FILE_NOT_EMAIL;\r
  89 -           goto DONE;\r
  90 -       }\r
  91 +    if (is_mbox && ! g_mime_parser_eos (parser)) {\r
  92         /*\r
  93 -        * For historical reasons, we support single-message mboxes,\r
  94 -        * but this behavior is likely to change in the future, so\r
  95 -        * warn.\r
  96 +        * This is a multi-message mbox. (For historical reasons, we\r
  97 +        * do support single-message mboxes.)\r
  98          */\r
  99 -       static notmuch_bool_t mbox_warning = FALSE;\r
 100 -       if (! mbox_warning) {\r
 101 -           mbox_warning = TRUE;\r
 102 -           fprintf (stderr, "\\r
 103 -Warning: %s is an mbox containing a single message,\n\\r
 104 -likely caused by misconfigured mail delivery.  Support for single-message\n\\r
 105 -mboxes is deprecated and may be removed in the future.\n", message->filename);\r
 106 -       }\r
 107 +       status = NOTMUCH_STATUS_FILE_NOT_EMAIL;\r
 108      }\r
 109  \r
 110    DONE:\r
 111 diff --git a/test/T050-new.sh b/test/T050-new.sh\r
 112 index ad46ee6d51b6..7119356fc9fe 100755\r
 113 --- a/test/T050-new.sh\r
 114 +++ b/test/T050-new.sh\r
 115 @@ -173,11 +173,7 @@ Subject: Test mbox message 1\r
 116  Body.\r
 117  EOF\r
 118  output=$(NOTMUCH_NEW 2>&1)\r
 119 -test_expect_equal "$output" \\r
 120 -"Warning: ${MAIL_DIR}/mbox_file1 is an mbox containing a single message,\r
 121 -likely caused by misconfigured mail delivery.  Support for single-message\r
 122 -mboxes is deprecated and may be removed in the future.\r
 123 -Added 1 new message to the database."\r
 124 +test_expect_equal "$output" "Added 1 new message to the database."\r
 125  \r
 126  # This test requires that notmuch new has been run at least once.\r
 127  test_begin_subtest "Skip and report non-mail files"\r
 128 -- \r
 129 2.1.3\r
 130 \r