[PATCH 1/1] Stores the folder (directory name) of the message in the database as a term with folder prefix.
This patch was originally sent by Andreas Kl?ckner in: id:200912141421.52561.lists at informa.tiker.net. It was then subsequently updated by Michal Sojka in: id:1264692317-9175-2-git-send-email-sojkam1 at fel.cvut.cz and then further improved again by Michal Sojka in: id:1265122868-12133-1-git-send-email-sojkam1 at fel.cvut.cz This is a rebase of the latest patch off of the current git HEAD. . The differences from the original patch are: - Rebased off of current git HEAD - Folder name is taken from strings generated during travesal. It no longer uses glib nor it allocates additional memory to determine the base name. The same approach as in id:87fx8bygi7.fsf at linux.vnet.ibm.com was used. - Removed unrelated change which was submitted separately as id:1264691584-8290-2-git-send-email-sojkam1 at fel.cvut.cz - Changed the comment describing database schema. TODO (see Carl's email: id:87zl5k0w6s.fsf at yoom.home.cworth.org): - Support hierarchical folders: this patch is only storing the final directory component. This should be hooked in differently with filename storage so the whole filename is indexed as text to provide arbitrary search phrases such as "folder:'foo/bar/baz'" --- lib/database.cc | 13 + lib/notmuch.h |1 + notmuch-new.c | 39 +-- 3 files changed, 47 insertions(+), 6 deletions(-) diff --git a/lib/database.cc b/lib/database.cc index c91e97c..6364623 100644 --- a/lib/database.cc +++ b/lib/database.cc @@ -84,9 +84,9 @@ typedef struct { * MESSAGE_ID: The unique ID of the mail mess (see "id" above) * * 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. But the database doesn't really care itself - * about any of these. + * "from", "to", "attachment", "subject" and "folder" prefixes for use + * by the user in searching. But the database doesn't really care + * itself about any of these. * * The data portion of a mail document is empty. * @@ -155,7 +155,8 @@ prefix_t PROBABILISTIC_PREFIX[]= { { "from", "XFROM" }, { "to","XTO" }, { "attachment","XATTACHMENT" }, -{ "subject", "XSUBJECT"} +{ "subject", "XSUBJECT"}, +{ "folder","XFOLDER"} }; int @@ -1362,6 +1363,7 @@ _notmuch_database_link_message (notmuch_database_t *notmuch, notmuch_status_t notmuch_database_add_message (notmuch_database_t *notmuch, const char *filename, + const char *folder_name, notmuch_message_t **message_ret) { notmuch_message_file_t *message_file; @@ -1477,6 +1479,9 @@ notmuch_database_add_message (notmuch_database_t *notmuch, date = notmuch_message_file_get_header (message_file, "date"); _notmuch_message_set_date (message, date); + if (folder_name != NULL) + _notmuch_message_gen_terms (message, "folder", folder_name); + _notmuch_message_index_file (message, filename); } else { ret = NOTMUCH_STATUS_DUPLICATE_MESSAGE_ID; diff --git a/lib/notmuch.h b/lib/notmuch.h index 88da078..ce81565 100644 --- a/lib/notmuch.h +++ b/lib/notmuch.h @@ -264,6 +264,7 @@ notmuch_database_get_directory (notmuch_database_t *database, notmuch_status_t notmuch_database_add_message (notmuch_database_t *database, const char *filename, + const char *folder_name, notmuch_message_t **message); /* Remove a message from the given notmuch database. diff --git a/notmuch-new.c b/notmuch-new.c index 44b50aa..6ad3c09 100644 --- a/notmuch-new.c +++ b/notmuch-new.c @@ -21,6 +21,7 @@ #include "notmuch-client.h" #include +#include typedef struct _filename_node { char *filename; @@ -169,6 +170,35 @@ _entries_resemble_maildir (struct dirent **entries, int count) return 0; } +static char* +_get_folder_base_name(const char *path) +{ + gchar *full_folder_name = NULL; + gchar *folder_base_name = NULL; + + /* Find name of "folder" containing the email. */ + full_folder_name = g_strdup(path); + while (1) { +folder_base_name = g_path_get_basename(full_folder_name); + +if (strcmp(folder_base_name, "cur") == 0 + || strcmp(folder_base_name, "new") == 0) { + gchar *parent_name = g_path_get_dirname(full_folder_name); + g_free(full_folder_name); + full_folder_name = parent_name; +} else + break; + } + + g_free(full_folder_name); + + if (strcmp(folder_base_name, ".") == 0) { +g_free(folder_base_name); +folder_base_name = NULL; + } + return folder_base_name; +} + /* Examine 'path' recursively as follows: * * o Ask the filesystem for the mtime of 'path' (fs_mtime) @@
[PATCH 1/1] Stores the folder (directory name) of the message in the database as a term with folder prefix.
This patch was originally sent by Andreas Klöckner in: id:200912141421.52561.li...@informa.tiker.net. It was then subsequently updated by Michal Sojka in: id:1264692317-9175-2-git-send-email-sojk...@fel.cvut.cz and then further improved again by Michal Sojka in: id:1265122868-12133-1-git-send-email-sojk...@fel.cvut.cz This is a rebase of the latest patch off of the current git HEAD. . The differences from the original patch are: - Rebased off of current git HEAD - Folder name is taken from strings generated during travesal. It no longer uses glib nor it allocates additional memory to determine the base name. The same approach as in id:87fx8bygi7@linux.vnet.ibm.com was used. - Removed unrelated change which was submitted separately as id:1264691584-8290-2-git-send-email-sojk...@fel.cvut.cz - Changed the comment describing database schema. TODO (see Carl's email: id:87zl5k0w6s@yoom.home.cworth.org): - Support hierarchical folders: this patch is only storing the final directory component. This should be hooked in differently with filename storage so the whole filename is indexed as text to provide arbitrary search phrases such as folder:'foo/bar/baz' --- lib/database.cc | 13 + lib/notmuch.h |1 + notmuch-new.c | 39 +-- 3 files changed, 47 insertions(+), 6 deletions(-) diff --git a/lib/database.cc b/lib/database.cc index c91e97c..6364623 100644 --- a/lib/database.cc +++ b/lib/database.cc @@ -84,9 +84,9 @@ typedef struct { * MESSAGE_ID: The unique ID of the mail mess (see id above) * * 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. But the database doesn't really care itself - * about any of these. + * from, to, attachment, subject and folder prefixes for use + * by the user in searching. But the database doesn't really care + * itself about any of these. * * The data portion of a mail document is empty. * @@ -155,7 +155,8 @@ prefix_t PROBABILISTIC_PREFIX[]= { { from, XFROM }, { to,XTO }, { attachment,XATTACHMENT }, -{ subject, XSUBJECT} +{ subject, XSUBJECT}, +{ folder,XFOLDER} }; int @@ -1362,6 +1363,7 @@ _notmuch_database_link_message (notmuch_database_t *notmuch, notmuch_status_t notmuch_database_add_message (notmuch_database_t *notmuch, const char *filename, + const char *folder_name, notmuch_message_t **message_ret) { notmuch_message_file_t *message_file; @@ -1477,6 +1479,9 @@ notmuch_database_add_message (notmuch_database_t *notmuch, date = notmuch_message_file_get_header (message_file, date); _notmuch_message_set_date (message, date); + if (folder_name != NULL) + _notmuch_message_gen_terms (message, folder, folder_name); + _notmuch_message_index_file (message, filename); } else { ret = NOTMUCH_STATUS_DUPLICATE_MESSAGE_ID; diff --git a/lib/notmuch.h b/lib/notmuch.h index 88da078..ce81565 100644 --- a/lib/notmuch.h +++ b/lib/notmuch.h @@ -264,6 +264,7 @@ notmuch_database_get_directory (notmuch_database_t *database, notmuch_status_t notmuch_database_add_message (notmuch_database_t *database, const char *filename, + const char *folder_name, notmuch_message_t **message); /* Remove a message from the given notmuch database. diff --git a/notmuch-new.c b/notmuch-new.c index 44b50aa..6ad3c09 100644 --- a/notmuch-new.c +++ b/notmuch-new.c @@ -21,6 +21,7 @@ #include notmuch-client.h #include unistd.h +#include glib.h typedef struct _filename_node { char *filename; @@ -169,6 +170,35 @@ _entries_resemble_maildir (struct dirent **entries, int count) return 0; } +static char* +_get_folder_base_name(const char *path) +{ + gchar *full_folder_name = NULL; + gchar *folder_base_name = NULL; + + /* Find name of folder containing the email. */ + full_folder_name = g_strdup(path); + while (1) { +folder_base_name = g_path_get_basename(full_folder_name); + +if (strcmp(folder_base_name, cur) == 0 + || strcmp(folder_base_name, new) == 0) { + gchar *parent_name = g_path_get_dirname(full_folder_name); + g_free(full_folder_name); + full_folder_name = parent_name; +} else + break; + } + + g_free(full_folder_name); + + if (strcmp(folder_base_name, .) == 0) { +g_free(folder_base_name); +folder_base_name = NULL; + } + return folder_base_name; +} + /* Examine 'path' recursively as follows: * * o Ask the filesystem for the mtime of 'path' (fs_mtime) @@ -222,6 +252,7 @@ add_files_recursive (notmuch_database_t