73/6b6593f20f98171bce2e057efbdc79839b376a

   1 Return-Path: <bremner@tesseract.cs.unb.ca>\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 arlo.cworth.org (Postfix) with ESMTP id 350C26DE0FF1\r
   6  for <notmuch@notmuchmail.org>; Mon,  3 Aug 2015 23:50:59 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0.146\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.146 tagged_above=-999 required=5 tests=[AWL=0.136, \r
  12  T_HEADER_FROM_DIFFERENT_DOMAINS=0.01] autolearn=disabled\r
  13 Received: from arlo.cworth.org ([127.0.0.1])\r
  14  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15  with ESMTP id bV9WZMaD5piQ for <notmuch@notmuchmail.org>;\r
  16  Mon,  3 Aug 2015 23:50:57 -0700 (PDT)\r
  17 Received: from gitolite.debian.net (gitolite.debian.net [87.98.215.224])\r
  18  by arlo.cworth.org (Postfix) with ESMTPS id 828E06DE1557\r
  19  for <notmuch@notmuchmail.org>; Mon,  3 Aug 2015 23:50:57 -0700 (PDT)\r
  20 Received: from remotemail by gitolite.debian.net with local (Exim 4.80)\r
  21  (envelope-from <bremner@tesseract.cs.unb.ca>)\r
  22  id 1ZMW2l-0000Qc-Pd; Tue, 04 Aug 2015 06:49:47 +0000\r
  23 Received: (nullmailer pid 17855 invoked by uid 1000); Tue, 04 Aug 2015\r
  24  06:48:55 -0000\r
  25 From: David Bremner <david@tethera.net>\r
  26 To: Tomi Ollila <tomi.ollila@iki.fi>, David Bremner <david@tethera.net>\r
  27 Subject: [PATCH 2/3] cli/new: add more debugging output\r
  28 Date: Tue,  4 Aug 2015 08:48:33 +0200\r
  29 Message-Id: <1438670914-17203-3-git-send-email-david@tethera.net>\r
  30 X-Mailer: git-send-email 2.1.4\r
  31 In-Reply-To: <1438670914-17203-1-git-send-email-david@tethera.net>\r
  32 References: <m2a8ua2gm9.fsf@guru.guru-group.fi>\r
  33  <1438670914-17203-1-git-send-email-david@tethera.net>\r
  34 Cc: notmuch@notmuchmail.org\r
  35 X-BeenThere: notmuch@notmuchmail.org\r
  36 X-Mailman-Version: 2.1.18\r
  37 Precedence: list\r
  38 List-Id: "Use and development of the notmuch mail system."\r
  39  <notmuch.notmuchmail.org>\r
  40 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  41  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  42 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  43 List-Post: <mailto:notmuch@notmuchmail.org>\r
  44 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  45 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  46  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  47 X-List-Received-Date: Tue, 04 Aug 2015 06:50:59 -0000\r
  48 \r
  49 Try to narrow down what part of the code adds files and directories to\r
  50 the queue(s) to be deleted.\r
  51 \r
  52 Update one test. The output is slightly confusing, but I believe it is\r
  53 correct, resulting from a directory being discovered but containing only ignored files.\r
  54 ---\r
  55  notmuch-new.c    | 14 ++++++++++++++\r
  56  test/T050-new.sh |  1 +\r
  57  2 files changed, 15 insertions(+)\r
  58 \r
  59 diff --git a/notmuch-new.c b/notmuch-new.c\r
  60 index 8ff1ade..ee786a3 100644\r
  61 --- a/notmuch-new.c\r
  62 +++ b/notmuch-new.c\r
  63 @@ -528,6 +528,10 @@ add_files (notmuch_database_t *notmuch,\r
  64                                               "%s/%s", path,\r
  65                                               notmuch_filenames_get (db_files));\r
  66  \r
  67 +           if (state->debug)\r
  68 +               printf ("(D) add_files_recursive, pass 2: queuing passed file %s for deletion from database\n",\r
  69 +                       absolute);\r
  70 +\r
  71             _filename_list_add (state->removed_files, absolute);\r
  72  \r
  73             notmuch_filenames_move_to_next (db_files);\r
  74 @@ -542,6 +546,9 @@ add_files (notmuch_database_t *notmuch,\r
  75             {\r
  76                 char *absolute = talloc_asprintf (state->removed_directories,\r
  77                                                   "%s/%s", path, filename);\r
  78 +               if (state->debug)\r
  79 +                   printf ("(D) add_files_recursive, pass 2: queuing passed directory %s for deletion from database\n",\r
  80 +                       absolute);\r
  81  \r
  82                 _filename_list_add (state->removed_directories, absolute);\r
  83             }\r
  84 @@ -610,6 +617,9 @@ add_files (notmuch_database_t *notmuch,\r
  85         char *absolute = talloc_asprintf (state->removed_files,\r
  86                                           "%s/%s", path,\r
  87                                           notmuch_filenames_get (db_files));\r
  88 +       if (state->debug)\r
  89 +           printf ("(D) add_files_recursive, pass 3: queuing leftover file %s for deletion from database\n",\r
  90 +                   absolute);\r
  91  \r
  92         _filename_list_add (state->removed_files, absolute);\r
  93  \r
  94 @@ -622,6 +632,10 @@ add_files (notmuch_database_t *notmuch,\r
  95                                           "%s/%s", path,\r
  96                                           notmuch_filenames_get (db_subdirs));\r
  97  \r
  98 +       if (state->debug)\r
  99 +           printf ("(D) add_files_recursive, pass 3: queuing leftover directory %s for deletion from database\n",\r
 100 +                   absolute);\r
 101 +\r
 102         _filename_list_add (state->removed_directories, absolute);\r
 103  \r
 104         notmuch_filenames_move_to_next (db_subdirs);\r
 105 diff --git a/test/T050-new.sh b/test/T050-new.sh\r
 106 index 66ea10f..ab15ca1 100755\r
 107 --- a/test/T050-new.sh\r
 108 +++ b/test/T050-new.sh\r
 109 @@ -238,6 +238,7 @@ test_expect_equal "$output" \\r
 110  (D) add_files_recursive, pass 2: explicitly ignoring ${MAIL_DIR}/one/two/ignored_file\r
 111  (D) add_files_recursive, pass 2: explicitly ignoring ${MAIL_DIR}/one/two/three/.git\r
 112  (D) add_files_recursive, pass 2: explicitly ignoring ${MAIL_DIR}/one/two/three/ignored_file\r
 113 +(D) add_files_recursive, pass 3: queuing leftover directory ${MAIL_DIR}/two for deletion from database\r
 114  No new mail."\r
 115  \r
 116  \r
 117 -- \r
 118 2.1.4\r
 119 \r