unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH v2 00/13] literal folder: prefix, new path: prefix
@ 2014-02-22 22:25 Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 01/13] lib: refactor folder term update after filename removal Jani Nikula
                   ` (14 more replies)
  0 siblings, 15 replies; 25+ messages in thread
From: Jani Nikula @ 2014-02-22 22:25 UTC (permalink / raw)
  To: notmuch

Hi all, this is v2 of id:cover.1389304779.git.jani@nikula.org.

The new path: prefix is a literal boolean prefix matching the paths,
relative from the maildir root, of the message files. There's no
interpretation of the maildir special cur/new folders, but a recursive
match is provided with "/**" suffix. See the patch for details.

The folder: prefix becomes a literal boolean prefix, similar to path:,
except it matches the maildir cur/new folders in addition to the
specified path. There's no recursive version.

Patches 1-5 add the above.

Patches 6-8 change the test infrastructure to make it easier to add
multiple corpuses, and adds a new test for path: and folder:.

Patches 9-11 add support for testing the database upgrade.

Patches 12-13 update man pages.


I've dropped most of the content in patches 7 and 10 due to their
size. The patches (and the whole series) are available in the
boolean-folder-and-path-v2 branch at
git://gitorious.org/jani/notmuch.git. Web interface at
https://gitorious.org/jani/notmuch/commits/0b3dd2d1cc6c413ea07ea326883ac448499c0e79.


WARNING! The change requires a database format version bump, and a
database upgrade, which is automatically done on 'notmuch new'. The
upgrade is irreversible if you want to try this on your database! A
complete database rebuild is required for reverting the database format
version. Make sure your backups are in order!


BR,
Jani.


Jani Nikula (13):
  lib: refactor folder term update after filename removal
  lib: add support for path: prefix searches
  test: make insert test use the path: prefix
  lib: make folder: prefix literal
  test: fix test for literal folder: search
  test: make it possible to have several corpora
  test: add new corpus with folders
  test: add tests for the new boolean folder: and path: prefixes
  devel: add script to generate test databases
  test: add test database in format version 1
  test: add database upgrade test from format version 1 to 2
  man: update man pages for folder: and path: search terms
  man: try to clarify the folder: and path: vs. --output=files confusion

 devel/gen-testdb.sh                    | 124 ++++++++++++
 lib/database.cc                        |  45 ++++-
 lib/message.cc                         | 249 ++++++++++++++++-------
 lib/notmuch-private.h                  |   3 +
 man/man1/notmuch-search.1              |  10 +-
 man/man7/notmuch-search-terms.7        |  28 ++-
 test/.gitignore                        |   2 +-
 test/Makefile.local                    |   2 +-
 test/T070-insert.sh                    |  10 +-
 test/T100-search-by-folder.sh          |  24 ++-
 test/T101-search-by-folder-and-path.sh |  83 ++++++++
 test/T480-hex-escaping.sh              |   4 +-
 test/T530-upgrade.sh                   | 103 ++++++++++
 test/corpus/{ => default}/cur/01:2,    |   0
 test/corpus/{ => default}/cur/02:2,    |   0
 test/corpus/{ => default}/cur/03:2,    |   0
 test/corpus/{ => default}/cur/04:2,    |   0
 test/corpus/{ => default}/cur/05:2,    |   0
 test/corpus/{ => default}/cur/06:2,    |   0
 test/corpus/{ => default}/cur/07:2,    |   0
 test/corpus/{ => default}/cur/08:2,    |   0
 test/corpus/{ => default}/cur/09:2,    |   0
 test/corpus/{ => default}/cur/10:2,    |   0
 test/corpus/{ => default}/cur/11:2,    |   0
 test/corpus/{ => default}/cur/12:2,    |   0
 test/corpus/{ => default}/cur/13:2,    |   0
 test/corpus/{ => default}/cur/14:2,    |   0
 test/corpus/{ => default}/cur/15:2,    |   0
 test/corpus/{ => default}/cur/16:2,    |   0
 test/corpus/{ => default}/cur/17:2,    |   0
 test/corpus/{ => default}/cur/18:2,    |   0
 test/corpus/{ => default}/cur/19:2,    |   0
 test/corpus/{ => default}/cur/20:2,    |   0
 test/corpus/{ => default}/cur/21:2,    |   0
 test/corpus/{ => default}/cur/22:2,    |   0
 test/corpus/{ => default}/cur/23:2,    |   0
 test/corpus/{ => default}/cur/24:2,    |   0
 test/corpus/{ => default}/cur/25:2,    |   0
 test/corpus/{ => default}/cur/26:2,    |   0
 test/corpus/{ => default}/cur/27:2,    |   0
 test/corpus/{ => default}/cur/28:2,    |   0
 test/corpus/{ => default}/cur/29:2,    |   0
 test/corpus/{ => default}/cur/30:2,    |   0
 test/corpus/{ => default}/cur/31:2,    |   0
 test/corpus/{ => default}/cur/32:2,    |   0
 test/corpus/{ => default}/cur/33:2,    |   0
 test/corpus/{ => default}/cur/34:2,    |   0
 test/corpus/{ => default}/cur/35:2,    |   0
 test/corpus/{ => default}/cur/36:2,    |   0
 test/corpus/{ => default}/cur/37:2,    |   0
 test/corpus/{ => default}/cur/38:2,    |   0
 test/corpus/{ => default}/cur/39:2,    |   0
 test/corpus/{ => default}/cur/40:2,    |   0
 test/corpus/{ => default}/cur/41:2,    |   0
 test/corpus/{ => default}/cur/42:2,    |   0
 test/corpus/{ => default}/cur/43:2,    |   0
 test/corpus/{ => default}/cur/44:2,    |   0
 test/corpus/{ => default}/cur/45:2,    |   0
 test/corpus/{ => default}/cur/46:2,    |   0
 test/corpus/{ => default}/cur/47:2,    |   0
 test/corpus/{ => default}/cur/48:2,    |   0
 test/corpus/{ => default}/cur/49:2,    |   0
 test/corpus/{ => default}/cur/50:2,    |   0
 test/corpus/{ => default}/cur/51:2,    |   0
 test/corpus/{ => default}/cur/52:2,    |   0
 test/corpus/{ => default}/cur/53:2,    |   0
 test/corpus/folders/01:2,              |  34 ++++
 test/corpus/folders/02:2,              |  32 +++
 test/corpus/folders/bar/17:2,          |  23 +++
 test/corpus/folders/bar/18:2,          |  12 ++
 test/corpus/folders/bar/baz/05:2,      | 104 ++++++++++
 test/corpus/folders/bar/baz/23:2,      | 145 +++++++++++++
 test/corpus/folders/bar/baz/24:2,      | 204 +++++++++++++++++++
 test/corpus/folders/bar/baz/cur/25:2,  |  32 +++
 test/corpus/folders/bar/baz/cur/26:2,  | 121 +++++++++++
 test/corpus/folders/bar/baz/new/27:2,  |  21 ++
 test/corpus/folders/bar/baz/new/28:2,  |  38 ++++
 test/corpus/folders/bar/cur/19:2,      | 360 +++++++++++++++++++++++++++++++++
 test/corpus/folders/bar/cur/20:2,      | 101 +++++++++
 test/corpus/folders/bar/new/21:2,      | 102 ++++++++++
 test/corpus/folders/bar/new/22:2,      |  84 ++++++++
 test/corpus/folders/cur/29:2,          |  21 ++
 test/corpus/folders/cur/30:2,          |  75 +++++++
 test/corpus/folders/cur/31:2,          |  31 +++
 test/corpus/folders/cur/32:2,          | 165 +++++++++++++++
 test/corpus/folders/cur/33:2,          |  13 ++
 test/corpus/folders/cur/34:2,          |  46 +++++
 test/corpus/folders/cur/35:2,          |  24 +++
 test/corpus/folders/cur/36:2,          |  25 +++
 test/corpus/folders/cur/37:2,          |  22 ++
 test/corpus/folders/cur/38:2,          |  40 ++++
 test/corpus/folders/cur/39:2,          |  32 +++
 test/corpus/folders/cur/40:2,          |  31 +++
 test/corpus/folders/cur/41:2,          |  37 ++++
 test/corpus/folders/cur/42:2,          |  30 +++
 test/corpus/folders/cur/43:2,          |  26 +++
 test/corpus/folders/cur/44:2,          |  29 +++
 test/corpus/folders/cur/45:2,          |  41 ++++
 test/corpus/folders/cur/46:2,          |  57 ++++++
 test/corpus/folders/cur/47:2,          |  84 ++++++++
 test/corpus/folders/cur/48:2,          |  17 ++
 test/corpus/folders/cur/49:2,          |  33 +++
 test/corpus/folders/cur/50:2,          |  39 ++++
 test/corpus/folders/cur/52:2,          |  39 ++++
 test/corpus/folders/cur/53:2,          |  20 ++
 test/corpus/folders/foo/05:2,          | 104 ++++++++++
 test/corpus/folders/foo/06:2,          |  36 ++++
 test/corpus/folders/foo/baz/11:2,      |  27 +++
 test/corpus/folders/foo/baz/12:2,      |  27 +++
 test/corpus/folders/foo/baz/cur/13:2,  | 178 ++++++++++++++++
 test/corpus/folders/foo/baz/cur/14:2,  |  39 ++++
 test/corpus/folders/foo/baz/new/15:2,  |  22 ++
 test/corpus/folders/foo/baz/new/16:2,  |  27 +++
 test/corpus/folders/foo/cur/07:2,      |  57 ++++++
 test/corpus/folders/foo/cur/08:2,      |  87 ++++++++
 test/corpus/folders/foo/new/03:2,      |  93 +++++++++
 test/corpus/folders/foo/new/09:2,      |  33 +++
 test/corpus/folders/foo/new/10:2,      |  54 +++++
 test/corpus/folders/new/03:2,          |  93 +++++++++
 test/corpus/folders/new/04:2,          |  84 ++++++++
 test/notmuch-test                      |   2 +-
 test/test-databases/README             |   5 +
 test/test-databases/database-v1.tar.gz | Bin 0 -> 262063 bytes
 test/test-lib.sh                       |  21 +-
 124 files changed, 3954 insertions(+), 112 deletions(-)
 create mode 100755 devel/gen-testdb.sh
 create mode 100755 test/T101-search-by-folder-and-path.sh
 create mode 100755 test/T530-upgrade.sh
 rename test/corpus/{ => default}/cur/01:2, (100%)
 rename test/corpus/{ => default}/cur/02:2, (100%)
 rename test/corpus/{ => default}/cur/03:2, (100%)
 rename test/corpus/{ => default}/cur/04:2, (100%)
 rename test/corpus/{ => default}/cur/05:2, (100%)
 rename test/corpus/{ => default}/cur/06:2, (100%)
 rename test/corpus/{ => default}/cur/07:2, (100%)
 rename test/corpus/{ => default}/cur/08:2, (100%)
 rename test/corpus/{ => default}/cur/09:2, (100%)
 rename test/corpus/{ => default}/cur/10:2, (100%)
 rename test/corpus/{ => default}/cur/11:2, (100%)
 rename test/corpus/{ => default}/cur/12:2, (100%)
 rename test/corpus/{ => default}/cur/13:2, (100%)
 rename test/corpus/{ => default}/cur/14:2, (100%)
 rename test/corpus/{ => default}/cur/15:2, (100%)
 rename test/corpus/{ => default}/cur/16:2, (100%)
 rename test/corpus/{ => default}/cur/17:2, (100%)
 rename test/corpus/{ => default}/cur/18:2, (100%)
 rename test/corpus/{ => default}/cur/19:2, (100%)
 rename test/corpus/{ => default}/cur/20:2, (100%)
 rename test/corpus/{ => default}/cur/21:2, (100%)
 rename test/corpus/{ => default}/cur/22:2, (100%)
 rename test/corpus/{ => default}/cur/23:2, (100%)
 rename test/corpus/{ => default}/cur/24:2, (100%)
 rename test/corpus/{ => default}/cur/25:2, (100%)
 rename test/corpus/{ => default}/cur/26:2, (100%)
 rename test/corpus/{ => default}/cur/27:2, (100%)
 rename test/corpus/{ => default}/cur/28:2, (100%)
 rename test/corpus/{ => default}/cur/29:2, (100%)
 rename test/corpus/{ => default}/cur/30:2, (100%)
 rename test/corpus/{ => default}/cur/31:2, (100%)
 rename test/corpus/{ => default}/cur/32:2, (100%)
 rename test/corpus/{ => default}/cur/33:2, (100%)
 rename test/corpus/{ => default}/cur/34:2, (100%)
 rename test/corpus/{ => default}/cur/35:2, (100%)
 rename test/corpus/{ => default}/cur/36:2, (100%)
 rename test/corpus/{ => default}/cur/37:2, (100%)
 rename test/corpus/{ => default}/cur/38:2, (100%)
 rename test/corpus/{ => default}/cur/39:2, (100%)
 rename test/corpus/{ => default}/cur/40:2, (100%)
 rename test/corpus/{ => default}/cur/41:2, (100%)
 rename test/corpus/{ => default}/cur/42:2, (100%)
 rename test/corpus/{ => default}/cur/43:2, (100%)
 rename test/corpus/{ => default}/cur/44:2, (100%)
 rename test/corpus/{ => default}/cur/45:2, (100%)
 rename test/corpus/{ => default}/cur/46:2, (100%)
 rename test/corpus/{ => default}/cur/47:2, (100%)
 rename test/corpus/{ => default}/cur/48:2, (100%)
 rename test/corpus/{ => default}/cur/49:2, (100%)
 rename test/corpus/{ => default}/cur/50:2, (100%)
 rename test/corpus/{ => default}/cur/51:2, (100%)
 rename test/corpus/{ => default}/cur/52:2, (100%)
 rename test/corpus/{ => default}/cur/53:2, (100%)
 create mode 100644 test/corpus/folders/01:2,
 create mode 100644 test/corpus/folders/02:2,
 create mode 100644 test/corpus/folders/bar/17:2,
 create mode 100644 test/corpus/folders/bar/18:2,
 create mode 100644 test/corpus/folders/bar/baz/05:2,
 create mode 100644 test/corpus/folders/bar/baz/23:2,
 create mode 100644 test/corpus/folders/bar/baz/24:2,
 create mode 100644 test/corpus/folders/bar/baz/cur/25:2,
 create mode 100644 test/corpus/folders/bar/baz/cur/26:2,
 create mode 100644 test/corpus/folders/bar/baz/new/27:2,
 create mode 100644 test/corpus/folders/bar/baz/new/28:2,
 create mode 100644 test/corpus/folders/bar/cur/19:2,
 create mode 100644 test/corpus/folders/bar/cur/20:2,
 create mode 100644 test/corpus/folders/bar/new/21:2,
 create mode 100644 test/corpus/folders/bar/new/22:2,
 create mode 100644 test/corpus/folders/cur/29:2,
 create mode 100644 test/corpus/folders/cur/30:2,
 create mode 100644 test/corpus/folders/cur/31:2,
 create mode 100644 test/corpus/folders/cur/32:2,
 create mode 100644 test/corpus/folders/cur/33:2,
 create mode 100644 test/corpus/folders/cur/34:2,
 create mode 100644 test/corpus/folders/cur/35:2,
 create mode 100644 test/corpus/folders/cur/36:2,
 create mode 100644 test/corpus/folders/cur/37:2,
 create mode 100644 test/corpus/folders/cur/38:2,
 create mode 100644 test/corpus/folders/cur/39:2,
 create mode 100644 test/corpus/folders/cur/40:2,
 create mode 100644 test/corpus/folders/cur/41:2,
 create mode 100644 test/corpus/folders/cur/42:2,
 create mode 100644 test/corpus/folders/cur/43:2,
 create mode 100644 test/corpus/folders/cur/44:2,
 create mode 100644 test/corpus/folders/cur/45:2,
 create mode 100644 test/corpus/folders/cur/46:2,
 create mode 100644 test/corpus/folders/cur/47:2,
 create mode 100644 test/corpus/folders/cur/48:2,
 create mode 100644 test/corpus/folders/cur/49:2,
 create mode 100644 test/corpus/folders/cur/50:2,
 create mode 100644 test/corpus/folders/cur/52:2,
 create mode 100644 test/corpus/folders/cur/53:2,
 create mode 100644 test/corpus/folders/foo/05:2,
 create mode 100644 test/corpus/folders/foo/06:2,
 create mode 100644 test/corpus/folders/foo/baz/11:2,
 create mode 100644 test/corpus/folders/foo/baz/12:2,
 create mode 100644 test/corpus/folders/foo/baz/cur/13:2,
 create mode 100644 test/corpus/folders/foo/baz/cur/14:2,
 create mode 100644 test/corpus/folders/foo/baz/new/15:2,
 create mode 100644 test/corpus/folders/foo/baz/new/16:2,
 create mode 100644 test/corpus/folders/foo/cur/07:2,
 create mode 100644 test/corpus/folders/foo/cur/08:2,
 create mode 100644 test/corpus/folders/foo/new/03:2,
 create mode 100644 test/corpus/folders/foo/new/09:2,
 create mode 100644 test/corpus/folders/foo/new/10:2,
 create mode 100644 test/corpus/folders/new/03:2,
 create mode 100644 test/corpus/folders/new/04:2,
 create mode 100644 test/test-databases/README
 create mode 100644 test/test-databases/database-v1.tar.gz

-- 
1.8.5.3

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [PATCH v2 01/13] lib: refactor folder term update after filename removal
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
@ 2014-02-22 22:25 ` Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 02/13] lib: add support for path: prefix searches Jani Nikula
                   ` (13 subsequent siblings)
  14 siblings, 0 replies; 25+ messages in thread
From: Jani Nikula @ 2014-02-22 22:25 UTC (permalink / raw)
  To: notmuch

Abstract some blocks of code for reuse. No functional changes.
---
 lib/message.cc | 135 ++++++++++++++++++++++++++++-----------------------------
 1 file changed, 66 insertions(+), 69 deletions(-)

diff --git a/lib/message.cc b/lib/message.cc
index c91f3a59836f..7aff4ae5111a 100644
--- a/lib/message.cc
+++ b/lib/message.cc
@@ -481,6 +481,68 @@ notmuch_message_get_replies (notmuch_message_t *message)
     return _notmuch_messages_create (message->replies);
 }
 
+static void
+_notmuch_message_remove_terms (notmuch_message_t *message, const char *prefix)
+{
+    Xapian::TermIterator i;
+    size_t prefix_len = strlen (prefix);
+
+    while (1) {
+	i = message->doc.termlist_begin ();
+	i.skip_to (prefix);
+
+	/* Terminate loop when no terms remain with desired prefix. */
+	if (i == message->doc.termlist_end () ||
+	    strncmp ((*i).c_str (), prefix, prefix_len))
+	    break;
+
+	try {
+	    message->doc.remove_term ((*i));
+	} catch (const Xapian::InvalidArgumentError) {
+	    /* Ignore failure to remove non-existent term. */
+	}
+    }
+}
+
+/* Add directory based terms for all filenames of the message. */
+static notmuch_status_t
+_notmuch_message_add_directory_terms (void *ctx, notmuch_message_t *message)
+{
+    const char *direntry_prefix = _find_prefix ("file-direntry");
+    int direntry_prefix_len = strlen (direntry_prefix);
+    Xapian::TermIterator i = message->doc.termlist_begin ();
+    notmuch_status_t status = NOTMUCH_STATUS_SUCCESS;
+
+    for (i.skip_to (direntry_prefix); i != message->doc.termlist_end (); i++) {
+	unsigned int directory_id;
+	const char *direntry, *directory;
+	char *colon;
+
+	/* Terminate loop at first term without desired prefix. */
+	if (strncmp ((*i).c_str (), direntry_prefix, direntry_prefix_len))
+	    break;
+
+	/* Indicate that there are filenames remaining. */
+	status = NOTMUCH_STATUS_DUPLICATE_MESSAGE_ID;
+
+	direntry = (*i).c_str ();
+	direntry += direntry_prefix_len;
+
+	directory_id = strtol (direntry, &colon, 10);
+
+	if (colon == NULL || *colon != ':')
+	    INTERNAL_ERROR ("malformed direntry");
+
+	directory = _notmuch_database_get_directory_path (ctx,
+							  message->notmuch,
+							  directory_id);
+	if (strlen (directory))
+	    _notmuch_message_gen_terms (message, "folder", directory);
+    }
+
+    return status;
+}
+
 /* Add an additional 'filename' for 'message'.
  *
  * This change will not be reflected in the database until the next
@@ -536,17 +598,12 @@ notmuch_status_t
 _notmuch_message_remove_filename (notmuch_message_t *message,
 				  const char *filename)
 {
-    const char *direntry_prefix = _find_prefix ("file-direntry");
-    int direntry_prefix_len = strlen (direntry_prefix);
-    const char *folder_prefix = _find_prefix ("folder");
-    int folder_prefix_len = strlen (folder_prefix);
     void *local = talloc_new (message);
+    const char *folder_prefix = _find_prefix ("folder");
     char *zfolder_prefix = talloc_asprintf(local, "Z%s", folder_prefix);
-    int zfolder_prefix_len = strlen (zfolder_prefix);
     char *direntry;
     notmuch_private_status_t private_status;
     notmuch_status_t status;
-    Xapian::TermIterator i, last;
 
     status = _notmuch_database_filename_to_direntry (
 	local, message->notmuch, filename, NOTMUCH_FIND_LOOKUP, &direntry);
@@ -567,73 +624,13 @@ _notmuch_message_remove_filename (notmuch_message_t *message,
      *  3. adding back terms for all remaining filenames of the message. */
 
     /* 1. removing all "folder:" terms */
-    while (1) {
-	i = message->doc.termlist_begin ();
-	i.skip_to (folder_prefix);
-
-	/* Terminate loop when no terms remain with desired prefix. */
-	if (i == message->doc.termlist_end () ||
-	    strncmp ((*i).c_str (), folder_prefix, folder_prefix_len))
-	{
-	    break;
-	}
-
-	try {
-	    message->doc.remove_term ((*i));
-	} catch (const Xapian::InvalidArgumentError) {
-	    /* Ignore failure to remove non-existent term. */
-	}
-    }
+    _notmuch_message_remove_terms (message, folder_prefix);
 
     /* 2. removing all "folder:" stemmed terms */
-    while (1) {
-	i = message->doc.termlist_begin ();
-	i.skip_to (zfolder_prefix);
-
-	/* Terminate loop when no terms remain with desired prefix. */
-	if (i == message->doc.termlist_end () ||
-	    strncmp ((*i).c_str (), zfolder_prefix, zfolder_prefix_len))
-	{
-	    break;
-	}
-
-	try {
-	    message->doc.remove_term ((*i));
-	} catch (const Xapian::InvalidArgumentError) {
-	    /* Ignore failure to remove non-existent term. */
-	}
-    }
+    _notmuch_message_remove_terms (message, zfolder_prefix);
 
     /* 3. adding back terms for all remaining filenames of the message. */
-    i = message->doc.termlist_begin ();
-    i.skip_to (direntry_prefix);
-
-    for (; i != message->doc.termlist_end (); i++) {
-	unsigned int directory_id;
-	const char *direntry, *directory;
-	char *colon;
-
-	/* Terminate loop at first term without desired prefix. */
-	if (strncmp ((*i).c_str (), direntry_prefix, direntry_prefix_len))
-	    break;
-
-	/* Indicate that there are filenames remaining. */
-	status = NOTMUCH_STATUS_DUPLICATE_MESSAGE_ID;
-
-	direntry = (*i).c_str ();
-	direntry += direntry_prefix_len;
-
-	directory_id = strtol (direntry, &colon, 10);
-
-	if (colon == NULL || *colon != ':')
-	    INTERNAL_ERROR ("malformed direntry");
-
-	directory = _notmuch_database_get_directory_path (local,
-							  message->notmuch,
-							  directory_id);
-	if (strlen (directory))
-	    _notmuch_message_gen_terms (message, "folder", directory);
-    }
+    status = _notmuch_message_add_directory_terms (local, message);
 
     talloc_free (local);
 
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH v2 02/13] lib: add support for path: prefix searches
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 01/13] lib: refactor folder term update after filename removal Jani Nikula
@ 2014-02-22 22:25 ` Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 03/13] test: make insert test use the path: prefix Jani Nikula
                   ` (12 subsequent siblings)
  14 siblings, 0 replies; 25+ messages in thread
From: Jani Nikula @ 2014-02-22 22:25 UTC (permalink / raw)
  To: notmuch

The path: prefix is a literal boolean prefix matching the paths,
relative from the maildir root, of the message files.

path:foo matches all message files in foo (but not in foo/new or
foo/cur).

path:foo/new matches all message files in foo/new.

path:"" matches all message files in the top level maildir.

path:foo/** matches all message files in foo and recursively in all
subdirectories of foo.

path:** matches all message files recursively, i.e. all messages.
---
 lib/database.cc |  7 ++++---
 lib/message.cc  | 52 +++++++++++++++++++++++++++++++++++++++++++++-------
 2 files changed, 49 insertions(+), 10 deletions(-)

diff --git a/lib/database.cc b/lib/database.cc
index f395061e3a73..93cc7f57e9db 100644
--- a/lib/database.cc
+++ b/lib/database.cc
@@ -100,8 +100,8 @@ typedef struct {
  * In addition, terms from the content of the message are added with
  * "from", "to", "attachment", and "subject" prefixes for use by the
  * user in searching. Similarly, terms from the path of the mail
- * message are added with a "folder" prefix. But the database doesn't
- * really care itself about any of these.
+ * message are added with "folder" and "path" prefixes. But the
+ * database doesn't really care itself about any of these.
  *
  * The data portion of a mail document is empty.
  *
@@ -208,7 +208,8 @@ static prefix_t BOOLEAN_PREFIX_EXTERNAL[] = {
     { "thread",			"G" },
     { "tag",			"K" },
     { "is",			"K" },
-    { "id",			"Q" }
+    { "id",			"Q" },
+    { "path",			"P" },
 };
 
 static prefix_t PROBABILISTIC_PREFIX[]= {
diff --git a/lib/message.cc b/lib/message.cc
index 7aff4ae5111a..21abe8e12b9d 100644
--- a/lib/message.cc
+++ b/lib/message.cc
@@ -504,6 +504,40 @@ _notmuch_message_remove_terms (notmuch_message_t *message, const char *prefix)
     }
 }
 
+#define RECURSIVE_SUFFIX "/**"
+
+/* Add "path:" terms for directory. */
+static notmuch_status_t
+_notmuch_message_add_path_terms (notmuch_message_t *message,
+				 const char *directory)
+{
+    /* Add exact "path:" term. */
+    _notmuch_message_add_term (message, "path", directory);
+
+    if (strlen (directory)) {
+	char *path, *p;
+
+	path = talloc_asprintf (NULL, "%s%s", directory, RECURSIVE_SUFFIX);
+	if (! path)
+	    return NOTMUCH_STATUS_OUT_OF_MEMORY;
+
+	/* Add recursive "path:" terms for directory and all parents. */
+	for (p = path + strlen (path) - 1; p > path; p--) {
+	    if (*p == '/') {
+		strcpy (p, RECURSIVE_SUFFIX);
+		_notmuch_message_add_term (message, "path", path);
+	    }
+	}
+
+	talloc_free (path);
+    }
+
+    /* Recursive all-matching path:** for consistency. */
+    _notmuch_message_add_term (message, "path", "**");
+
+    return NOTMUCH_STATUS_SUCCESS;
+}
+
 /* Add directory based terms for all filenames of the message. */
 static notmuch_status_t
 _notmuch_message_add_directory_terms (void *ctx, notmuch_message_t *message)
@@ -538,6 +572,8 @@ _notmuch_message_add_directory_terms (void *ctx, notmuch_message_t *message)
 							  directory_id);
 	if (strlen (directory))
 	    _notmuch_message_gen_terms (message, "folder", directory);
+
+	_notmuch_message_add_path_terms (message, directory);
     }
 
     return status;
@@ -577,6 +613,8 @@ _notmuch_message_add_filename (notmuch_message_t *message,
     /* New terms allow user to search with folder: specification. */
     _notmuch_message_gen_terms (message, "folder", directory);
 
+    _notmuch_message_add_path_terms (message, directory);
+
     talloc_free (local);
 
     return NOTMUCH_STATUS_SUCCESS;
@@ -618,18 +656,18 @@ _notmuch_message_remove_filename (notmuch_message_t *message,
     if (status)
 	return status;
 
-    /* Re-synchronize "folder:" terms for this message. This requires:
-     *  1. removing all "folder:" terms
-     *  2. removing all "folder:" stemmed terms
-     *  3. adding back terms for all remaining filenames of the message. */
+    /* Re-synchronize "folder:" and "path:" terms for this message. */
 
-    /* 1. removing all "folder:" terms */
+    /* Remove all "folder:" terms. */
     _notmuch_message_remove_terms (message, folder_prefix);
 
-    /* 2. removing all "folder:" stemmed terms */
+    /* Remove all "folder:" stemmed terms. */
     _notmuch_message_remove_terms (message, zfolder_prefix);
 
-    /* 3. adding back terms for all remaining filenames of the message. */
+    /* Remove all "path:" terms. */
+    _notmuch_message_remove_terms (message, _find_prefix ("path"));
+
+    /* Add back terms for all remaining filenames of the message. */
     status = _notmuch_message_add_directory_terms (local, message);
 
     talloc_free (local);
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH v2 03/13] test: make insert test use the path: prefix
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 01/13] lib: refactor folder term update after filename removal Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 02/13] lib: add support for path: prefix searches Jani Nikula
@ 2014-02-22 22:25 ` Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 04/13] lib: make folder: prefix literal Jani Nikula
                   ` (11 subsequent siblings)
  14 siblings, 0 replies; 25+ messages in thread
From: Jani Nikula @ 2014-02-22 22:25 UTC (permalink / raw)
  To: notmuch

This is a more strict test for the insert test.
---
 test/T070-insert.sh | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/test/T070-insert.sh b/test/T070-insert.sh
index e8dc4c099ed1..5de5da43cebb 100755
--- a/test/T070-insert.sh
+++ b/test/T070-insert.sh
@@ -126,14 +126,14 @@ test_expect_equal "$dirname" "$MAIL_DIR/new"
 test_begin_subtest "Insert message into folder"
 gen_insert_msg
 notmuch insert --folder=Drafts < "$gen_msg_filename"
-output=$(notmuch search --output=files folder:Drafts)
+output=$(notmuch search --output=files path:Drafts/new)
 dirname=$(dirname "$output")
 test_expect_equal "$dirname" "$MAIL_DIR/Drafts/new"
 
 test_begin_subtest "Insert message into folder, add/remove tags"
 gen_insert_msg
 notmuch insert --folder=Drafts +draft -unread < "$gen_msg_filename"
-output=$(notmuch search --output=messages folder:Drafts tag:draft NOT tag:unread)
+output=$(notmuch search --output=messages path:Drafts/cur tag:draft NOT tag:unread)
 test_expect_equal "$output" "id:$gen_msg_id"
 
 gen_insert_msg
@@ -143,21 +143,21 @@ test_expect_code 1 "Insert message into non-existent folder" \
 test_begin_subtest "Insert message, create folder"
 gen_insert_msg
 notmuch insert --folder=F --create-folder +folder < "$gen_msg_filename"
-output=$(notmuch search --output=files folder:F tag:folder)
+output=$(notmuch search --output=files path:F/new tag:folder)
 basename=$(basename "$output")
 test_expect_equal_file "$gen_msg_filename" "$MAIL_DIR/F/new/${basename}"
 
 test_begin_subtest "Insert message, create subfolder"
 gen_insert_msg
 notmuch insert --folder=F/G/H/I/J --create-folder +folder < "$gen_msg_filename"
-output=$(notmuch search --output=files folder:F/G/H/I/J tag:folder)
+output=$(notmuch search --output=files path:F/G/H/I/J/new tag:folder)
 basename=$(basename "$output")
 test_expect_equal_file "$gen_msg_filename" "${MAIL_DIR}/F/G/H/I/J/new/${basename}"
 
 test_begin_subtest "Insert message, create existing subfolder"
 gen_insert_msg
 notmuch insert --folder=F/G/H/I/J --create-folder +folder < "$gen_msg_filename"
-output=$(notmuch count folder:F/G/H/I/J tag:folder)
+output=$(notmuch count path:F/G/H/I/J/new tag:folder)
 test_expect_equal "$output" "2"
 
 gen_insert_msg
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH v2 04/13] lib: make folder: prefix literal
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
                   ` (2 preceding siblings ...)
  2014-02-22 22:25 ` [PATCH v2 03/13] test: make insert test use the path: prefix Jani Nikula
@ 2014-02-22 22:25 ` Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 05/13] test: fix test for literal folder: search Jani Nikula
                   ` (10 subsequent siblings)
  14 siblings, 0 replies; 25+ messages in thread
From: Jani Nikula @ 2014-02-22 22:25 UTC (permalink / raw)
  To: notmuch

In xapian terms, convert folder: prefix from probabilistic to boolean
prefix, matching the paths, relative form the maildir root, of the
message files, ignoring the maildir new and cur leaf directories.

folder:foo matches all message files in foo, foo/new, and foo/cur.

folder:foo/new does *not* match message files in foo/new.

folder:"" matches all message files in the top level maildir and its
new and cur subdirectories.

This change constitutes a database change: bump the database version
and add database upgrade support for folder: terms. The upgrade also
adds path: terms.
---
 lib/database.cc       | 38 ++++++++++++++++++++++--
 lib/message.cc        | 80 ++++++++++++++++++++++++++++++++++++++++++++-------
 lib/notmuch-private.h |  3 ++
 3 files changed, 108 insertions(+), 13 deletions(-)

diff --git a/lib/database.cc b/lib/database.cc
index 93cc7f57e9db..186e3a7976fe 100644
--- a/lib/database.cc
+++ b/lib/database.cc
@@ -42,7 +42,7 @@ typedef struct {
     const char *prefix;
 } prefix_t;
 
-#define NOTMUCH_DATABASE_VERSION 1
+#define NOTMUCH_DATABASE_VERSION 2
 
 #define STRINGIFY(s) _SUB_STRINGIFY(s)
 #define _SUB_STRINGIFY(s) #s
@@ -210,6 +210,7 @@ static prefix_t BOOLEAN_PREFIX_EXTERNAL[] = {
     { "is",			"K" },
     { "id",			"Q" },
     { "path",			"P" },
+    { "folder",			"XFOLDER:" },
 };
 
 static prefix_t PROBABILISTIC_PREFIX[]= {
@@ -217,7 +218,6 @@ static prefix_t PROBABILISTIC_PREFIX[]= {
     { "to",			"XTO" },
     { "attachment",		"XATTACHMENT" },
     { "subject",		"XSUBJECT"},
-    { "folder",			"XFOLDER"}
 };
 
 const char *
@@ -1168,6 +1168,40 @@ notmuch_database_upgrade (notmuch_database_t *notmuch,
 	}
     }
 
+    /*
+     * Prior to version 2, the "folder:" prefix was probabilistic and
+     * stemmed. Change it to the current boolean prefix. Add "path:"
+     * prefixes while at it.
+     */
+    if (version < 2) {
+       notmuch_query_t *query = notmuch_query_create (notmuch, "");
+       notmuch_messages_t *messages;
+       notmuch_message_t *message;
+
+       count = 0;
+       total = notmuch_query_count_messages (query);
+
+       for (messages = notmuch_query_search_messages (query);
+            notmuch_messages_valid (messages);
+            notmuch_messages_move_to_next (messages)) {
+           if (do_progress_notify) {
+               progress_notify (closure, (double) count / total);
+               do_progress_notify = 0;
+           }
+
+           message = notmuch_messages_get (messages);
+
+           _notmuch_message_upgrade_folder (message);
+           _notmuch_message_sync (message);
+
+           notmuch_message_destroy (message);
+
+           count++;
+       }
+
+       notmuch_query_destroy (query);
+    }
+
     db->set_metadata ("version", STRINGIFY (NOTMUCH_DATABASE_VERSION));
     db->flush ();
 
diff --git a/lib/message.cc b/lib/message.cc
index 21abe8e12b9d..31cb9f107dd7 100644
--- a/lib/message.cc
+++ b/lib/message.cc
@@ -504,6 +504,56 @@ _notmuch_message_remove_terms (notmuch_message_t *message, const char *prefix)
     }
 }
 
+/* Return true if p points at "new" or "cur". */
+static bool is_maildir (const char *p)
+{
+    return strcmp (p, "cur") == 0 || strcmp (p, "new") == 0;
+}
+
+/* Add "folder:" term for directory. */
+static notmuch_status_t
+_notmuch_message_add_folder_terms (notmuch_message_t *message,
+				   const char *directory)
+{
+    char *folder, *last;
+
+    folder = talloc_strdup (NULL, directory);
+    if (! folder)
+       return NOTMUCH_STATUS_OUT_OF_MEMORY;
+
+    /*
+     * If the message file is in a leaf directory named "new" or
+     * "cur", presume maildir and index the parent directory. Thus a
+     * "folder:" prefix search matches messages in the specified
+     * maildir folder, i.e. in the specified directory and its "new"
+     * and "cur" subdirectories.
+     *
+     * Note that this means the "folder:" prefix can't be used for
+     * distinguishing between message files in "new" or "cur". The
+     * "path:" prefix needs to be used for that.
+     *
+     * Note the deliberate difference to _filename_is_in_maildir(). We
+     * don't want to index different things depending on the existence
+     * or non-existence of all maildir sibling directories "new",
+     * "cur", and "tmp". Doing so would be surprising, and difficult
+     * for the user to fix in case all subdirectories were not in
+     * place during indexing.
+     */
+    last = strrchr (folder, '/');
+    if (last) {
+       if (is_maildir (last + 1))
+           *last = '\0';
+    } else if (is_maildir (folder)) {
+       *folder = '\0';
+    }
+
+    _notmuch_message_add_term (message, "folder", folder);
+
+    talloc_free (folder);
+
+    return NOTMUCH_STATUS_SUCCESS;
+}
+
 #define RECURSIVE_SUFFIX "/**"
 
 /* Add "path:" terms for directory. */
@@ -570,9 +620,8 @@ _notmuch_message_add_directory_terms (void *ctx, notmuch_message_t *message)
 	directory = _notmuch_database_get_directory_path (ctx,
 							  message->notmuch,
 							  directory_id);
-	if (strlen (directory))
-	    _notmuch_message_gen_terms (message, "folder", directory);
 
+	_notmuch_message_add_folder_terms (message, directory);
 	_notmuch_message_add_path_terms (message, directory);
     }
 
@@ -610,9 +659,7 @@ _notmuch_message_add_filename (notmuch_message_t *message,
      * notmuch_directory_get_child_files() . */
     _notmuch_message_add_term (message, "file-direntry", direntry);
 
-    /* New terms allow user to search with folder: specification. */
-    _notmuch_message_gen_terms (message, "folder", directory);
-
+    _notmuch_message_add_folder_terms (message, directory);
     _notmuch_message_add_path_terms (message, directory);
 
     talloc_free (local);
@@ -637,8 +684,6 @@ _notmuch_message_remove_filename (notmuch_message_t *message,
 				  const char *filename)
 {
     void *local = talloc_new (message);
-    const char *folder_prefix = _find_prefix ("folder");
-    char *zfolder_prefix = talloc_asprintf(local, "Z%s", folder_prefix);
     char *direntry;
     notmuch_private_status_t private_status;
     notmuch_status_t status;
@@ -659,10 +704,7 @@ _notmuch_message_remove_filename (notmuch_message_t *message,
     /* Re-synchronize "folder:" and "path:" terms for this message. */
 
     /* Remove all "folder:" terms. */
-    _notmuch_message_remove_terms (message, folder_prefix);
-
-    /* Remove all "folder:" stemmed terms. */
-    _notmuch_message_remove_terms (message, zfolder_prefix);
+    _notmuch_message_remove_terms (message, _find_prefix ("folder"));
 
     /* Remove all "path:" terms. */
     _notmuch_message_remove_terms (message, _find_prefix ("path"));
@@ -675,6 +717,22 @@ _notmuch_message_remove_filename (notmuch_message_t *message,
     return status;
 }
 
+/* Upgrade the "folder:" prefix from V1 to V2. */
+#define FOLDER_PREFIX_V1       "XFOLDER"
+#define ZFOLDER_PREFIX_V1      "Z" FOLDER_PREFIX_V1
+void
+_notmuch_message_upgrade_folder (notmuch_message_t *message)
+{
+    /* Remove all old "folder:" terms. */
+    _notmuch_message_remove_terms (message, FOLDER_PREFIX_V1);
+
+    /* Remove all old "folder:" stemmed terms. */
+    _notmuch_message_remove_terms (message, ZFOLDER_PREFIX_V1);
+
+    /* Add new boolean "folder:" and "path:" terms. */
+    _notmuch_message_add_directory_terms (message, message);
+}
+
 char *
 _notmuch_message_talloc_copy_data (notmuch_message_t *message)
 {
diff --git a/lib/notmuch-private.h b/lib/notmuch-private.h
index af185c7c5ba8..59eb2bc285a5 100644
--- a/lib/notmuch-private.h
+++ b/lib/notmuch-private.h
@@ -263,6 +263,9 @@ _notmuch_message_gen_terms (notmuch_message_t *message,
 void
 _notmuch_message_upgrade_filename_storage (notmuch_message_t *message);
 
+void
+_notmuch_message_upgrade_folder (notmuch_message_t *message);
+
 notmuch_status_t
 _notmuch_message_add_filename (notmuch_message_t *message,
 			       const char *filename);
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH v2 05/13] test: fix test for literal folder: search
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
                   ` (3 preceding siblings ...)
  2014-02-22 22:25 ` [PATCH v2 04/13] lib: make folder: prefix literal Jani Nikula
@ 2014-02-22 22:25 ` Jani Nikula
  2014-02-23 14:02   ` David Bremner
  2014-02-22 22:25 ` [PATCH v2 06/13] test: make it possible to have several corpora Jani Nikula
                   ` (9 subsequent siblings)
  14 siblings, 1 reply; 25+ messages in thread
From: Jani Nikula @ 2014-02-22 22:25 UTC (permalink / raw)
  To: notmuch

Some of the folder: matching capabilities are lost in the
probabilistic to boolean prefix change. Fix them.
---
 test/T100-search-by-folder.sh | 24 +++++++++++++++++++++---
 1 file changed, 21 insertions(+), 3 deletions(-)

diff --git a/test/T100-search-by-folder.sh b/test/T100-search-by-folder.sh
index 5cc2ca8d388a..84ca43820031 100755
--- a/test/T100-search-by-folder.sh
+++ b/test/T100-search-by-folder.sh
@@ -3,6 +3,7 @@ test_description='"notmuch search" by folder: (with variations)'
 . ./test-lib.sh
 
 add_message '[dir]=bad' '[subject]="To the bone"'
+add_message '[dir]=.' '[subject]="Top level"'
 add_message '[dir]=bad/news' '[subject]="Bears"'
 mkdir -p "${MAIL_DIR}/duplicate/bad/news"
 cp "$gen_msg_filename" "${MAIL_DIR}/duplicate/bad/news"
@@ -12,29 +13,46 @@ add_message '[dir]=things/favorite' '[subject]="Raindrops, whiskers, kettles"'
 add_message '[dir]=things/bad' '[subject]="Bites, stings, sad feelings"'
 
 test_begin_subtest "Single-world folder: specification (multiple results)"
-output=$(notmuch search folder:bad | notmuch_search_sanitize)
+output=$(notmuch search folder:bad folder:bad/news folder:things/bad | notmuch_search_sanitize)
 test_expect_equal "$output" "thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; To the bone (inbox unread)
 thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; Bears (inbox unread)
 thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; Bites, stings, sad feelings (inbox unread)"
 
+test_begin_subtest "Top level folder"
+output=$(notmuch search folder:'""' | notmuch_search_sanitize)
+test_expect_equal "$output" "thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; Top level (inbox unread)"
+
 test_begin_subtest "Two-word path to narrow results to one"
 output=$(notmuch search folder:bad/news | notmuch_search_sanitize)
 test_expect_equal "$output" "thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; Bears (inbox unread)"
 
+test_begin_subtest "Folder search with --output=files"
+output=$(notmuch search --output=files folder:bad/news | sed -e "s,$MAIL_DIR,MAIL_DIR,")
+test_expect_equal "$output" "MAIL_DIR/bad/news/msg-003
+MAIL_DIR/duplicate/bad/news/msg-003"
+
 test_begin_subtest "After removing duplicate instance of matching path"
 rm -r "${MAIL_DIR}/bad/news"
 notmuch new
 output=$(notmuch search folder:bad/news | notmuch_search_sanitize)
+test_expect_equal "$output" ""
+
+test_begin_subtest "Folder search with --output=files part #2"
+output=$(notmuch search --output=files folder:duplicate/bad/news | sed -e "s,$MAIL_DIR,MAIL_DIR,")
+test_expect_equal "$output" "MAIL_DIR/duplicate/bad/news/msg-003"
+
+test_begin_subtest "After removing duplicate instance of matching path part #2"
+output=$(notmuch search folder:duplicate/bad/news | notmuch_search_sanitize)
 test_expect_equal "$output" "thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; Bears (inbox unread)"
 
 test_begin_subtest "After rename, old path returns nothing"
 mv "${MAIL_DIR}/duplicate/bad/news" "${MAIL_DIR}/duplicate/bad/olds"
 notmuch new
-output=$(notmuch search folder:bad/news | notmuch_search_sanitize)
+output=$(notmuch search folder:duplicate/bad/news | notmuch_search_sanitize)
 test_expect_equal "$output" ""
 
 test_begin_subtest "After rename, new path returns result"
-output=$(notmuch search folder:bad/olds | notmuch_search_sanitize)
+output=$(notmuch search folder:duplicate/bad/olds | notmuch_search_sanitize)
 test_expect_equal "$output" "thread:XXX   2001-01-05 [1/1] Notmuch Test Suite; Bears (inbox unread)"
 
 test_done
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH v2 06/13] test: make it possible to have several corpora
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
                   ` (4 preceding siblings ...)
  2014-02-22 22:25 ` [PATCH v2 05/13] test: fix test for literal folder: search Jani Nikula
@ 2014-02-22 22:25 ` Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 07/13] test: add new corpus with folders Jani Nikula
                   ` (8 subsequent siblings)
  14 siblings, 0 replies; 25+ messages in thread
From: Jani Nikula @ 2014-02-22 22:25 UTC (permalink / raw)
  To: notmuch

Move the existing corpus under corpus/default, and make it possible to
have multiple corpora under the directory.
---
 test/.gitignore                     |  2 +-
 test/Makefile.local                 |  2 +-
 test/T480-hex-escaping.sh           |  4 ++--
 test/corpus/{ => default}/cur/01:2, |  0
 test/corpus/{ => default}/cur/02:2, |  0
 test/corpus/{ => default}/cur/03:2, |  0
 test/corpus/{ => default}/cur/04:2, |  0
 test/corpus/{ => default}/cur/05:2, |  0
 test/corpus/{ => default}/cur/06:2, |  0
 test/corpus/{ => default}/cur/07:2, |  0
 test/corpus/{ => default}/cur/08:2, |  0
 test/corpus/{ => default}/cur/09:2, |  0
 test/corpus/{ => default}/cur/10:2, |  0
 test/corpus/{ => default}/cur/11:2, |  0
 test/corpus/{ => default}/cur/12:2, |  0
 test/corpus/{ => default}/cur/13:2, |  0
 test/corpus/{ => default}/cur/14:2, |  0
 test/corpus/{ => default}/cur/15:2, |  0
 test/corpus/{ => default}/cur/16:2, |  0
 test/corpus/{ => default}/cur/17:2, |  0
 test/corpus/{ => default}/cur/18:2, |  0
 test/corpus/{ => default}/cur/19:2, |  0
 test/corpus/{ => default}/cur/20:2, |  0
 test/corpus/{ => default}/cur/21:2, |  0
 test/corpus/{ => default}/cur/22:2, |  0
 test/corpus/{ => default}/cur/23:2, |  0
 test/corpus/{ => default}/cur/24:2, |  0
 test/corpus/{ => default}/cur/25:2, |  0
 test/corpus/{ => default}/cur/26:2, |  0
 test/corpus/{ => default}/cur/27:2, |  0
 test/corpus/{ => default}/cur/28:2, |  0
 test/corpus/{ => default}/cur/29:2, |  0
 test/corpus/{ => default}/cur/30:2, |  0
 test/corpus/{ => default}/cur/31:2, |  0
 test/corpus/{ => default}/cur/32:2, |  0
 test/corpus/{ => default}/cur/33:2, |  0
 test/corpus/{ => default}/cur/34:2, |  0
 test/corpus/{ => default}/cur/35:2, |  0
 test/corpus/{ => default}/cur/36:2, |  0
 test/corpus/{ => default}/cur/37:2, |  0
 test/corpus/{ => default}/cur/38:2, |  0
 test/corpus/{ => default}/cur/39:2, |  0
 test/corpus/{ => default}/cur/40:2, |  0
 test/corpus/{ => default}/cur/41:2, |  0
 test/corpus/{ => default}/cur/42:2, |  0
 test/corpus/{ => default}/cur/43:2, |  0
 test/corpus/{ => default}/cur/44:2, |  0
 test/corpus/{ => default}/cur/45:2, |  0
 test/corpus/{ => default}/cur/46:2, |  0
 test/corpus/{ => default}/cur/47:2, |  0
 test/corpus/{ => default}/cur/48:2, |  0
 test/corpus/{ => default}/cur/49:2, |  0
 test/corpus/{ => default}/cur/50:2, |  0
 test/corpus/{ => default}/cur/51:2, |  0
 test/corpus/{ => default}/cur/52:2, |  0
 test/corpus/{ => default}/cur/53:2, |  0
 test/notmuch-test                   |  2 +-
 test/test-lib.sh                    | 21 +++++++++++++--------
 58 files changed, 18 insertions(+), 13 deletions(-)
 rename test/corpus/{ => default}/cur/01:2, (100%)
 rename test/corpus/{ => default}/cur/02:2, (100%)
 rename test/corpus/{ => default}/cur/03:2, (100%)
 rename test/corpus/{ => default}/cur/04:2, (100%)
 rename test/corpus/{ => default}/cur/05:2, (100%)
 rename test/corpus/{ => default}/cur/06:2, (100%)
 rename test/corpus/{ => default}/cur/07:2, (100%)
 rename test/corpus/{ => default}/cur/08:2, (100%)
 rename test/corpus/{ => default}/cur/09:2, (100%)
 rename test/corpus/{ => default}/cur/10:2, (100%)
 rename test/corpus/{ => default}/cur/11:2, (100%)
 rename test/corpus/{ => default}/cur/12:2, (100%)
 rename test/corpus/{ => default}/cur/13:2, (100%)
 rename test/corpus/{ => default}/cur/14:2, (100%)
 rename test/corpus/{ => default}/cur/15:2, (100%)
 rename test/corpus/{ => default}/cur/16:2, (100%)
 rename test/corpus/{ => default}/cur/17:2, (100%)
 rename test/corpus/{ => default}/cur/18:2, (100%)
 rename test/corpus/{ => default}/cur/19:2, (100%)
 rename test/corpus/{ => default}/cur/20:2, (100%)
 rename test/corpus/{ => default}/cur/21:2, (100%)
 rename test/corpus/{ => default}/cur/22:2, (100%)
 rename test/corpus/{ => default}/cur/23:2, (100%)
 rename test/corpus/{ => default}/cur/24:2, (100%)
 rename test/corpus/{ => default}/cur/25:2, (100%)
 rename test/corpus/{ => default}/cur/26:2, (100%)
 rename test/corpus/{ => default}/cur/27:2, (100%)
 rename test/corpus/{ => default}/cur/28:2, (100%)
 rename test/corpus/{ => default}/cur/29:2, (100%)
 rename test/corpus/{ => default}/cur/30:2, (100%)
 rename test/corpus/{ => default}/cur/31:2, (100%)
 rename test/corpus/{ => default}/cur/32:2, (100%)
 rename test/corpus/{ => default}/cur/33:2, (100%)
 rename test/corpus/{ => default}/cur/34:2, (100%)
 rename test/corpus/{ => default}/cur/35:2, (100%)
 rename test/corpus/{ => default}/cur/36:2, (100%)
 rename test/corpus/{ => default}/cur/37:2, (100%)
 rename test/corpus/{ => default}/cur/38:2, (100%)
 rename test/corpus/{ => default}/cur/39:2, (100%)
 rename test/corpus/{ => default}/cur/40:2, (100%)
 rename test/corpus/{ => default}/cur/41:2, (100%)
 rename test/corpus/{ => default}/cur/42:2, (100%)
 rename test/corpus/{ => default}/cur/43:2, (100%)
 rename test/corpus/{ => default}/cur/44:2, (100%)
 rename test/corpus/{ => default}/cur/45:2, (100%)
 rename test/corpus/{ => default}/cur/46:2, (100%)
 rename test/corpus/{ => default}/cur/47:2, (100%)
 rename test/corpus/{ => default}/cur/48:2, (100%)
 rename test/corpus/{ => default}/cur/49:2, (100%)
 rename test/corpus/{ => default}/cur/50:2, (100%)
 rename test/corpus/{ => default}/cur/51:2, (100%)
 rename test/corpus/{ => default}/cur/52:2, (100%)
 rename test/corpus/{ => default}/cur/53:2, (100%)

diff --git a/test/.gitignore b/test/.gitignore
index 97e024878728..ed20d1ac6e42 100644
--- a/test/.gitignore
+++ b/test/.gitignore
@@ -1,5 +1,5 @@
 test-results
-corpus.mail
+corpus.mail.*
 smtp-dummy
 symbol-test
 arg-test
diff --git a/test/Makefile.local b/test/Makefile.local
index 8870ca374848..20f17629dda3 100644
--- a/test/Makefile.local
+++ b/test/Makefile.local
@@ -57,4 +57,4 @@ CLEAN := $(CLEAN) $(dir)/smtp-dummy $(dir)/smtp-dummy.o \
 	 $(dir)/database-test.o \
 	 $(dir)/random-corpus $(dir)/random-corpus.o \
 	 $(dir)/parse-time $(dir)/parse-time.o \
-	 $(dir)/corpus.mail $(dir)/test-results $(dir)/tmp.*
+	 $(dir)/corpus.mail.* $(dir)/test-results $(dir)/tmp.*
diff --git a/test/T480-hex-escaping.sh b/test/T480-hex-escaping.sh
index ad50e1bcc071..15701650df83 100755
--- a/test/T480-hex-escaping.sh
+++ b/test/T480-hex-escaping.sh
@@ -3,7 +3,7 @@ test_description="hex encoding and decoding"
 . ./test-lib.sh
 
 test_begin_subtest "round trip"
-find $TEST_DIRECTORY/corpus -type f -print | sort | xargs cat > EXPECTED
+find $TEST_DIRECTORY/corpus/default -type f -print | sort | xargs cat > EXPECTED
 $TEST_DIRECTORY/hex-xcode --direction=encode < EXPECTED | $TEST_DIRECTORY/hex-xcode --direction=decode > OUTPUT
 test_expect_equal_file OUTPUT EXPECTED
 
@@ -25,7 +25,7 @@ $TEST_DIRECTORY/hex-xcode --direction=decode  < EXPECTED.$test_count |\
 test_expect_equal_file EXPECTED.$test_count OUTPUT.$test_count
 
 test_begin_subtest "round trip (in-place)"
-find $TEST_DIRECTORY/corpus -type f -print | sort | xargs cat > EXPECTED
+find $TEST_DIRECTORY/corpus/default -type f -print | sort | xargs cat > EXPECTED
 $TEST_DIRECTORY/hex-xcode --in-place --direction=encode < EXPECTED |\
      $TEST_DIRECTORY/hex-xcode --in-place --direction=decode > OUTPUT
 test_expect_equal_file OUTPUT EXPECTED
diff --git a/test/corpus/cur/01:2, b/test/corpus/default/cur/01:2,
similarity index 100%
rename from test/corpus/cur/01:2,
rename to test/corpus/default/cur/01:2,
diff --git a/test/corpus/cur/02:2, b/test/corpus/default/cur/02:2,
similarity index 100%
rename from test/corpus/cur/02:2,
rename to test/corpus/default/cur/02:2,
diff --git a/test/corpus/cur/03:2, b/test/corpus/default/cur/03:2,
similarity index 100%
rename from test/corpus/cur/03:2,
rename to test/corpus/default/cur/03:2,
diff --git a/test/corpus/cur/04:2, b/test/corpus/default/cur/04:2,
similarity index 100%
rename from test/corpus/cur/04:2,
rename to test/corpus/default/cur/04:2,
diff --git a/test/corpus/cur/05:2, b/test/corpus/default/cur/05:2,
similarity index 100%
rename from test/corpus/cur/05:2,
rename to test/corpus/default/cur/05:2,
diff --git a/test/corpus/cur/06:2, b/test/corpus/default/cur/06:2,
similarity index 100%
rename from test/corpus/cur/06:2,
rename to test/corpus/default/cur/06:2,
diff --git a/test/corpus/cur/07:2, b/test/corpus/default/cur/07:2,
similarity index 100%
rename from test/corpus/cur/07:2,
rename to test/corpus/default/cur/07:2,
diff --git a/test/corpus/cur/08:2, b/test/corpus/default/cur/08:2,
similarity index 100%
rename from test/corpus/cur/08:2,
rename to test/corpus/default/cur/08:2,
diff --git a/test/corpus/cur/09:2, b/test/corpus/default/cur/09:2,
similarity index 100%
rename from test/corpus/cur/09:2,
rename to test/corpus/default/cur/09:2,
diff --git a/test/corpus/cur/10:2, b/test/corpus/default/cur/10:2,
similarity index 100%
rename from test/corpus/cur/10:2,
rename to test/corpus/default/cur/10:2,
diff --git a/test/corpus/cur/11:2, b/test/corpus/default/cur/11:2,
similarity index 100%
rename from test/corpus/cur/11:2,
rename to test/corpus/default/cur/11:2,
diff --git a/test/corpus/cur/12:2, b/test/corpus/default/cur/12:2,
similarity index 100%
rename from test/corpus/cur/12:2,
rename to test/corpus/default/cur/12:2,
diff --git a/test/corpus/cur/13:2, b/test/corpus/default/cur/13:2,
similarity index 100%
rename from test/corpus/cur/13:2,
rename to test/corpus/default/cur/13:2,
diff --git a/test/corpus/cur/14:2, b/test/corpus/default/cur/14:2,
similarity index 100%
rename from test/corpus/cur/14:2,
rename to test/corpus/default/cur/14:2,
diff --git a/test/corpus/cur/15:2, b/test/corpus/default/cur/15:2,
similarity index 100%
rename from test/corpus/cur/15:2,
rename to test/corpus/default/cur/15:2,
diff --git a/test/corpus/cur/16:2, b/test/corpus/default/cur/16:2,
similarity index 100%
rename from test/corpus/cur/16:2,
rename to test/corpus/default/cur/16:2,
diff --git a/test/corpus/cur/17:2, b/test/corpus/default/cur/17:2,
similarity index 100%
rename from test/corpus/cur/17:2,
rename to test/corpus/default/cur/17:2,
diff --git a/test/corpus/cur/18:2, b/test/corpus/default/cur/18:2,
similarity index 100%
rename from test/corpus/cur/18:2,
rename to test/corpus/default/cur/18:2,
diff --git a/test/corpus/cur/19:2, b/test/corpus/default/cur/19:2,
similarity index 100%
rename from test/corpus/cur/19:2,
rename to test/corpus/default/cur/19:2,
diff --git a/test/corpus/cur/20:2, b/test/corpus/default/cur/20:2,
similarity index 100%
rename from test/corpus/cur/20:2,
rename to test/corpus/default/cur/20:2,
diff --git a/test/corpus/cur/21:2, b/test/corpus/default/cur/21:2,
similarity index 100%
rename from test/corpus/cur/21:2,
rename to test/corpus/default/cur/21:2,
diff --git a/test/corpus/cur/22:2, b/test/corpus/default/cur/22:2,
similarity index 100%
rename from test/corpus/cur/22:2,
rename to test/corpus/default/cur/22:2,
diff --git a/test/corpus/cur/23:2, b/test/corpus/default/cur/23:2,
similarity index 100%
rename from test/corpus/cur/23:2,
rename to test/corpus/default/cur/23:2,
diff --git a/test/corpus/cur/24:2, b/test/corpus/default/cur/24:2,
similarity index 100%
rename from test/corpus/cur/24:2,
rename to test/corpus/default/cur/24:2,
diff --git a/test/corpus/cur/25:2, b/test/corpus/default/cur/25:2,
similarity index 100%
rename from test/corpus/cur/25:2,
rename to test/corpus/default/cur/25:2,
diff --git a/test/corpus/cur/26:2, b/test/corpus/default/cur/26:2,
similarity index 100%
rename from test/corpus/cur/26:2,
rename to test/corpus/default/cur/26:2,
diff --git a/test/corpus/cur/27:2, b/test/corpus/default/cur/27:2,
similarity index 100%
rename from test/corpus/cur/27:2,
rename to test/corpus/default/cur/27:2,
diff --git a/test/corpus/cur/28:2, b/test/corpus/default/cur/28:2,
similarity index 100%
rename from test/corpus/cur/28:2,
rename to test/corpus/default/cur/28:2,
diff --git a/test/corpus/cur/29:2, b/test/corpus/default/cur/29:2,
similarity index 100%
rename from test/corpus/cur/29:2,
rename to test/corpus/default/cur/29:2,
diff --git a/test/corpus/cur/30:2, b/test/corpus/default/cur/30:2,
similarity index 100%
rename from test/corpus/cur/30:2,
rename to test/corpus/default/cur/30:2,
diff --git a/test/corpus/cur/31:2, b/test/corpus/default/cur/31:2,
similarity index 100%
rename from test/corpus/cur/31:2,
rename to test/corpus/default/cur/31:2,
diff --git a/test/corpus/cur/32:2, b/test/corpus/default/cur/32:2,
similarity index 100%
rename from test/corpus/cur/32:2,
rename to test/corpus/default/cur/32:2,
diff --git a/test/corpus/cur/33:2, b/test/corpus/default/cur/33:2,
similarity index 100%
rename from test/corpus/cur/33:2,
rename to test/corpus/default/cur/33:2,
diff --git a/test/corpus/cur/34:2, b/test/corpus/default/cur/34:2,
similarity index 100%
rename from test/corpus/cur/34:2,
rename to test/corpus/default/cur/34:2,
diff --git a/test/corpus/cur/35:2, b/test/corpus/default/cur/35:2,
similarity index 100%
rename from test/corpus/cur/35:2,
rename to test/corpus/default/cur/35:2,
diff --git a/test/corpus/cur/36:2, b/test/corpus/default/cur/36:2,
similarity index 100%
rename from test/corpus/cur/36:2,
rename to test/corpus/default/cur/36:2,
diff --git a/test/corpus/cur/37:2, b/test/corpus/default/cur/37:2,
similarity index 100%
rename from test/corpus/cur/37:2,
rename to test/corpus/default/cur/37:2,
diff --git a/test/corpus/cur/38:2, b/test/corpus/default/cur/38:2,
similarity index 100%
rename from test/corpus/cur/38:2,
rename to test/corpus/default/cur/38:2,
diff --git a/test/corpus/cur/39:2, b/test/corpus/default/cur/39:2,
similarity index 100%
rename from test/corpus/cur/39:2,
rename to test/corpus/default/cur/39:2,
diff --git a/test/corpus/cur/40:2, b/test/corpus/default/cur/40:2,
similarity index 100%
rename from test/corpus/cur/40:2,
rename to test/corpus/default/cur/40:2,
diff --git a/test/corpus/cur/41:2, b/test/corpus/default/cur/41:2,
similarity index 100%
rename from test/corpus/cur/41:2,
rename to test/corpus/default/cur/41:2,
diff --git a/test/corpus/cur/42:2, b/test/corpus/default/cur/42:2,
similarity index 100%
rename from test/corpus/cur/42:2,
rename to test/corpus/default/cur/42:2,
diff --git a/test/corpus/cur/43:2, b/test/corpus/default/cur/43:2,
similarity index 100%
rename from test/corpus/cur/43:2,
rename to test/corpus/default/cur/43:2,
diff --git a/test/corpus/cur/44:2, b/test/corpus/default/cur/44:2,
similarity index 100%
rename from test/corpus/cur/44:2,
rename to test/corpus/default/cur/44:2,
diff --git a/test/corpus/cur/45:2, b/test/corpus/default/cur/45:2,
similarity index 100%
rename from test/corpus/cur/45:2,
rename to test/corpus/default/cur/45:2,
diff --git a/test/corpus/cur/46:2, b/test/corpus/default/cur/46:2,
similarity index 100%
rename from test/corpus/cur/46:2,
rename to test/corpus/default/cur/46:2,
diff --git a/test/corpus/cur/47:2, b/test/corpus/default/cur/47:2,
similarity index 100%
rename from test/corpus/cur/47:2,
rename to test/corpus/default/cur/47:2,
diff --git a/test/corpus/cur/48:2, b/test/corpus/default/cur/48:2,
similarity index 100%
rename from test/corpus/cur/48:2,
rename to test/corpus/default/cur/48:2,
diff --git a/test/corpus/cur/49:2, b/test/corpus/default/cur/49:2,
similarity index 100%
rename from test/corpus/cur/49:2,
rename to test/corpus/default/cur/49:2,
diff --git a/test/corpus/cur/50:2, b/test/corpus/default/cur/50:2,
similarity index 100%
rename from test/corpus/cur/50:2,
rename to test/corpus/default/cur/50:2,
diff --git a/test/corpus/cur/51:2, b/test/corpus/default/cur/51:2,
similarity index 100%
rename from test/corpus/cur/51:2,
rename to test/corpus/default/cur/51:2,
diff --git a/test/corpus/cur/52:2, b/test/corpus/default/cur/52:2,
similarity index 100%
rename from test/corpus/cur/52:2,
rename to test/corpus/default/cur/52:2,
diff --git a/test/corpus/cur/53:2, b/test/corpus/default/cur/53:2,
similarity index 100%
rename from test/corpus/cur/53:2,
rename to test/corpus/default/cur/53:2,
diff --git a/test/notmuch-test b/test/notmuch-test
index b8437127c01f..6598f428d14f 100755
--- a/test/notmuch-test
+++ b/test/notmuch-test
@@ -49,6 +49,6 @@ echo
 ev=$?
 
 # Clean up
-rm -rf test-results corpus.mail
+rm -rf test-results corpus.mail.*
 
 exit $ev
diff --git a/test/test-lib.sh b/test/test-lib.sh
index 78af170fe739..69070f26e4fe 100644
--- a/test/test-lib.sh
+++ b/test/test-lib.sh
@@ -516,19 +516,24 @@ emacs_fcc_message ()
 
 # Generate a corpus of email and add it to the database.
 #
-# This corpus is fixed, (it happens to be 50 messages from early in
-# the history of the notmuch mailing list), which allows for reliably
-# testing commands that need to operate on a not-totally-trivial
-# number of messages.
+# The first parameter defines the corpus to pick under the corpus
+# directory, defaulting to "default".
+#
+# The default corpus is fixed, (it happens to be 50 messages from
+# early in the history of the notmuch mailing list), which allows for
+# reliably testing commands that need to operate on a
+# not-totally-trivial number of messages.
 add_email_corpus ()
 {
+    local corpus=${1:-default}
+
     rm -rf ${MAIL_DIR}
-    if [ -d $TEST_DIRECTORY/corpus.mail ]; then
-	cp -a $TEST_DIRECTORY/corpus.mail ${MAIL_DIR}
+    if [ -d $TEST_DIRECTORY/corpus.mail.${corpus} ]; then
+	cp -a $TEST_DIRECTORY/corpus.mail.${corpus} ${MAIL_DIR}
     else
-	cp -a $TEST_DIRECTORY/corpus ${MAIL_DIR}
+	cp -a $TEST_DIRECTORY/corpus/${corpus} ${MAIL_DIR}
 	notmuch new >/dev/null
-	cp -a ${MAIL_DIR} $TEST_DIRECTORY/corpus.mail
+	cp -a ${MAIL_DIR} $TEST_DIRECTORY/corpus.mail.${corpus}
     fi
 }
 
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH v2 07/13] test: add new corpus with folders
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
                   ` (5 preceding siblings ...)
  2014-02-22 22:25 ` [PATCH v2 06/13] test: make it possible to have several corpora Jani Nikula
@ 2014-02-22 22:25 ` Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 08/13] test: add tests for the new boolean folder: and path: prefixes Jani Nikula
                   ` (7 subsequent siblings)
  14 siblings, 0 replies; 25+ messages in thread
From: Jani Nikula @ 2014-02-22 22:25 UTC (permalink / raw)
  To: notmuch

---

corpus content dropped to save bandwidth, see

https://gitorious.org/jani/notmuch/commit/3aff18b55de46e3b27748376a59cda41b6cd7f6d

 test/corpus/folders/01:2,             |  34 ++++
 test/corpus/folders/02:2,             |  32 +++
 test/corpus/folders/bar/17:2,         |  23 +++
 test/corpus/folders/bar/18:2,         |  12 ++
 test/corpus/folders/bar/baz/05:2,     | 104 ++++++++++
 test/corpus/folders/bar/baz/23:2,     | 145 ++++++++++++++
 test/corpus/folders/bar/baz/24:2,     | 204 +++++++++++++++++++
 test/corpus/folders/bar/baz/cur/25:2, |  32 +++
 test/corpus/folders/bar/baz/cur/26:2, | 121 ++++++++++++
 test/corpus/folders/bar/baz/new/27:2, |  21 ++
 test/corpus/folders/bar/baz/new/28:2, |  38 ++++
 test/corpus/folders/bar/cur/19:2,     | 360 ++++++++++++++++++++++++++++++++++
 test/corpus/folders/bar/cur/20:2,     | 101 ++++++++++
 test/corpus/folders/bar/new/21:2,     | 102 ++++++++++
 test/corpus/folders/bar/new/22:2,     |  84 ++++++++
 test/corpus/folders/cur/29:2,         |  21 ++
 test/corpus/folders/cur/30:2,         |  75 +++++++
 test/corpus/folders/cur/31:2,         |  31 +++
 test/corpus/folders/cur/32:2,         | 165 ++++++++++++++++
 test/corpus/folders/cur/33:2,         |  13 ++
 test/corpus/folders/cur/34:2,         |  46 +++++
 test/corpus/folders/cur/35:2,         |  24 +++
 test/corpus/folders/cur/36:2,         |  25 +++
 test/corpus/folders/cur/37:2,         |  22 +++
 test/corpus/folders/cur/38:2,         |  40 ++++
 test/corpus/folders/cur/39:2,         |  32 +++
 test/corpus/folders/cur/40:2,         |  31 +++
 test/corpus/folders/cur/41:2,         |  37 ++++
 test/corpus/folders/cur/42:2,         |  30 +++
 test/corpus/folders/cur/43:2,         |  26 +++
 test/corpus/folders/cur/44:2,         |  29 +++
 test/corpus/folders/cur/45:2,         |  41 ++++
 test/corpus/folders/cur/46:2,         |  57 ++++++
 test/corpus/folders/cur/47:2,         |  84 ++++++++
 test/corpus/folders/cur/48:2,         |  17 ++
 test/corpus/folders/cur/49:2,         |  33 ++++
 test/corpus/folders/cur/50:2,         |  39 ++++
 test/corpus/folders/cur/52:2,         |  39 ++++
 test/corpus/folders/cur/53:2,         |  20 ++
 test/corpus/folders/foo/05:2,         | 104 ++++++++++
 test/corpus/folders/foo/06:2,         |  36 ++++
 test/corpus/folders/foo/baz/11:2,     |  27 +++
 test/corpus/folders/foo/baz/12:2,     |  27 +++
 test/corpus/folders/foo/baz/cur/13:2, | 178 +++++++++++++++++
 test/corpus/folders/foo/baz/cur/14:2, |  39 ++++
 test/corpus/folders/foo/baz/new/15:2, |  22 +++
 test/corpus/folders/foo/baz/new/16:2, |  27 +++
 test/corpus/folders/foo/cur/07:2,     |  57 ++++++
 test/corpus/folders/foo/cur/08:2,     |  87 ++++++++
 test/corpus/folders/foo/new/03:2,     |  93 +++++++++
 test/corpus/folders/foo/new/09:2,     |  33 ++++
 test/corpus/folders/foo/new/10:2,     |  54 +++++
 test/corpus/folders/new/03:2,         |  93 +++++++++
 test/corpus/folders/new/04:2,         |  84 ++++++++
 54 files changed, 3351 insertions(+)
 create mode 100644 test/corpus/folders/01:2,
 create mode 100644 test/corpus/folders/02:2,
 create mode 100644 test/corpus/folders/bar/17:2,
 create mode 100644 test/corpus/folders/bar/18:2,
 create mode 100644 test/corpus/folders/bar/baz/05:2,
 create mode 100644 test/corpus/folders/bar/baz/23:2,
 create mode 100644 test/corpus/folders/bar/baz/24:2,
 create mode 100644 test/corpus/folders/bar/baz/cur/25:2,
 create mode 100644 test/corpus/folders/bar/baz/cur/26:2,
 create mode 100644 test/corpus/folders/bar/baz/new/27:2,
 create mode 100644 test/corpus/folders/bar/baz/new/28:2,
 create mode 100644 test/corpus/folders/bar/cur/19:2,
 create mode 100644 test/corpus/folders/bar/cur/20:2,
 create mode 100644 test/corpus/folders/bar/new/21:2,
 create mode 100644 test/corpus/folders/bar/new/22:2,
 create mode 100644 test/corpus/folders/cur/29:2,
 create mode 100644 test/corpus/folders/cur/30:2,
 create mode 100644 test/corpus/folders/cur/31:2,
 create mode 100644 test/corpus/folders/cur/32:2,
 create mode 100644 test/corpus/folders/cur/33:2,
 create mode 100644 test/corpus/folders/cur/34:2,
 create mode 100644 test/corpus/folders/cur/35:2,
 create mode 100644 test/corpus/folders/cur/36:2,
 create mode 100644 test/corpus/folders/cur/37:2,
 create mode 100644 test/corpus/folders/cur/38:2,
 create mode 100644 test/corpus/folders/cur/39:2,
 create mode 100644 test/corpus/folders/cur/40:2,
 create mode 100644 test/corpus/folders/cur/41:2,
 create mode 100644 test/corpus/folders/cur/42:2,
 create mode 100644 test/corpus/folders/cur/43:2,
 create mode 100644 test/corpus/folders/cur/44:2,
 create mode 100644 test/corpus/folders/cur/45:2,
 create mode 100644 test/corpus/folders/cur/46:2,
 create mode 100644 test/corpus/folders/cur/47:2,
 create mode 100644 test/corpus/folders/cur/48:2,
 create mode 100644 test/corpus/folders/cur/49:2,
 create mode 100644 test/corpus/folders/cur/50:2,
 create mode 100644 test/corpus/folders/cur/52:2,
 create mode 100644 test/corpus/folders/cur/53:2,
 create mode 100644 test/corpus/folders/foo/05:2,
 create mode 100644 test/corpus/folders/foo/06:2,
 create mode 100644 test/corpus/folders/foo/baz/11:2,
 create mode 100644 test/corpus/folders/foo/baz/12:2,
 create mode 100644 test/corpus/folders/foo/baz/cur/13:2,
 create mode 100644 test/corpus/folders/foo/baz/cur/14:2,
 create mode 100644 test/corpus/folders/foo/baz/new/15:2,
 create mode 100644 test/corpus/folders/foo/baz/new/16:2,
 create mode 100644 test/corpus/folders/foo/cur/07:2,
 create mode 100644 test/corpus/folders/foo/cur/08:2,
 create mode 100644 test/corpus/folders/foo/new/03:2,
 create mode 100644 test/corpus/folders/foo/new/09:2,
 create mode 100644 test/corpus/folders/foo/new/10:2,
 create mode 100644 test/corpus/folders/new/03:2,
 create mode 100644 test/corpus/folders/new/04:2,

-- 
1.8.5.3

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [PATCH v2 08/13] test: add tests for the new boolean folder: and path: prefixes
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
                   ` (6 preceding siblings ...)
  2014-02-22 22:25 ` [PATCH v2 07/13] test: add new corpus with folders Jani Nikula
@ 2014-02-22 22:25 ` Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 09/13] devel: add script to generate test databases Jani Nikula
                   ` (6 subsequent siblings)
  14 siblings, 0 replies; 25+ messages in thread
From: Jani Nikula @ 2014-02-22 22:25 UTC (permalink / raw)
  To: notmuch

Additional tests for the boolean folder: and path: prefixes.
---
 test/T101-search-by-folder-and-path.sh | 83 ++++++++++++++++++++++++++++++++++
 1 file changed, 83 insertions(+)
 create mode 100755 test/T101-search-by-folder-and-path.sh

diff --git a/test/T101-search-by-folder-and-path.sh b/test/T101-search-by-folder-and-path.sh
new file mode 100755
index 000000000000..9f809e46e110
--- /dev/null
+++ b/test/T101-search-by-folder-and-path.sh
@@ -0,0 +1,83 @@
+#!/usr/bin/env bash
+test_description='"notmuch search" by folder: and path:'
+. ./test-lib.sh
+
+add_email_corpus folders
+
+test_begin_subtest "folder: search"
+output=$(notmuch search --output=files folder:foo | sed -e "s,$MAIL_DIR,MAIL_DIR," | sort)
+# bar/baz/05:2, and new/03:2, are duplicates of foo/05:2, and
+# foo/new/03:2, respectively
+test_expect_equal "$output" "MAIL_DIR/bar/baz/05:2,
+MAIL_DIR/foo/05:2,
+MAIL_DIR/foo/06:2,
+MAIL_DIR/foo/cur/07:2,
+MAIL_DIR/foo/cur/08:2,
+MAIL_DIR/foo/new/03:2,
+MAIL_DIR/foo/new/09:2,
+MAIL_DIR/foo/new/10:2,
+MAIL_DIR/new/03:2,"
+
+test_begin_subtest "top level folder: search"
+output=$(notmuch search --output=files folder:'""' | sed -e "s,$MAIL_DIR,MAIL_DIR," | sort)
+# foo/new/03:2, is a duplicate of new/03:2,
+test_expect_equal "$output" "MAIL_DIR/01:2,
+MAIL_DIR/02:2,
+MAIL_DIR/cur/29:2,
+MAIL_DIR/cur/30:2,
+MAIL_DIR/cur/31:2,
+MAIL_DIR/cur/32:2,
+MAIL_DIR/cur/33:2,
+MAIL_DIR/cur/34:2,
+MAIL_DIR/cur/35:2,
+MAIL_DIR/cur/36:2,
+MAIL_DIR/cur/37:2,
+MAIL_DIR/cur/38:2,
+MAIL_DIR/cur/39:2,
+MAIL_DIR/cur/40:2,
+MAIL_DIR/cur/41:2,
+MAIL_DIR/cur/42:2,
+MAIL_DIR/cur/43:2,
+MAIL_DIR/cur/44:2,
+MAIL_DIR/cur/45:2,
+MAIL_DIR/cur/46:2,
+MAIL_DIR/cur/47:2,
+MAIL_DIR/cur/48:2,
+MAIL_DIR/cur/49:2,
+MAIL_DIR/cur/50:2,
+MAIL_DIR/cur/52:2,
+MAIL_DIR/cur/53:2,
+MAIL_DIR/foo/new/03:2,
+MAIL_DIR/new/03:2,
+MAIL_DIR/new/04:2,"
+
+test_begin_subtest "path: search"
+output=$(notmuch search --output=files path:"bar" | sed -e "s,$MAIL_DIR,MAIL_DIR," | sort)
+# foo/05:2, is a duplicate of bar/baz/05:2,
+test_expect_equal "$output" "MAIL_DIR/bar/17:2,
+MAIL_DIR/bar/18:2,"
+
+test_begin_subtest "top level path: search"
+output=$(notmuch search --output=files path:'""' | sed -e "s,$MAIL_DIR,MAIL_DIR," | sort)
+test_expect_equal "$output" "MAIL_DIR/01:2,
+MAIL_DIR/02:2,"
+
+test_begin_subtest "recursive path: search"
+output=$(notmuch search --output=files path:"bar/**" | sed -e "s,$MAIL_DIR,MAIL_DIR," | sort)
+# foo/05:2, is a duplicate of bar/baz/05:2,
+test_expect_equal "$output" "MAIL_DIR/bar/17:2,
+MAIL_DIR/bar/18:2,
+MAIL_DIR/bar/baz/05:2,
+MAIL_DIR/bar/baz/23:2,
+MAIL_DIR/bar/baz/24:2,
+MAIL_DIR/bar/baz/cur/25:2,
+MAIL_DIR/bar/baz/cur/26:2,
+MAIL_DIR/bar/baz/new/27:2,
+MAIL_DIR/bar/baz/new/28:2,
+MAIL_DIR/bar/cur/19:2,
+MAIL_DIR/bar/cur/20:2,
+MAIL_DIR/bar/new/21:2,
+MAIL_DIR/bar/new/22:2,
+MAIL_DIR/foo/05:2,"
+
+test_done
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH v2 09/13] devel: add script to generate test databases
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
                   ` (7 preceding siblings ...)
  2014-02-22 22:25 ` [PATCH v2 08/13] test: add tests for the new boolean folder: and path: prefixes Jani Nikula
@ 2014-02-22 22:25 ` Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 10/13] test: add test database in format version 1 Jani Nikula
                   ` (5 subsequent siblings)
  14 siblings, 0 replies; 25+ messages in thread
From: Jani Nikula @ 2014-02-22 22:25 UTC (permalink / raw)
  To: notmuch

Add script to generate notmuch test databases using specified versions
of notmuch. This is useful for generating material for database
upgrade tests.

This reuses the test infrastructure to have a sandbox environment for
notmuch new etc.
---
 devel/gen-testdb.sh | 124 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 124 insertions(+)
 create mode 100755 devel/gen-testdb.sh

diff --git a/devel/gen-testdb.sh b/devel/gen-testdb.sh
new file mode 100755
index 000000000000..c291dfff98c9
--- /dev/null
+++ b/devel/gen-testdb.sh
@@ -0,0 +1,124 @@
+#!/usr/bin/env bash
+#
+# NAME
+#	gen-testdb.sh - generate test databases
+#
+# SYNOPSIS
+#	gen-testdb.sh -v NOTMUCH-VERSION [-c CORPUS-PATH] [-s TAR-SUFFIX]
+#
+# DESCRIPTION
+#	Generate a tarball containing the specified test corpus and
+#	the corresponding notmuch database, indexed using a specific
+#	version of notmuch, resulting in a specific version of the
+#	database.
+#
+#	The specific version of notmuch will be built on the fly.
+#	Therefore the script must be run within a git repository to be
+#	able to build the old versions of notmuch.
+#
+#	This script reuses the test infrastructure, and the script
+#	must be run from within the test directory.
+#
+#	The output tarballs, named database-<TAR-SUFFIX>.tar.gz, are
+#	placed in the test/test-databases directory.
+#
+# OPTIONS
+#	-v NOTMUCH-VERSION
+#		Notmuch version in terms of a git tag or commit to use
+#		for generating the database. Required.
+#
+#	-c CORPUS-PATH
+#		Path to a corpus to use for generating the
+#		database. Due to CWD changes within the test
+#		infrastructure, use absolute paths. Defaults to the
+#		test corpus.
+#
+#	-s TAR-SUFFIX
+#		Suffix for the tarball basename. Empty by default.
+#
+# EXAMPLE
+#
+#	Generate a database indexed with notmuch 0.17. Use the default
+#	test corpus. Name the tarball database-v1.tar.gz to reflect
+#	the fact that notmuch 0.17 used database version 1.
+#
+#	$ cd test
+#	$ ../devel/gen-testdb.sh -v 0.17 -s v1
+#
+# CAVEATS
+#	Test infrastructure options won't work.
+#
+#	Any existing databases with the same name will be overwritten.
+#
+#	It may not be possible to build old versions of notmuch with
+#	the set of dependencies that satisfy building the current
+#	version of notmuch.
+#
+# AUTHOR
+#	Jani Nikula <jani@nikula.org>
+#
+# LICENSE
+#	Same as notmuch test infrastructure (GPLv2+).
+#
+
+test_description="database generation abusing test infrastructure"
+
+# immediate exit on subtest failure; see test_failure_ in test-lib.sh
+immediate=t
+
+VERSION=
+CORPUS=
+SUFFIX=
+
+while getopts v:c:s: opt; do
+    case "$opt" in
+	v) VERSION="$OPTARG";;
+	c) CORPUS="$OPTARG";;
+	s) SUFFIX="-$OPTARG";;
+    esac
+done
+shift `expr $OPTIND - 1`
+
+. ./test-lib.sh
+
+CORPUS=${CORPUS:-${TEST_DIRECTORY}/corpus}
+
+test_expect_code 0 "notmuch version specified on the command line" \
+    "test -n ${VERSION}"
+
+test_expect_code 0 "the specified version ${VERSION} refers to a commit" \
+    "git show ${VERSION} >/dev/null 2>&1"
+
+BUILD_DIR="notmuch-${VERSION}"
+test_expect_code 0 "generate snapshot of notmuch version ${VERSION}" \
+    "git -C $TEST_DIRECTORY/.. archive --prefix=${BUILD_DIR}/ --format=tar ${VERSION} | tar x"
+
+# force version string
+git describe --match '[0-9.]*' ${VERSION} > ${BUILD_DIR}/version
+
+test_expect_code 0 "configure and build notmuch version ${VERSION}" \
+    "make -C ${BUILD_DIR}"
+
+# use the newly built notmuch
+export PATH=./${BUILD_DIR}:$PATH
+
+test_begin_subtest "verify the newly built notmuch version"
+test_expect_equal "`notmuch --version`" "notmuch `cat ${BUILD_DIR}/version`"
+
+# replace the existing mails, if any, with the specified corpus
+rm -rf ${MAIL_DIR}
+cp -a ${CORPUS} ${MAIL_DIR}
+
+test_expect_code 0 "index the corpus" \
+    "notmuch new"
+
+# finally, wrap the resulting mail store and database in a tarball
+DBNAME=database${SUFFIX}
+cp -a ${MAIL_DIR} ${TMP_DIRECTORY}/${DBNAME}
+tar zcf ${TMP_DIRECTORY}/${DBNAME}.tar.gz -C ${TMP_DIRECTORY} ${DBNAME}
+mkdir -p  ${TEST_DIRECTORY}/test-databases
+cp -a ${TMP_DIRECTORY}/${DBNAME}.tar.gz ${TEST_DIRECTORY}/test-databases
+test_expect_code 0 "create the output tarball ${DBNAME}.tar.gz" \
+    "test -f ${TEST_DIRECTORY}/test-databases/${DBNAME}.tar.gz"
+
+test_done
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH v2 10/13] test: add test database in format version 1
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
                   ` (8 preceding siblings ...)
  2014-02-22 22:25 ` [PATCH v2 09/13] devel: add script to generate test databases Jani Nikula
@ 2014-02-22 22:25 ` Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 11/13] test: add database upgrade test from format version 1 to 2 Jani Nikula
                   ` (4 subsequent siblings)
  14 siblings, 0 replies; 25+ messages in thread
From: Jani Nikula @ 2014-02-22 22:25 UTC (permalink / raw)
  To: notmuch

Generated using:

$ cd test
$ ../devel/gen-testdb.sh -v 0.17 -c $PWD/corpus/folders -s v1

This database contains old style probabilistic folder: terms and no
path: terms.
---

corpus/database content dropped to save bandwidth, see

https://gitorious.org/jani/notmuch/commit/88bcffe31441e62d19769a2a941a2965f8c91486

 test/test-databases/README             |   5 +++++
 test/test-databases/database-v1.tar.gz | Bin 0 -> 262063 bytes
 2 files changed, 5 insertions(+)
 create mode 100644 test/test-databases/README
 create mode 100644 test/test-databases/database-v1.tar.gz

diff --git a/test/test-databases/README b/test/test-databases/README
new file mode 100644
index 000000000000..af5defe80728
--- /dev/null
+++ b/test/test-databases/README
@@ -0,1 +1,5 @@
+Notmuch test databases
+======================
+
+This directory contains pre-generated databases with their source
+corpus, chiefly for the purpose of testing database upgrade.
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH v2 11/13] test: add database upgrade test from format version 1 to 2
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
                   ` (9 preceding siblings ...)
  2014-02-22 22:25 ` [PATCH v2 10/13] test: add test database in format version 1 Jani Nikula
@ 2014-02-22 22:25 ` Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 12/13] man: update man pages for folder: and path: search terms Jani Nikula
                   ` (3 subsequent siblings)
  14 siblings, 0 replies; 25+ messages in thread
From: Jani Nikula @ 2014-02-22 22:25 UTC (permalink / raw)
  To: notmuch

Test the upgrade from probabilistic to boolean folder: terms, and
addition of path: terms.
---
 test/T530-upgrade.sh | 103 +++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 103 insertions(+)
 create mode 100755 test/T530-upgrade.sh

diff --git a/test/T530-upgrade.sh b/test/T530-upgrade.sh
new file mode 100755
index 000000000000..cf9914e380be
--- /dev/null
+++ b/test/T530-upgrade.sh
@@ -0,0 +1,103 @@
+#!/usr/bin/env bash
+test_description="database upgrade"
+
+. ./test-lib.sh
+
+tar zxf $TEST_DIRECTORY/test-databases/database-v1.tar.gz -C ${MAIL_DIR} --strip-components=1
+
+test_begin_subtest "folder: search does not work with old database version"
+output=$(notmuch search folder:foo)
+test_expect_equal "$output" ""
+
+test_begin_subtest "path: search does not work with old database version"
+output=$(notmuch search path:foo)
+test_expect_equal "$output" ""
+
+test_begin_subtest "database upgrade from format version 1"
+output=$(notmuch new)
+test_expect_equal "$output" "\
+Welcome to a new version of notmuch! Your database will now be upgraded.
+Your notmuch database has now been upgraded to database format version 2.
+No new mail."
+
+test_begin_subtest "folder: no longer matches in the middle of path"
+output=$(notmuch search folder:baz)
+test_expect_equal "$output" ""
+
+test_begin_subtest "folder: search"
+output=$(notmuch search --output=files folder:foo | sed -e "s,$MAIL_DIR,MAIL_DIR," | sort)
+# bar/baz/05:2, and new/03:2, are duplicates of foo/05:2, and
+# foo/new/03:2, respectively
+test_expect_equal "$output" "MAIL_DIR/bar/baz/05:2,
+MAIL_DIR/foo/05:2,
+MAIL_DIR/foo/06:2,
+MAIL_DIR/foo/cur/07:2,
+MAIL_DIR/foo/cur/08:2,
+MAIL_DIR/foo/new/03:2,
+MAIL_DIR/foo/new/09:2,
+MAIL_DIR/foo/new/10:2,
+MAIL_DIR/new/03:2,"
+
+test_begin_subtest "top level folder: search"
+output=$(notmuch search --output=files folder:'""' | sed -e "s,$MAIL_DIR,MAIL_DIR," | sort)
+# foo/new/03:2, is a duplicate of new/03:2,
+test_expect_equal "$output" "MAIL_DIR/01:2,
+MAIL_DIR/02:2,
+MAIL_DIR/cur/29:2,
+MAIL_DIR/cur/30:2,
+MAIL_DIR/cur/31:2,
+MAIL_DIR/cur/32:2,
+MAIL_DIR/cur/33:2,
+MAIL_DIR/cur/34:2,
+MAIL_DIR/cur/35:2,
+MAIL_DIR/cur/36:2,
+MAIL_DIR/cur/37:2,
+MAIL_DIR/cur/38:2,
+MAIL_DIR/cur/39:2,
+MAIL_DIR/cur/40:2,
+MAIL_DIR/cur/41:2,
+MAIL_DIR/cur/42:2,
+MAIL_DIR/cur/43:2,
+MAIL_DIR/cur/44:2,
+MAIL_DIR/cur/45:2,
+MAIL_DIR/cur/46:2,
+MAIL_DIR/cur/47:2,
+MAIL_DIR/cur/48:2,
+MAIL_DIR/cur/49:2,
+MAIL_DIR/cur/50:2,
+MAIL_DIR/cur/52:2,
+MAIL_DIR/cur/53:2,
+MAIL_DIR/foo/new/03:2,
+MAIL_DIR/new/03:2,
+MAIL_DIR/new/04:2,"
+
+test_begin_subtest "path: search"
+output=$(notmuch search --output=files path:"bar" | sed -e "s,$MAIL_DIR,MAIL_DIR," | sort)
+# foo/05:2, is a duplicate of bar/baz/05:2,
+test_expect_equal "$output" "MAIL_DIR/bar/17:2,
+MAIL_DIR/bar/18:2,"
+
+test_begin_subtest "top level path: search"
+output=$(notmuch search --output=files path:'""' | sed -e "s,$MAIL_DIR,MAIL_DIR," | sort)
+test_expect_equal "$output" "MAIL_DIR/01:2,
+MAIL_DIR/02:2,"
+
+test_begin_subtest "recursive path: search"
+output=$(notmuch search --output=files path:"bar/**" | sed -e "s,$MAIL_DIR,MAIL_DIR," | sort)
+# foo/05:2, is a duplicate of bar/baz/05:2,
+test_expect_equal "$output" "MAIL_DIR/bar/17:2,
+MAIL_DIR/bar/18:2,
+MAIL_DIR/bar/baz/05:2,
+MAIL_DIR/bar/baz/23:2,
+MAIL_DIR/bar/baz/24:2,
+MAIL_DIR/bar/baz/cur/25:2,
+MAIL_DIR/bar/baz/cur/26:2,
+MAIL_DIR/bar/baz/new/27:2,
+MAIL_DIR/bar/baz/new/28:2,
+MAIL_DIR/bar/cur/19:2,
+MAIL_DIR/bar/cur/20:2,
+MAIL_DIR/bar/new/21:2,
+MAIL_DIR/bar/new/22:2,
+MAIL_DIR/foo/05:2,"
+
+test_done
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH v2 12/13] man: update man pages for folder: and path: search terms
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
                   ` (10 preceding siblings ...)
  2014-02-22 22:25 ` [PATCH v2 11/13] test: add database upgrade test from format version 1 to 2 Jani Nikula
@ 2014-02-22 22:25 ` Jani Nikula
  2014-02-22 22:25 ` [PATCH v2 13/13] man: try to clarify the folder: and path: vs. --output=files confusion Jani Nikula
                   ` (2 subsequent siblings)
  14 siblings, 0 replies; 25+ messages in thread
From: Jani Nikula @ 2014-02-22 22:25 UTC (permalink / raw)
  To: notmuch

---
 man/man7/notmuch-search-terms.7 | 28 ++++++++++++++++++++++------
 1 file changed, 22 insertions(+), 6 deletions(-)

diff --git a/man/man7/notmuch-search-terms.7 b/man/man7/notmuch-search-terms.7
index a768b630a4d1..907403dd6f2e 100644
--- a/man/man7/notmuch-search-terms.7
+++ b/man/man7/notmuch-search-terms.7
@@ -54,6 +54,8 @@ terms to match against specific portions of an email, (where
 
 	folder:<directory-path>
 
+	path:<directory-path> or path:<directory-path>/**
+
 	date:<since>..<until>
 
 The
@@ -101,12 +103,26 @@ thread ID values can be seen in the first column of output from
 
 The
 .B folder:
-prefix can be used to search for email message files that are
-contained within particular directories within the mail store. If the
-same email message has multiple message files associated with it, it's
-sufficient for a match that at least one of the files is contained
-within a matching directory. Only the directory components below the
-top-level mail database path are available to be searched.
+and
+.B path:
+prefixes can be used to search for email message files that are
+contained within particular directories within the mail store. The
+directories are specified relative from the top-level mail database
+path, and thus only the directory components below that are available
+to be searched.
+
+The
+.B folder:
+prefix matches messages in the specified maildir folder, i.e. in the
+specified directory and its "new" and "cur" subdirectories. The
+.B path:
+prefix matches messages in the specified directory only, unless the
+"/**" suffix is used to denote the specified directory and all its
+subdirectories recursively. For both, the empty string "" matches the
+top level maildir folder or directory. If the same email message has
+multiple message files associated with it, it's sufficient for a match
+that at least one of the files is contained within a matching
+directory.
 
 The
 .B date:
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH v2 13/13] man: try to clarify the folder: and path: vs. --output=files confusion
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
                   ` (11 preceding siblings ...)
  2014-02-22 22:25 ` [PATCH v2 12/13] man: update man pages for folder: and path: search terms Jani Nikula
@ 2014-02-22 22:25 ` Jani Nikula
  2014-02-22 23:57 ` [PATCH v2 00/13] literal folder: prefix, new path: prefix Mark Walters
  2014-03-02 19:11 ` David Bremner
  14 siblings, 0 replies; 25+ messages in thread
From: Jani Nikula @ 2014-02-22 22:25 UTC (permalink / raw)
  To: notmuch

---
 man/man1/notmuch-search.1 | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/man/man1/notmuch-search.1 b/man/man1/notmuch-search.1
index 55a81e79fce4..a2b1ae43f411 100644
--- a/man/man1/notmuch-search.1
+++ b/man/man1/notmuch-search.1
@@ -82,8 +82,14 @@ one per line (\-\-format=text), separated by null characters
 S-Expression list (\-\-format=sexp).
 
 Note that each message may have multiple filenames associated with it.
-All of them are included in the output, unless limited with the
-\-\-duplicate=N option.
+All of them are included in the output (unless limited with the
+\-\-duplicate=N option). This may be particularly confusing for
+.B folder:
+or
+.B path:
+searches in a specified directory, as the messages may have duplicates
+in other directories that are included in the output, although these
+files alone would not match the search.
 .RE
 .RS 4
 .TP 4
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
                   ` (12 preceding siblings ...)
  2014-02-22 22:25 ` [PATCH v2 13/13] man: try to clarify the folder: and path: vs. --output=files confusion Jani Nikula
@ 2014-02-22 23:57 ` Mark Walters
  2014-02-23 11:15   ` Tomi Ollila
  2014-03-02 19:11 ` David Bremner
  14 siblings, 1 reply; 25+ messages in thread
From: Mark Walters @ 2014-02-22 23:57 UTC (permalink / raw)
  To: Jani Nikula, notmuch


I have read most of this series, tested it and run the tests and LGTM +1.

I read the C code fairly carefully, the tests rather less so but they
looked sane, and I didn't really look at patch 9 for building old
databases.

Best wishes

Mark




On Sat, 22 Feb 2014, Jani Nikula <jani@nikula.org> wrote:
> Hi all, this is v2 of id:cover.1389304779.git.jani@nikula.org.
>
> The new path: prefix is a literal boolean prefix matching the paths,
> relative from the maildir root, of the message files. There's no
> interpretation of the maildir special cur/new folders, but a recursive
> match is provided with "/**" suffix. See the patch for details.
>
> The folder: prefix becomes a literal boolean prefix, similar to path:,
> except it matches the maildir cur/new folders in addition to the
> specified path. There's no recursive version.
>
> Patches 1-5 add the above.
>
> Patches 6-8 change the test infrastructure to make it easier to add
> multiple corpuses, and adds a new test for path: and folder:.
>
> Patches 9-11 add support for testing the database upgrade.
>
> Patches 12-13 update man pages.
>
>
> I've dropped most of the content in patches 7 and 10 due to their
> size. The patches (and the whole series) are available in the
> boolean-folder-and-path-v2 branch at
> git://gitorious.org/jani/notmuch.git. Web interface at
> https://gitorious.org/jani/notmuch/commits/0b3dd2d1cc6c413ea07ea326883ac448499c0e79.
>
>
> WARNING! The change requires a database format version bump, and a
> database upgrade, which is automatically done on 'notmuch new'. The
> upgrade is irreversible if you want to try this on your database! A
> complete database rebuild is required for reverting the database format
> version. Make sure your backups are in order!
>
>
> BR,
> Jani.
>
>
> Jani Nikula (13):
>   lib: refactor folder term update after filename removal
>   lib: add support for path: prefix searches
>   test: make insert test use the path: prefix
>   lib: make folder: prefix literal
>   test: fix test for literal folder: search
>   test: make it possible to have several corpora
>   test: add new corpus with folders
>   test: add tests for the new boolean folder: and path: prefixes
>   devel: add script to generate test databases
>   test: add test database in format version 1
>   test: add database upgrade test from format version 1 to 2
>   man: update man pages for folder: and path: search terms
>   man: try to clarify the folder: and path: vs. --output=files confusion
>
>  devel/gen-testdb.sh                    | 124 ++++++++++++
>  lib/database.cc                        |  45 ++++-
>  lib/message.cc                         | 249 ++++++++++++++++-------
>  lib/notmuch-private.h                  |   3 +
>  man/man1/notmuch-search.1              |  10 +-
>  man/man7/notmuch-search-terms.7        |  28 ++-
>  test/.gitignore                        |   2 +-
>  test/Makefile.local                    |   2 +-
>  test/T070-insert.sh                    |  10 +-
>  test/T100-search-by-folder.sh          |  24 ++-
>  test/T101-search-by-folder-and-path.sh |  83 ++++++++
>  test/T480-hex-escaping.sh              |   4 +-
>  test/T530-upgrade.sh                   | 103 ++++++++++
>  test/corpus/{ => default}/cur/01:2,    |   0
>  test/corpus/{ => default}/cur/02:2,    |   0
>  test/corpus/{ => default}/cur/03:2,    |   0
>  test/corpus/{ => default}/cur/04:2,    |   0
>  test/corpus/{ => default}/cur/05:2,    |   0
>  test/corpus/{ => default}/cur/06:2,    |   0
>  test/corpus/{ => default}/cur/07:2,    |   0
>  test/corpus/{ => default}/cur/08:2,    |   0
>  test/corpus/{ => default}/cur/09:2,    |   0
>  test/corpus/{ => default}/cur/10:2,    |   0
>  test/corpus/{ => default}/cur/11:2,    |   0
>  test/corpus/{ => default}/cur/12:2,    |   0
>  test/corpus/{ => default}/cur/13:2,    |   0
>  test/corpus/{ => default}/cur/14:2,    |   0
>  test/corpus/{ => default}/cur/15:2,    |   0
>  test/corpus/{ => default}/cur/16:2,    |   0
>  test/corpus/{ => default}/cur/17:2,    |   0
>  test/corpus/{ => default}/cur/18:2,    |   0
>  test/corpus/{ => default}/cur/19:2,    |   0
>  test/corpus/{ => default}/cur/20:2,    |   0
>  test/corpus/{ => default}/cur/21:2,    |   0
>  test/corpus/{ => default}/cur/22:2,    |   0
>  test/corpus/{ => default}/cur/23:2,    |   0
>  test/corpus/{ => default}/cur/24:2,    |   0
>  test/corpus/{ => default}/cur/25:2,    |   0
>  test/corpus/{ => default}/cur/26:2,    |   0
>  test/corpus/{ => default}/cur/27:2,    |   0
>  test/corpus/{ => default}/cur/28:2,    |   0
>  test/corpus/{ => default}/cur/29:2,    |   0
>  test/corpus/{ => default}/cur/30:2,    |   0
>  test/corpus/{ => default}/cur/31:2,    |   0
>  test/corpus/{ => default}/cur/32:2,    |   0
>  test/corpus/{ => default}/cur/33:2,    |   0
>  test/corpus/{ => default}/cur/34:2,    |   0
>  test/corpus/{ => default}/cur/35:2,    |   0
>  test/corpus/{ => default}/cur/36:2,    |   0
>  test/corpus/{ => default}/cur/37:2,    |   0
>  test/corpus/{ => default}/cur/38:2,    |   0
>  test/corpus/{ => default}/cur/39:2,    |   0
>  test/corpus/{ => default}/cur/40:2,    |   0
>  test/corpus/{ => default}/cur/41:2,    |   0
>  test/corpus/{ => default}/cur/42:2,    |   0
>  test/corpus/{ => default}/cur/43:2,    |   0
>  test/corpus/{ => default}/cur/44:2,    |   0
>  test/corpus/{ => default}/cur/45:2,    |   0
>  test/corpus/{ => default}/cur/46:2,    |   0
>  test/corpus/{ => default}/cur/47:2,    |   0
>  test/corpus/{ => default}/cur/48:2,    |   0
>  test/corpus/{ => default}/cur/49:2,    |   0
>  test/corpus/{ => default}/cur/50:2,    |   0
>  test/corpus/{ => default}/cur/51:2,    |   0
>  test/corpus/{ => default}/cur/52:2,    |   0
>  test/corpus/{ => default}/cur/53:2,    |   0
>  test/corpus/folders/01:2,              |  34 ++++
>  test/corpus/folders/02:2,              |  32 +++
>  test/corpus/folders/bar/17:2,          |  23 +++
>  test/corpus/folders/bar/18:2,          |  12 ++
>  test/corpus/folders/bar/baz/05:2,      | 104 ++++++++++
>  test/corpus/folders/bar/baz/23:2,      | 145 +++++++++++++
>  test/corpus/folders/bar/baz/24:2,      | 204 +++++++++++++++++++
>  test/corpus/folders/bar/baz/cur/25:2,  |  32 +++
>  test/corpus/folders/bar/baz/cur/26:2,  | 121 +++++++++++
>  test/corpus/folders/bar/baz/new/27:2,  |  21 ++
>  test/corpus/folders/bar/baz/new/28:2,  |  38 ++++
>  test/corpus/folders/bar/cur/19:2,      | 360 +++++++++++++++++++++++++++++++++
>  test/corpus/folders/bar/cur/20:2,      | 101 +++++++++
>  test/corpus/folders/bar/new/21:2,      | 102 ++++++++++
>  test/corpus/folders/bar/new/22:2,      |  84 ++++++++
>  test/corpus/folders/cur/29:2,          |  21 ++
>  test/corpus/folders/cur/30:2,          |  75 +++++++
>  test/corpus/folders/cur/31:2,          |  31 +++
>  test/corpus/folders/cur/32:2,          | 165 +++++++++++++++
>  test/corpus/folders/cur/33:2,          |  13 ++
>  test/corpus/folders/cur/34:2,          |  46 +++++
>  test/corpus/folders/cur/35:2,          |  24 +++
>  test/corpus/folders/cur/36:2,          |  25 +++
>  test/corpus/folders/cur/37:2,          |  22 ++
>  test/corpus/folders/cur/38:2,          |  40 ++++
>  test/corpus/folders/cur/39:2,          |  32 +++
>  test/corpus/folders/cur/40:2,          |  31 +++
>  test/corpus/folders/cur/41:2,          |  37 ++++
>  test/corpus/folders/cur/42:2,          |  30 +++
>  test/corpus/folders/cur/43:2,          |  26 +++
>  test/corpus/folders/cur/44:2,          |  29 +++
>  test/corpus/folders/cur/45:2,          |  41 ++++
>  test/corpus/folders/cur/46:2,          |  57 ++++++
>  test/corpus/folders/cur/47:2,          |  84 ++++++++
>  test/corpus/folders/cur/48:2,          |  17 ++
>  test/corpus/folders/cur/49:2,          |  33 +++
>  test/corpus/folders/cur/50:2,          |  39 ++++
>  test/corpus/folders/cur/52:2,          |  39 ++++
>  test/corpus/folders/cur/53:2,          |  20 ++
>  test/corpus/folders/foo/05:2,          | 104 ++++++++++
>  test/corpus/folders/foo/06:2,          |  36 ++++
>  test/corpus/folders/foo/baz/11:2,      |  27 +++
>  test/corpus/folders/foo/baz/12:2,      |  27 +++
>  test/corpus/folders/foo/baz/cur/13:2,  | 178 ++++++++++++++++
>  test/corpus/folders/foo/baz/cur/14:2,  |  39 ++++
>  test/corpus/folders/foo/baz/new/15:2,  |  22 ++
>  test/corpus/folders/foo/baz/new/16:2,  |  27 +++
>  test/corpus/folders/foo/cur/07:2,      |  57 ++++++
>  test/corpus/folders/foo/cur/08:2,      |  87 ++++++++
>  test/corpus/folders/foo/new/03:2,      |  93 +++++++++
>  test/corpus/folders/foo/new/09:2,      |  33 +++
>  test/corpus/folders/foo/new/10:2,      |  54 +++++
>  test/corpus/folders/new/03:2,          |  93 +++++++++
>  test/corpus/folders/new/04:2,          |  84 ++++++++
>  test/notmuch-test                      |   2 +-
>  test/test-databases/README             |   5 +
>  test/test-databases/database-v1.tar.gz | Bin 0 -> 262063 bytes
>  test/test-lib.sh                       |  21 +-
>  124 files changed, 3954 insertions(+), 112 deletions(-)
>  create mode 100755 devel/gen-testdb.sh
>  create mode 100755 test/T101-search-by-folder-and-path.sh
>  create mode 100755 test/T530-upgrade.sh
>  rename test/corpus/{ => default}/cur/01:2, (100%)
>  rename test/corpus/{ => default}/cur/02:2, (100%)
>  rename test/corpus/{ => default}/cur/03:2, (100%)
>  rename test/corpus/{ => default}/cur/04:2, (100%)
>  rename test/corpus/{ => default}/cur/05:2, (100%)
>  rename test/corpus/{ => default}/cur/06:2, (100%)
>  rename test/corpus/{ => default}/cur/07:2, (100%)
>  rename test/corpus/{ => default}/cur/08:2, (100%)
>  rename test/corpus/{ => default}/cur/09:2, (100%)
>  rename test/corpus/{ => default}/cur/10:2, (100%)
>  rename test/corpus/{ => default}/cur/11:2, (100%)
>  rename test/corpus/{ => default}/cur/12:2, (100%)
>  rename test/corpus/{ => default}/cur/13:2, (100%)
>  rename test/corpus/{ => default}/cur/14:2, (100%)
>  rename test/corpus/{ => default}/cur/15:2, (100%)
>  rename test/corpus/{ => default}/cur/16:2, (100%)
>  rename test/corpus/{ => default}/cur/17:2, (100%)
>  rename test/corpus/{ => default}/cur/18:2, (100%)
>  rename test/corpus/{ => default}/cur/19:2, (100%)
>  rename test/corpus/{ => default}/cur/20:2, (100%)
>  rename test/corpus/{ => default}/cur/21:2, (100%)
>  rename test/corpus/{ => default}/cur/22:2, (100%)
>  rename test/corpus/{ => default}/cur/23:2, (100%)
>  rename test/corpus/{ => default}/cur/24:2, (100%)
>  rename test/corpus/{ => default}/cur/25:2, (100%)
>  rename test/corpus/{ => default}/cur/26:2, (100%)
>  rename test/corpus/{ => default}/cur/27:2, (100%)
>  rename test/corpus/{ => default}/cur/28:2, (100%)
>  rename test/corpus/{ => default}/cur/29:2, (100%)
>  rename test/corpus/{ => default}/cur/30:2, (100%)
>  rename test/corpus/{ => default}/cur/31:2, (100%)
>  rename test/corpus/{ => default}/cur/32:2, (100%)
>  rename test/corpus/{ => default}/cur/33:2, (100%)
>  rename test/corpus/{ => default}/cur/34:2, (100%)
>  rename test/corpus/{ => default}/cur/35:2, (100%)
>  rename test/corpus/{ => default}/cur/36:2, (100%)
>  rename test/corpus/{ => default}/cur/37:2, (100%)
>  rename test/corpus/{ => default}/cur/38:2, (100%)
>  rename test/corpus/{ => default}/cur/39:2, (100%)
>  rename test/corpus/{ => default}/cur/40:2, (100%)
>  rename test/corpus/{ => default}/cur/41:2, (100%)
>  rename test/corpus/{ => default}/cur/42:2, (100%)
>  rename test/corpus/{ => default}/cur/43:2, (100%)
>  rename test/corpus/{ => default}/cur/44:2, (100%)
>  rename test/corpus/{ => default}/cur/45:2, (100%)
>  rename test/corpus/{ => default}/cur/46:2, (100%)
>  rename test/corpus/{ => default}/cur/47:2, (100%)
>  rename test/corpus/{ => default}/cur/48:2, (100%)
>  rename test/corpus/{ => default}/cur/49:2, (100%)
>  rename test/corpus/{ => default}/cur/50:2, (100%)
>  rename test/corpus/{ => default}/cur/51:2, (100%)
>  rename test/corpus/{ => default}/cur/52:2, (100%)
>  rename test/corpus/{ => default}/cur/53:2, (100%)
>  create mode 100644 test/corpus/folders/01:2,
>  create mode 100644 test/corpus/folders/02:2,
>  create mode 100644 test/corpus/folders/bar/17:2,
>  create mode 100644 test/corpus/folders/bar/18:2,
>  create mode 100644 test/corpus/folders/bar/baz/05:2,
>  create mode 100644 test/corpus/folders/bar/baz/23:2,
>  create mode 100644 test/corpus/folders/bar/baz/24:2,
>  create mode 100644 test/corpus/folders/bar/baz/cur/25:2,
>  create mode 100644 test/corpus/folders/bar/baz/cur/26:2,
>  create mode 100644 test/corpus/folders/bar/baz/new/27:2,
>  create mode 100644 test/corpus/folders/bar/baz/new/28:2,
>  create mode 100644 test/corpus/folders/bar/cur/19:2,
>  create mode 100644 test/corpus/folders/bar/cur/20:2,
>  create mode 100644 test/corpus/folders/bar/new/21:2,
>  create mode 100644 test/corpus/folders/bar/new/22:2,
>  create mode 100644 test/corpus/folders/cur/29:2,
>  create mode 100644 test/corpus/folders/cur/30:2,
>  create mode 100644 test/corpus/folders/cur/31:2,
>  create mode 100644 test/corpus/folders/cur/32:2,
>  create mode 100644 test/corpus/folders/cur/33:2,
>  create mode 100644 test/corpus/folders/cur/34:2,
>  create mode 100644 test/corpus/folders/cur/35:2,
>  create mode 100644 test/corpus/folders/cur/36:2,
>  create mode 100644 test/corpus/folders/cur/37:2,
>  create mode 100644 test/corpus/folders/cur/38:2,
>  create mode 100644 test/corpus/folders/cur/39:2,
>  create mode 100644 test/corpus/folders/cur/40:2,
>  create mode 100644 test/corpus/folders/cur/41:2,
>  create mode 100644 test/corpus/folders/cur/42:2,
>  create mode 100644 test/corpus/folders/cur/43:2,
>  create mode 100644 test/corpus/folders/cur/44:2,
>  create mode 100644 test/corpus/folders/cur/45:2,
>  create mode 100644 test/corpus/folders/cur/46:2,
>  create mode 100644 test/corpus/folders/cur/47:2,
>  create mode 100644 test/corpus/folders/cur/48:2,
>  create mode 100644 test/corpus/folders/cur/49:2,
>  create mode 100644 test/corpus/folders/cur/50:2,
>  create mode 100644 test/corpus/folders/cur/52:2,
>  create mode 100644 test/corpus/folders/cur/53:2,
>  create mode 100644 test/corpus/folders/foo/05:2,
>  create mode 100644 test/corpus/folders/foo/06:2,
>  create mode 100644 test/corpus/folders/foo/baz/11:2,
>  create mode 100644 test/corpus/folders/foo/baz/12:2,
>  create mode 100644 test/corpus/folders/foo/baz/cur/13:2,
>  create mode 100644 test/corpus/folders/foo/baz/cur/14:2,
>  create mode 100644 test/corpus/folders/foo/baz/new/15:2,
>  create mode 100644 test/corpus/folders/foo/baz/new/16:2,
>  create mode 100644 test/corpus/folders/foo/cur/07:2,
>  create mode 100644 test/corpus/folders/foo/cur/08:2,
>  create mode 100644 test/corpus/folders/foo/new/03:2,
>  create mode 100644 test/corpus/folders/foo/new/09:2,
>  create mode 100644 test/corpus/folders/foo/new/10:2,
>  create mode 100644 test/corpus/folders/new/03:2,
>  create mode 100644 test/corpus/folders/new/04:2,
>  create mode 100644 test/test-databases/README
>  create mode 100644 test/test-databases/database-v1.tar.gz
>
> -- 
> 1.8.5.3
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix
  2014-02-22 23:57 ` [PATCH v2 00/13] literal folder: prefix, new path: prefix Mark Walters
@ 2014-02-23 11:15   ` Tomi Ollila
  0 siblings, 0 replies; 25+ messages in thread
From: Tomi Ollila @ 2014-02-23 11:15 UTC (permalink / raw)
  To: Mark Walters, Jani Nikula, notmuch

On Sun, Feb 23 2014, Mark Walters <markwalters1009@gmail.com> wrote:

> I have read most of this series, tested it and run the tests and LGTM +1.
>
> I read the C code fairly carefully, the tests rather less so but they
> looked sane, and I didn't really look at patch 9 for building old
> databases.

Well, I can add (at this point) that patch 9 is tolerable...

>
> Best wishes
>
> Mark

Tomi


>
>
>
>
> On Sat, 22 Feb 2014, Jani Nikula <jani@nikula.org> wrote:
>> Hi all, this is v2 of id:cover.1389304779.git.jani@nikula.org.
>>
>> The new path: prefix is a literal boolean prefix matching the paths,
>> relative from the maildir root, of the message files. There's no
>> interpretation of the maildir special cur/new folders, but a recursive
>> match is provided with "/**" suffix. See the patch for details.
>>
>> The folder: prefix becomes a literal boolean prefix, similar to path:,
>> except it matches the maildir cur/new folders in addition to the
>> specified path. There's no recursive version.
>>
>> Patches 1-5 add the above.
>>
>> Patches 6-8 change the test infrastructure to make it easier to add
>> multiple corpuses, and adds a new test for path: and folder:.
>>
>> Patches 9-11 add support for testing the database upgrade.
>>
>> Patches 12-13 update man pages.
>>
>>
>> I've dropped most of the content in patches 7 and 10 due to their
>> size. The patches (and the whole series) are available in the
>> boolean-folder-and-path-v2 branch at
>> git://gitorious.org/jani/notmuch.git. Web interface at
>> https://gitorious.org/jani/notmuch/commits/0b3dd2d1cc6c413ea07ea326883ac448499c0e79.
>>
>>
>> WARNING! The change requires a database format version bump, and a
>> database upgrade, which is automatically done on 'notmuch new'. The
>> upgrade is irreversible if you want to try this on your database! A
>> complete database rebuild is required for reverting the database format
>> version. Make sure your backups are in order!
>>
>>
>> BR,
>> Jani.
>>
>>
>> Jani Nikula (13):
>>   lib: refactor folder term update after filename removal
>>   lib: add support for path: prefix searches
>>   test: make insert test use the path: prefix
>>   lib: make folder: prefix literal
>>   test: fix test for literal folder: search
>>   test: make it possible to have several corpora
>>   test: add new corpus with folders
>>   test: add tests for the new boolean folder: and path: prefixes
>>   devel: add script to generate test databases
>>   test: add test database in format version 1
>>   test: add database upgrade test from format version 1 to 2
>>   man: update man pages for folder: and path: search terms
>>   man: try to clarify the folder: and path: vs. --output=files confusion
>>
>>  devel/gen-testdb.sh                    | 124 ++++++++++++
>>  lib/database.cc                        |  45 ++++-
>>  lib/message.cc                         | 249 ++++++++++++++++-------
>>  lib/notmuch-private.h                  |   3 +
>>  man/man1/notmuch-search.1              |  10 +-
>>  man/man7/notmuch-search-terms.7        |  28 ++-
>>  test/.gitignore                        |   2 +-
>>  test/Makefile.local                    |   2 +-
>>  test/T070-insert.sh                    |  10 +-
>>  test/T100-search-by-folder.sh          |  24 ++-
>>  test/T101-search-by-folder-and-path.sh |  83 ++++++++
>>  test/T480-hex-escaping.sh              |   4 +-
>>  test/T530-upgrade.sh                   | 103 ++++++++++
>>  test/corpus/{ => default}/cur/01:2,    |   0
>>  test/corpus/{ => default}/cur/02:2,    |   0
>>  test/corpus/{ => default}/cur/03:2,    |   0
>>  test/corpus/{ => default}/cur/04:2,    |   0
>>  test/corpus/{ => default}/cur/05:2,    |   0
>>  test/corpus/{ => default}/cur/06:2,    |   0
>>  test/corpus/{ => default}/cur/07:2,    |   0
>>  test/corpus/{ => default}/cur/08:2,    |   0
>>  test/corpus/{ => default}/cur/09:2,    |   0
>>  test/corpus/{ => default}/cur/10:2,    |   0
>>  test/corpus/{ => default}/cur/11:2,    |   0
>>  test/corpus/{ => default}/cur/12:2,    |   0
>>  test/corpus/{ => default}/cur/13:2,    |   0
>>  test/corpus/{ => default}/cur/14:2,    |   0
>>  test/corpus/{ => default}/cur/15:2,    |   0
>>  test/corpus/{ => default}/cur/16:2,    |   0
>>  test/corpus/{ => default}/cur/17:2,    |   0
>>  test/corpus/{ => default}/cur/18:2,    |   0
>>  test/corpus/{ => default}/cur/19:2,    |   0
>>  test/corpus/{ => default}/cur/20:2,    |   0
>>  test/corpus/{ => default}/cur/21:2,    |   0
>>  test/corpus/{ => default}/cur/22:2,    |   0
>>  test/corpus/{ => default}/cur/23:2,    |   0
>>  test/corpus/{ => default}/cur/24:2,    |   0
>>  test/corpus/{ => default}/cur/25:2,    |   0
>>  test/corpus/{ => default}/cur/26:2,    |   0
>>  test/corpus/{ => default}/cur/27:2,    |   0
>>  test/corpus/{ => default}/cur/28:2,    |   0
>>  test/corpus/{ => default}/cur/29:2,    |   0
>>  test/corpus/{ => default}/cur/30:2,    |   0
>>  test/corpus/{ => default}/cur/31:2,    |   0
>>  test/corpus/{ => default}/cur/32:2,    |   0
>>  test/corpus/{ => default}/cur/33:2,    |   0
>>  test/corpus/{ => default}/cur/34:2,    |   0
>>  test/corpus/{ => default}/cur/35:2,    |   0
>>  test/corpus/{ => default}/cur/36:2,    |   0
>>  test/corpus/{ => default}/cur/37:2,    |   0
>>  test/corpus/{ => default}/cur/38:2,    |   0
>>  test/corpus/{ => default}/cur/39:2,    |   0
>>  test/corpus/{ => default}/cur/40:2,    |   0
>>  test/corpus/{ => default}/cur/41:2,    |   0
>>  test/corpus/{ => default}/cur/42:2,    |   0
>>  test/corpus/{ => default}/cur/43:2,    |   0
>>  test/corpus/{ => default}/cur/44:2,    |   0
>>  test/corpus/{ => default}/cur/45:2,    |   0
>>  test/corpus/{ => default}/cur/46:2,    |   0
>>  test/corpus/{ => default}/cur/47:2,    |   0
>>  test/corpus/{ => default}/cur/48:2,    |   0
>>  test/corpus/{ => default}/cur/49:2,    |   0
>>  test/corpus/{ => default}/cur/50:2,    |   0
>>  test/corpus/{ => default}/cur/51:2,    |   0
>>  test/corpus/{ => default}/cur/52:2,    |   0
>>  test/corpus/{ => default}/cur/53:2,    |   0
>>  test/corpus/folders/01:2,              |  34 ++++
>>  test/corpus/folders/02:2,              |  32 +++
>>  test/corpus/folders/bar/17:2,          |  23 +++
>>  test/corpus/folders/bar/18:2,          |  12 ++
>>  test/corpus/folders/bar/baz/05:2,      | 104 ++++++++++
>>  test/corpus/folders/bar/baz/23:2,      | 145 +++++++++++++
>>  test/corpus/folders/bar/baz/24:2,      | 204 +++++++++++++++++++
>>  test/corpus/folders/bar/baz/cur/25:2,  |  32 +++
>>  test/corpus/folders/bar/baz/cur/26:2,  | 121 +++++++++++
>>  test/corpus/folders/bar/baz/new/27:2,  |  21 ++
>>  test/corpus/folders/bar/baz/new/28:2,  |  38 ++++
>>  test/corpus/folders/bar/cur/19:2,      | 360 +++++++++++++++++++++++++++++++++
>>  test/corpus/folders/bar/cur/20:2,      | 101 +++++++++
>>  test/corpus/folders/bar/new/21:2,      | 102 ++++++++++
>>  test/corpus/folders/bar/new/22:2,      |  84 ++++++++
>>  test/corpus/folders/cur/29:2,          |  21 ++
>>  test/corpus/folders/cur/30:2,          |  75 +++++++
>>  test/corpus/folders/cur/31:2,          |  31 +++
>>  test/corpus/folders/cur/32:2,          | 165 +++++++++++++++
>>  test/corpus/folders/cur/33:2,          |  13 ++
>>  test/corpus/folders/cur/34:2,          |  46 +++++
>>  test/corpus/folders/cur/35:2,          |  24 +++
>>  test/corpus/folders/cur/36:2,          |  25 +++
>>  test/corpus/folders/cur/37:2,          |  22 ++
>>  test/corpus/folders/cur/38:2,          |  40 ++++
>>  test/corpus/folders/cur/39:2,          |  32 +++
>>  test/corpus/folders/cur/40:2,          |  31 +++
>>  test/corpus/folders/cur/41:2,          |  37 ++++
>>  test/corpus/folders/cur/42:2,          |  30 +++
>>  test/corpus/folders/cur/43:2,          |  26 +++
>>  test/corpus/folders/cur/44:2,          |  29 +++
>>  test/corpus/folders/cur/45:2,          |  41 ++++
>>  test/corpus/folders/cur/46:2,          |  57 ++++++
>>  test/corpus/folders/cur/47:2,          |  84 ++++++++
>>  test/corpus/folders/cur/48:2,          |  17 ++
>>  test/corpus/folders/cur/49:2,          |  33 +++
>>  test/corpus/folders/cur/50:2,          |  39 ++++
>>  test/corpus/folders/cur/52:2,          |  39 ++++
>>  test/corpus/folders/cur/53:2,          |  20 ++
>>  test/corpus/folders/foo/05:2,          | 104 ++++++++++
>>  test/corpus/folders/foo/06:2,          |  36 ++++
>>  test/corpus/folders/foo/baz/11:2,      |  27 +++
>>  test/corpus/folders/foo/baz/12:2,      |  27 +++
>>  test/corpus/folders/foo/baz/cur/13:2,  | 178 ++++++++++++++++
>>  test/corpus/folders/foo/baz/cur/14:2,  |  39 ++++
>>  test/corpus/folders/foo/baz/new/15:2,  |  22 ++
>>  test/corpus/folders/foo/baz/new/16:2,  |  27 +++
>>  test/corpus/folders/foo/cur/07:2,      |  57 ++++++
>>  test/corpus/folders/foo/cur/08:2,      |  87 ++++++++
>>  test/corpus/folders/foo/new/03:2,      |  93 +++++++++
>>  test/corpus/folders/foo/new/09:2,      |  33 +++
>>  test/corpus/folders/foo/new/10:2,      |  54 +++++
>>  test/corpus/folders/new/03:2,          |  93 +++++++++
>>  test/corpus/folders/new/04:2,          |  84 ++++++++
>>  test/notmuch-test                      |   2 +-
>>  test/test-databases/README             |   5 +
>>  test/test-databases/database-v1.tar.gz | Bin 0 -> 262063 bytes
>>  test/test-lib.sh                       |  21 +-
>>  124 files changed, 3954 insertions(+), 112 deletions(-)
>>  create mode 100755 devel/gen-testdb.sh
>>  create mode 100755 test/T101-search-by-folder-and-path.sh
>>  create mode 100755 test/T530-upgrade.sh
>>  rename test/corpus/{ => default}/cur/01:2, (100%)
>>  rename test/corpus/{ => default}/cur/02:2, (100%)
>>  rename test/corpus/{ => default}/cur/03:2, (100%)
>>  rename test/corpus/{ => default}/cur/04:2, (100%)
>>  rename test/corpus/{ => default}/cur/05:2, (100%)
>>  rename test/corpus/{ => default}/cur/06:2, (100%)
>>  rename test/corpus/{ => default}/cur/07:2, (100%)
>>  rename test/corpus/{ => default}/cur/08:2, (100%)
>>  rename test/corpus/{ => default}/cur/09:2, (100%)
>>  rename test/corpus/{ => default}/cur/10:2, (100%)
>>  rename test/corpus/{ => default}/cur/11:2, (100%)
>>  rename test/corpus/{ => default}/cur/12:2, (100%)
>>  rename test/corpus/{ => default}/cur/13:2, (100%)
>>  rename test/corpus/{ => default}/cur/14:2, (100%)
>>  rename test/corpus/{ => default}/cur/15:2, (100%)
>>  rename test/corpus/{ => default}/cur/16:2, (100%)
>>  rename test/corpus/{ => default}/cur/17:2, (100%)
>>  rename test/corpus/{ => default}/cur/18:2, (100%)
>>  rename test/corpus/{ => default}/cur/19:2, (100%)
>>  rename test/corpus/{ => default}/cur/20:2, (100%)
>>  rename test/corpus/{ => default}/cur/21:2, (100%)
>>  rename test/corpus/{ => default}/cur/22:2, (100%)
>>  rename test/corpus/{ => default}/cur/23:2, (100%)
>>  rename test/corpus/{ => default}/cur/24:2, (100%)
>>  rename test/corpus/{ => default}/cur/25:2, (100%)
>>  rename test/corpus/{ => default}/cur/26:2, (100%)
>>  rename test/corpus/{ => default}/cur/27:2, (100%)
>>  rename test/corpus/{ => default}/cur/28:2, (100%)
>>  rename test/corpus/{ => default}/cur/29:2, (100%)
>>  rename test/corpus/{ => default}/cur/30:2, (100%)
>>  rename test/corpus/{ => default}/cur/31:2, (100%)
>>  rename test/corpus/{ => default}/cur/32:2, (100%)
>>  rename test/corpus/{ => default}/cur/33:2, (100%)
>>  rename test/corpus/{ => default}/cur/34:2, (100%)
>>  rename test/corpus/{ => default}/cur/35:2, (100%)
>>  rename test/corpus/{ => default}/cur/36:2, (100%)
>>  rename test/corpus/{ => default}/cur/37:2, (100%)
>>  rename test/corpus/{ => default}/cur/38:2, (100%)
>>  rename test/corpus/{ => default}/cur/39:2, (100%)
>>  rename test/corpus/{ => default}/cur/40:2, (100%)
>>  rename test/corpus/{ => default}/cur/41:2, (100%)
>>  rename test/corpus/{ => default}/cur/42:2, (100%)
>>  rename test/corpus/{ => default}/cur/43:2, (100%)
>>  rename test/corpus/{ => default}/cur/44:2, (100%)
>>  rename test/corpus/{ => default}/cur/45:2, (100%)
>>  rename test/corpus/{ => default}/cur/46:2, (100%)
>>  rename test/corpus/{ => default}/cur/47:2, (100%)
>>  rename test/corpus/{ => default}/cur/48:2, (100%)
>>  rename test/corpus/{ => default}/cur/49:2, (100%)
>>  rename test/corpus/{ => default}/cur/50:2, (100%)
>>  rename test/corpus/{ => default}/cur/51:2, (100%)
>>  rename test/corpus/{ => default}/cur/52:2, (100%)
>>  rename test/corpus/{ => default}/cur/53:2, (100%)
>>  create mode 100644 test/corpus/folders/01:2,
>>  create mode 100644 test/corpus/folders/02:2,
>>  create mode 100644 test/corpus/folders/bar/17:2,
>>  create mode 100644 test/corpus/folders/bar/18:2,
>>  create mode 100644 test/corpus/folders/bar/baz/05:2,
>>  create mode 100644 test/corpus/folders/bar/baz/23:2,
>>  create mode 100644 test/corpus/folders/bar/baz/24:2,
>>  create mode 100644 test/corpus/folders/bar/baz/cur/25:2,
>>  create mode 100644 test/corpus/folders/bar/baz/cur/26:2,
>>  create mode 100644 test/corpus/folders/bar/baz/new/27:2,
>>  create mode 100644 test/corpus/folders/bar/baz/new/28:2,
>>  create mode 100644 test/corpus/folders/bar/cur/19:2,
>>  create mode 100644 test/corpus/folders/bar/cur/20:2,
>>  create mode 100644 test/corpus/folders/bar/new/21:2,
>>  create mode 100644 test/corpus/folders/bar/new/22:2,
>>  create mode 100644 test/corpus/folders/cur/29:2,
>>  create mode 100644 test/corpus/folders/cur/30:2,
>>  create mode 100644 test/corpus/folders/cur/31:2,
>>  create mode 100644 test/corpus/folders/cur/32:2,
>>  create mode 100644 test/corpus/folders/cur/33:2,
>>  create mode 100644 test/corpus/folders/cur/34:2,
>>  create mode 100644 test/corpus/folders/cur/35:2,
>>  create mode 100644 test/corpus/folders/cur/36:2,
>>  create mode 100644 test/corpus/folders/cur/37:2,
>>  create mode 100644 test/corpus/folders/cur/38:2,
>>  create mode 100644 test/corpus/folders/cur/39:2,
>>  create mode 100644 test/corpus/folders/cur/40:2,
>>  create mode 100644 test/corpus/folders/cur/41:2,
>>  create mode 100644 test/corpus/folders/cur/42:2,
>>  create mode 100644 test/corpus/folders/cur/43:2,
>>  create mode 100644 test/corpus/folders/cur/44:2,
>>  create mode 100644 test/corpus/folders/cur/45:2,
>>  create mode 100644 test/corpus/folders/cur/46:2,
>>  create mode 100644 test/corpus/folders/cur/47:2,
>>  create mode 100644 test/corpus/folders/cur/48:2,
>>  create mode 100644 test/corpus/folders/cur/49:2,
>>  create mode 100644 test/corpus/folders/cur/50:2,
>>  create mode 100644 test/corpus/folders/cur/52:2,
>>  create mode 100644 test/corpus/folders/cur/53:2,
>>  create mode 100644 test/corpus/folders/foo/05:2,
>>  create mode 100644 test/corpus/folders/foo/06:2,
>>  create mode 100644 test/corpus/folders/foo/baz/11:2,
>>  create mode 100644 test/corpus/folders/foo/baz/12:2,
>>  create mode 100644 test/corpus/folders/foo/baz/cur/13:2,
>>  create mode 100644 test/corpus/folders/foo/baz/cur/14:2,
>>  create mode 100644 test/corpus/folders/foo/baz/new/15:2,
>>  create mode 100644 test/corpus/folders/foo/baz/new/16:2,
>>  create mode 100644 test/corpus/folders/foo/cur/07:2,
>>  create mode 100644 test/corpus/folders/foo/cur/08:2,
>>  create mode 100644 test/corpus/folders/foo/new/03:2,
>>  create mode 100644 test/corpus/folders/foo/new/09:2,
>>  create mode 100644 test/corpus/folders/foo/new/10:2,
>>  create mode 100644 test/corpus/folders/new/03:2,
>>  create mode 100644 test/corpus/folders/new/04:2,
>>  create mode 100644 test/test-databases/README
>>  create mode 100644 test/test-databases/database-v1.tar.gz
>>
>> -- 
>> 1.8.5.3
>>
>> _______________________________________________
>> notmuch mailing list
>> notmuch@notmuchmail.org
>> http://notmuchmail.org/mailman/listinfo/notmuch
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH v2 05/13] test: fix test for literal folder: search
  2014-02-22 22:25 ` [PATCH v2 05/13] test: fix test for literal folder: search Jani Nikula
@ 2014-02-23 14:02   ` David Bremner
  0 siblings, 0 replies; 25+ messages in thread
From: David Bremner @ 2014-02-23 14:02 UTC (permalink / raw)
  To: Jani Nikula, notmuch

Jani Nikula <jani@nikula.org> writes:

> Some of the folder: matching capabilities are lost in the
> probabilistic to boolean prefix change. Fix them.

the first 5 look copacetic.

d

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix
  2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
                   ` (13 preceding siblings ...)
  2014-02-22 23:57 ` [PATCH v2 00/13] literal folder: prefix, new path: prefix Mark Walters
@ 2014-03-02 19:11 ` David Bremner
  2014-03-04 19:37   ` Jani Nikula
  14 siblings, 1 reply; 25+ messages in thread
From: David Bremner @ 2014-03-02 19:11 UTC (permalink / raw)
  To: Jani Nikula, notmuch

Jani Nikula <jani@nikula.org> writes:
>
> I've dropped most of the content in patches 7 and 10 due to their
> size. The patches (and the whole series) are available in the
> boolean-folder-and-path-v2 branch at
> git://gitorious.org/jani/notmuch.git. Web interface at
> https://gitorious.org/jani/notmuch/commits/0b3dd2d1cc6c413ea07ea326883ac448499c0e79.
>

Hi Jani;

I put a some proposed changes at 

  http://pivot.cs.unb.ca/git?p=notmuch.git;a=shortlog;h=refs/heads/boolean-folder-and-path-v2

After playing with it a bit, I decided to 

      - leave the source for the corpuses in the main tree. As we
        discussed, we don't expect them to change much, and they are
        text files.

      - store the binary database blobs externally. We could make the
        same argument about infrequent changes here, except that even
        the tiniest change will in practice require a whole new copy be
        stored.

        - compute a checksum file
        - test: add machinery to fetch and publish test databases
        - test: commit database-v1.tar.xz checksum, ignore actual database

       - this is an independant bug fix that should probably be pushed anyway:

         - test: don't use $(dir) in recipes.

      - I also made some unrelated changes including switching to xz
        compression. I intentionally didn't squash my patches into yours
        so you see and decide

        - convert to XZ compression
        - convert test to XZ

        Originally I thought metadata in the database tarball is
        needed. It turns out not. I think it's harmless, but feel free
        to drop it.

        - add some metadata to the database tarball
        
      - I struggled to find a way that would allow people to manage the
        tarballs in a nice way while not requiring everyone to
        install/learn new tools. My conclusion is in
      
        - test: add support for git annex managing test databases

We still need to think about what to do for people installing from
tarballs. As it stands they will need to either have network access or
skip the relevant tests. Probably at least one followup patch is needed
here. The alternative would be to include the database tarballs in the
source; this will pretty quickly double the size of the source tarball.

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix
  2014-03-02 19:11 ` David Bremner
@ 2014-03-04 19:37   ` Jani Nikula
  2014-03-04 21:15     ` David Bremner
  0 siblings, 1 reply; 25+ messages in thread
From: Jani Nikula @ 2014-03-04 19:37 UTC (permalink / raw)
  To: David Bremner, notmuch

On Sun, 02 Mar 2014, David Bremner <david@tethera.net> wrote:
> Jani Nikula <jani@nikula.org> writes:
>>
>> I've dropped most of the content in patches 7 and 10 due to their
>> size. The patches (and the whole series) are available in the
>> boolean-folder-and-path-v2 branch at
>> git://gitorious.org/jani/notmuch.git. Web interface at
>> https://gitorious.org/jani/notmuch/commits/0b3dd2d1cc6c413ea07ea326883ac448499c0e79.
>>
>
> Hi Jani;
>
> I put a some proposed changes at 
>
>   http://pivot.cs.unb.ca/git?p=notmuch.git;a=shortlog;h=refs/heads/boolean-folder-and-path-v2
>
> After playing with it a bit, I decided to 
>
>       - leave the source for the corpuses in the main tree. As we
>         discussed, we don't expect them to change much, and they are
>         text files.
>
>       - store the binary database blobs externally. We could make the
>         same argument about infrequent changes here, except that even
>         the tiniest change will in practice require a whole new copy be
>         stored.
>
>         - compute a checksum file
>         - test: add machinery to fetch and publish test databases
>         - test: commit database-v1.tar.xz checksum, ignore actual database
>
>        - this is an independant bug fix that should probably be pushed anyway:
>
>          - test: don't use $(dir) in recipes.
>
>       - I also made some unrelated changes including switching to xz
>         compression. I intentionally didn't squash my patches into yours
>         so you see and decide
>
>         - convert to XZ compression
>         - convert test to XZ
>
>         Originally I thought metadata in the database tarball is
>         needed. It turns out not. I think it's harmless, but feel free
>         to drop it.
>
>         - add some metadata to the database tarball

I'm fine with all of the above.

>       - I struggled to find a way that would allow people to manage the
>         tarballs in a nice way while not requiring everyone to
>         install/learn new tools. My conclusion is in
>       
>         - test: add support for git annex managing test databases
>
> We still need to think about what to do for people installing from
> tarballs. As it stands they will need to either have network access or
> skip the relevant tests. Probably at least one followup patch is needed
> here. The alternative would be to include the database tarballs in the
> source; this will pretty quickly double the size of the source tarball.

I think we should use the prereq mechanism in the tests to check if a
required test database is in place, skipping the test if not. This
should be fairly straightforward to do, and does not care *how* the test
database is to be fetched. (I'm undecided whether the test should verify
the checksum, or the part that fetches the database. Minor detail.)

I know git annex in principle, but don't have any practical experience
with it. I don't really have any ideas for that.


BR,
Jani.

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix
  2014-03-04 19:37   ` Jani Nikula
@ 2014-03-04 21:15     ` David Bremner
  2014-03-05  8:39       ` Tomi Ollila
  0 siblings, 1 reply; 25+ messages in thread
From: David Bremner @ 2014-03-04 21:15 UTC (permalink / raw)
  To: Jani Nikula, notmuch

Jani Nikula <jani@nikula.org> writes:

>
> I think we should use the prereq mechanism in the tests to check if a
> required test database is in place, skipping the test if not. This
> should be fairly straightforward to do, and does not care *how* the test
> database is to be fetched. (I'm undecided whether the test should verify
> the checksum, or the part that fetches the database. Minor detail.)

Yep, prereqs sounds fine.  SHA256 verification is pretty fast (on the
scale of test suite operations), so I'd lean to doing it late, but
I could be convinced otherwise.

>
> I know git annex in principle, but don't have any practical experience
> with it. I don't really have any ideas for that.

Well, if you don't mind the rest of the setup (SHA256 sums checked,
tarballs fetched if needed), we can forge ahead for now.

d

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix
  2014-03-04 21:15     ` David Bremner
@ 2014-03-05  8:39       ` Tomi Ollila
  2014-03-05 11:48         ` David Bremner
  0 siblings, 1 reply; 25+ messages in thread
From: Tomi Ollila @ 2014-03-05  8:39 UTC (permalink / raw)
  To: David Bremner, Jani Nikula, notmuch

On Tue, Mar 04 2014, David Bremner <david@tethera.net> wrote:

> Jani Nikula <jani@nikula.org> writes:
>
>>
>> I think we should use the prereq mechanism in the tests to check if a
>> required test database is in place, skipping the test if not. This
>> should be fairly straightforward to do, and does not care *how* the test
>> database is to be fetched. (I'm undecided whether the test should verify
>> the checksum, or the part that fetches the database. Minor detail.)
>
> Yep, prereqs sounds fine.  SHA256 verification is pretty fast (on the
> scale of test suite operations), so I'd lean to doing it late, but
> I could be convinced otherwise.

I decline commenting on prereqs... ;/

>>
>> I know git annex in principle, but don't have any practical experience
>> with it. I don't really have any ideas for that.
>
> Well, if you don't mind the rest of the setup (SHA256 sums checked,
> tarballs fetched if needed), we can forge ahead for now.

I can contribute script with interface 

./fetch-and-check.sh [--verify-only] destdir url sha256sum 


>
> d


Tomi

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix
  2014-03-05  8:39       ` Tomi Ollila
@ 2014-03-05 11:48         ` David Bremner
  2014-03-05 13:10           ` Tomi Ollila
  0 siblings, 1 reply; 25+ messages in thread
From: David Bremner @ 2014-03-05 11:48 UTC (permalink / raw)
  To: Tomi Ollila, Jani Nikula, notmuch

Tomi Ollila <tomi.ollila@iki.fi> writes:

>
> I can contribute script with interface 
>
> ./fetch-and-check.sh [--verify-only] destdir url sha256sum 
>

This script is basically embedded in 


http://pivot.cs.unb.ca/git?p=notmuch.git;a=commitdiff;h=f93e32b11e517e160c755355de55ab465206dc62

except that I rebased away  the verification in the actual test :(

Something like 

diff --git a/test/T530-upgrade.sh b/test/T530-upgrade.sh
index 3c0134f..768242d 100755
--- a/test/T530-upgrade.sh
+++ b/test/T530-upgrade.sh
@@ -5,6 +5,10 @@ test_description="database upgrade"
 
 dbtarball=database-v1.tar.xz
 
+cd $TEST_DIRECTORY/test-databases
+sha256sum --quiet --check ${dbtarball}.sha256 || error "checksum of database failed"
+cd - >& /dev/null
+
 tar Jxf $TEST_DIRECTORY/test-databases/${dbtarball} -C ${MAIL_DIR} --strip-component
 
 test_begin_subtest "folder: search does not work with old database version"

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix
  2014-03-05 11:48         ` David Bremner
@ 2014-03-05 13:10           ` Tomi Ollila
  2014-03-05 14:41             ` David Bremner
  0 siblings, 1 reply; 25+ messages in thread
From: Tomi Ollila @ 2014-03-05 13:10 UTC (permalink / raw)
  To: David Bremner, Jani Nikula, notmuch

On Wed, Mar 05 2014, David Bremner <david@tethera.net> wrote:

> Tomi Ollila <tomi.ollila@iki.fi> writes:
>
>>
>> I can contribute script with interface 
>>
>> ./fetch-and-check.sh [--verify-only] destdir url sha256sum 
>>
>
> This script is basically embedded in 
>
>
> http://pivot.cs.unb.ca/git?p=notmuch.git;a=commitdiff;h=f93e32b11e517e160c755355de55ab465206dc62
>
> except that I rebased away  the verification in the actual test :(
>
> Something like 
>
> diff --git a/test/T530-upgrade.sh b/test/T530-upgrade.sh
> index 3c0134f..768242d 100755
> --- a/test/T530-upgrade.sh
> +++ b/test/T530-upgrade.sh
> @@ -5,6 +5,10 @@ test_description="database upgrade"
>  
>  dbtarball=database-v1.tar.xz
>  
> +cd $TEST_DIRECTORY/test-databases
> +sha256sum --quiet --check ${dbtarball}.sha256 || error "checksum of database failed"
> +cd - >& /dev/null

more portable alternative:

read sha256sum_exp < ${dbtarball}.sha256
#sha256sum_act=`openssl sha256 "${dbtarball}" | sed 's/ .*//'`
sha256sum_act=`openssl sha256 "${dbtarball}"`
test "$sha256sum_exp" = "$sha256sum_act" || error "checksum of database failed"

(IIRC openssl sha256 is available at least in Mac OS X, probably in other
BSD:s & Solaris too)


> +
>  tar Jxf $TEST_DIRECTORY/test-databases/${dbtarball} -C ${MAIL_DIR} --strip-component
>  
>  test_begin_subtest "folder: search does not work with old database version"

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix
  2014-03-05 13:10           ` Tomi Ollila
@ 2014-03-05 14:41             ` David Bremner
  2014-03-05 15:40               ` Tomi Ollila
  0 siblings, 1 reply; 25+ messages in thread
From: David Bremner @ 2014-03-05 14:41 UTC (permalink / raw)
  To: Tomi Ollila, Jani Nikula, notmuch

Tomi Ollila <tomi.ollila@iki.fi> writes:
>
> (IIRC openssl sha256 is available at least in Mac OS X, probably in other
> BSD:s & Solaris too)
>

OTOH sha256sum is in coreutils, so everywhere with a gnu userland has
it. Some might not have openssl (it's priority optional on Debian,
e.g.).

d

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH v2 00/13] literal folder: prefix, new path: prefix
  2014-03-05 14:41             ` David Bremner
@ 2014-03-05 15:40               ` Tomi Ollila
  0 siblings, 0 replies; 25+ messages in thread
From: Tomi Ollila @ 2014-03-05 15:40 UTC (permalink / raw)
  To: David Bremner, Jani Nikula, notmuch

On Wed, Mar 05 2014, David Bremner <david@tethera.net> wrote:

> Tomi Ollila <tomi.ollila@iki.fi> writes:
>>
>> (IIRC openssl sha256 is available at least in Mac OS X, probably in other
>> BSD:s & Solaris too)
>>
>
> OTOH sha256sum is in coreutils, so everywhere with a gnu userland has
> it. Some might not have openssl (it's priority optional on Debian,
> e.g.).

Aaah these response times are a pleasure... :D

cd $TEST_DIRECTORY/test-databases

if hash sha256sum 2>/dev/null
then
     sha256sum --quiet --check ${dbtarball}.sha256 || error "checksum of database failed"
else
     # opportunistically try to run openssl sha256 in case sha256sum is not available
     read sha256sum_exp filename < ${dbtarball}.sha256
     sha256sum_act=`openssl sha256 "${dbtarball}" | sed 's/ .*//'`
     test "$sha256sum_exp" = "$sha256sum_act" || error "checksum of database failed"
fi
cd - >& /dev/null


>
> d

Tomi

^ permalink raw reply	[flat|nested] 25+ messages in thread

end of thread, other threads:[~2014-03-05 15:40 UTC | newest]

Thread overview: 25+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-02-22 22:25 [PATCH v2 00/13] literal folder: prefix, new path: prefix Jani Nikula
2014-02-22 22:25 ` [PATCH v2 01/13] lib: refactor folder term update after filename removal Jani Nikula
2014-02-22 22:25 ` [PATCH v2 02/13] lib: add support for path: prefix searches Jani Nikula
2014-02-22 22:25 ` [PATCH v2 03/13] test: make insert test use the path: prefix Jani Nikula
2014-02-22 22:25 ` [PATCH v2 04/13] lib: make folder: prefix literal Jani Nikula
2014-02-22 22:25 ` [PATCH v2 05/13] test: fix test for literal folder: search Jani Nikula
2014-02-23 14:02   ` David Bremner
2014-02-22 22:25 ` [PATCH v2 06/13] test: make it possible to have several corpora Jani Nikula
2014-02-22 22:25 ` [PATCH v2 07/13] test: add new corpus with folders Jani Nikula
2014-02-22 22:25 ` [PATCH v2 08/13] test: add tests for the new boolean folder: and path: prefixes Jani Nikula
2014-02-22 22:25 ` [PATCH v2 09/13] devel: add script to generate test databases Jani Nikula
2014-02-22 22:25 ` [PATCH v2 10/13] test: add test database in format version 1 Jani Nikula
2014-02-22 22:25 ` [PATCH v2 11/13] test: add database upgrade test from format version 1 to 2 Jani Nikula
2014-02-22 22:25 ` [PATCH v2 12/13] man: update man pages for folder: and path: search terms Jani Nikula
2014-02-22 22:25 ` [PATCH v2 13/13] man: try to clarify the folder: and path: vs. --output=files confusion Jani Nikula
2014-02-22 23:57 ` [PATCH v2 00/13] literal folder: prefix, new path: prefix Mark Walters
2014-02-23 11:15   ` Tomi Ollila
2014-03-02 19:11 ` David Bremner
2014-03-04 19:37   ` Jani Nikula
2014-03-04 21:15     ` David Bremner
2014-03-05  8:39       ` Tomi Ollila
2014-03-05 11:48         ` David Bremner
2014-03-05 13:10           ` Tomi Ollila
2014-03-05 14:41             ` David Bremner
2014-03-05 15:40               ` Tomi Ollila

Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).