[Patch v6 4/8] lib: add "verbose" versions of notmuch_database_{open,create}

diff --git a/24/b99a25d7cadef251eec094926cf4bd5e0b5add b/24/b99a25d7cadef251eec094926cf4bd5e0b5add
new file mode 100644 (file)
index 0000000..d38995a
--- /dev/null
+++ b/24/b99a25d7cadef251eec094926cf4bd5e0b5add
@@ -0,0 +1,418 @@
+Return-Path: <bremner@tesseract.cs.unb.ca>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id 2BA62431FDA\r
+       for <notmuch@notmuchmail.org>; Fri, 27 Mar 2015 15:13:22 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: 2.438\r
+X-Spam-Level: **\r
+X-Spam-Status: No, score=2.438 tagged_above=-999 required=5\r
+       tests=[DNS_FROM_AHBL_RHSBL=2.438] autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+       by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+       with ESMTP id 3lF5hEF9fxYZ for <notmuch@notmuchmail.org>;\r
+       Fri, 27 Mar 2015 15:13:20 -0700 (PDT)\r
+Received: from mx.xen14.node3324.gplhost.com (gitolite.debian.net\r
+       [87.98.215.224])\r
+       (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
+       (No client certificate requested)\r
+       by olra.theworths.org (Postfix) with ESMTPS id F2561431FC9\r
+       for <notmuch@notmuchmail.org>; Fri, 27 Mar 2015 15:13:19 -0700 (PDT)\r
+Received: from remotemail by mx.xen14.node3324.gplhost.com with local (Exim\r
+       4.80) (envelope-from <bremner@tesseract.cs.unb.ca>)\r
+       id 1YbcUV-0000EQ-ME; Fri, 27 Mar 2015 22:12:35 +0000\r
+Received: (nullmailer pid 1813 invoked by uid 1000); Fri, 27 Mar 2015\r
+       22:12:15 -0000\r
+From: David Bremner <david@tethera.net>\r
+To: notmuch@notmuchmail.org\r
+Subject: [Patch v6 4/8] lib: add "verbose" versions of\r
+       notmuch_database_{open,create}\r
+Date: Fri, 27 Mar 2015 23:11:56 +0100\r
+Message-Id: <1427494320-1483-5-git-send-email-david@tethera.net>\r
+X-Mailer: git-send-email 2.1.4\r
+In-Reply-To: <1427494320-1483-1-git-send-email-david@tethera.net>\r
+References: <1427494320-1483-1-git-send-email-david@tethera.net>\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+       <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Fri, 27 Mar 2015 22:13:22 -0000\r
+\r
+The compatibility wrapper ensures that clients calling\r
+notmuch_database_open will receive consistent output for now.\r
+\r
+The changes to notmuch-{new,search} and test/symbol-test are just to\r
+make the test suite pass.\r
+\r
+The use of IGNORE_RESULT is justified by two things. 1) I don't know\r
+what else to do.  2) asprintf guarantees the output string is NULL if\r
+an error occurs, so at least we are not passing garbage back.\r
+---\r
+ lib/database.cc     | 109 ++++++++++++++++++++++++++++++++++++++--------------\r
+ lib/notmuch.h       |  21 ++++++++++\r
+ notmuch-new.c       |  11 +++++-\r
+ notmuch-search.c    |  13 ++++++-\r
+ test/symbol-test.cc |   9 ++++-\r
+ 5 files changed, 130 insertions(+), 33 deletions(-)\r
+\r
+diff --git a/lib/database.cc b/lib/database.cc\r
+index 3974e2e..5d973b9 100644\r
+--- a/lib/database.cc\r
++++ b/lib/database.cc\r
+@@ -608,29 +608,50 @@ parse_references (void *ctx,\r
+ notmuch_status_t\r
+ notmuch_database_create (const char *path, notmuch_database_t **database)\r
+ {\r
++    char *status_string = NULL;\r
++    notmuch_status_t status;\r
++\r
++    status = notmuch_database_create_verbose (path, database,\r
++                                            &status_string);\r
++\r
++    if (status_string) {\r
++      fputs (status_string, stderr);\r
++      free (status_string);\r
++    }\r
++\r
++    return status;\r
++}\r
++\r
++notmuch_status_t\r
++notmuch_database_create_verbose (const char *path,\r
++                               notmuch_database_t **database,\r
++                               char **status_string)\r
++{\r
+     notmuch_status_t status = NOTMUCH_STATUS_SUCCESS;\r
+     notmuch_database_t *notmuch = NULL;\r
+     char *notmuch_path = NULL;\r
++    char *message = NULL;\r
+     struct stat st;\r
+     int err;\r
+ \r
+     if (path == NULL) {\r
+-      fprintf (stderr, "Error: Cannot create a database for a NULL path.\n");\r
++      message = strdup ("Error: Cannot create a database for a NULL path.\n");\r
+       status = NOTMUCH_STATUS_NULL_POINTER;\r
+       goto DONE;\r
+     }\r
+ \r
+     err = stat (path, &st);\r
+     if (err) {\r
+-      fprintf (stderr, "Error: Cannot create database at %s: %s.\n",\r
+-               path, strerror (errno));\r
++      IGNORE_RESULT (asprintf (&message, "Error: Cannot create database at %s: %s.\n",\r
++                              path, strerror (errno)));\r
+       status = NOTMUCH_STATUS_FILE_ERROR;\r
+       goto DONE;\r
+     }\r
+ \r
+     if (! S_ISDIR (st.st_mode)) {\r
+-      fprintf (stderr, "Error: Cannot create database at %s: Not a directory.\n",\r
+-               path);\r
++      IGNORE_RESULT (asprintf (&message, "Error: Cannot create database at %s: "\r
++                               "Not a directory.\n",\r
++                               path));\r
+       status = NOTMUCH_STATUS_FILE_ERROR;\r
+       goto DONE;\r
+     }\r
+@@ -640,15 +661,15 @@ notmuch_database_create (const char *path, notmuch_database_t **database)\r
+     err = mkdir (notmuch_path, 0755);\r
+ \r
+     if (err) {\r
+-      fprintf (stderr, "Error: Cannot create directory %s: %s.\n",\r
+-               notmuch_path, strerror (errno));\r
++      IGNORE_RESULT (asprintf (&message, "Error: Cannot create directory %s: %s.\n",\r
++                               notmuch_path, strerror (errno)));\r
+       status = NOTMUCH_STATUS_FILE_ERROR;\r
+       goto DONE;\r
+     }\r
+ \r
+-    status = notmuch_database_open (path,\r
+-                                  NOTMUCH_DATABASE_MODE_READ_WRITE,\r
+-                                  &notmuch);\r
++    status = notmuch_database_open_verbose (path,\r
++                                          NOTMUCH_DATABASE_MODE_READ_WRITE,\r
++                                          &notmuch, &message);\r
+     if (status)\r
+       goto DONE;\r
+ \r
+@@ -667,6 +688,9 @@ notmuch_database_create (const char *path, notmuch_database_t **database)\r
+     if (notmuch_path)\r
+       talloc_free (notmuch_path);\r
+ \r
++    if (status_string && message)\r
++      *status_string = message;\r
++\r
+     if (database)\r
+       *database = notmuch;\r
+     else\r
+@@ -767,37 +791,58 @@ notmuch_database_open (const char *path,\r
+                      notmuch_database_mode_t mode,\r
+                      notmuch_database_t **database)\r
+ {\r
++    char *status_string = NULL;\r
++    notmuch_status_t status;\r
++\r
++    status = notmuch_database_open_verbose (path, mode, database,\r
++                                         &status_string);\r
++\r
++    if (status_string) {\r
++      fputs (status_string, stderr);\r
++      free (status_string);\r
++    }\r
++\r
++    return status;\r
++}\r
++\r
++notmuch_status_t\r
++notmuch_database_open_verbose (const char *path,\r
++                             notmuch_database_mode_t mode,\r
++                             notmuch_database_t **database,\r
++                             char **status_string)\r
++{\r
+     notmuch_status_t status = NOTMUCH_STATUS_SUCCESS;\r
+     void *local = talloc_new (NULL);\r
+     notmuch_database_t *notmuch = NULL;\r
+     char *notmuch_path, *xapian_path, *incompat_features;\r
++    char *message = NULL;\r
+     struct stat st;\r
+     int err;\r
+     unsigned int i, version;\r
+     static int initialized = 0;\r
+ \r
+     if (path == NULL) {\r
+-      fprintf (stderr, "Error: Cannot open a database for a NULL path.\n");\r
++      message = strdup ("Error: Cannot open a database for a NULL path.\n");\r
+       status = NOTMUCH_STATUS_NULL_POINTER;\r
+       goto DONE;\r
+     }\r
+ \r
+     if (! (notmuch_path = talloc_asprintf (local, "%s/%s", path, ".notmuch"))) {\r
+-      fprintf (stderr, "Out of memory\n");\r
++      message = strdup ("Out of memory\n");\r
+       status = NOTMUCH_STATUS_OUT_OF_MEMORY;\r
+       goto DONE;\r
+     }\r
+ \r
+     err = stat (notmuch_path, &st);\r
+     if (err) {\r
+-      fprintf (stderr, "Error opening database at %s: %s\n",\r
+-               notmuch_path, strerror (errno));\r
++      IGNORE_RESULT (asprintf (&message, "Error opening database at %s: %s\n",\r
++                               notmuch_path, strerror (errno)));\r
+       status = NOTMUCH_STATUS_FILE_ERROR;\r
+       goto DONE;\r
+     }\r
+ \r
+     if (! (xapian_path = talloc_asprintf (local, "%s/%s", notmuch_path, "xapian"))) {\r
+-      fprintf (stderr, "Out of memory\n");\r
++      message = strdup ("Out of memory\n");\r
+       status = NOTMUCH_STATUS_OUT_OF_MEMORY;\r
+       goto DONE;\r
+     }\r
+@@ -837,11 +882,11 @@ notmuch_database_open (const char *path,\r
+        * means a dramatically incompatible change. */\r
+       version = notmuch_database_get_version (notmuch);\r
+       if (version > NOTMUCH_DATABASE_VERSION) {\r
+-          fprintf (stderr,\r
+-                   "Error: Notmuch database at %s\n"\r
+-                   "       has a newer database format version (%u) than supported by this\n"\r
+-                   "       version of notmuch (%u).\n",\r
+-                   notmuch_path, version, NOTMUCH_DATABASE_VERSION);\r
++          IGNORE_RESULT (asprintf (&message,\r
++                    "Error: Notmuch database at %s\n"\r
++                    "       has a newer database format version (%u) than supported by this\n"\r
++                    "       version of notmuch (%u).\n",\r
++                                   notmuch_path, version, NOTMUCH_DATABASE_VERSION));\r
+           notmuch->mode = NOTMUCH_DATABASE_MODE_READ_ONLY;\r
+           notmuch_database_destroy (notmuch);\r
+           notmuch = NULL;\r
+@@ -856,11 +901,11 @@ notmuch_database_open (const char *path,\r
+           version, mode == NOTMUCH_DATABASE_MODE_READ_WRITE ? 'w' : 'r',\r
+           &incompat_features);\r
+       if (incompat_features) {\r
+-          fprintf (stderr,\r
+-                   "Error: Notmuch database at %s\n"\r
+-                   "       requires features (%s)\n"\r
+-                   "       not supported by this version of notmuch.\n",\r
+-                   notmuch_path, incompat_features);\r
++          IGNORE_RESULT (asprintf (&message,\r
++              "Error: Notmuch database at %s\n"\r
++              "       requires features (%s)\n"\r
++              "       not supported by this version of notmuch.\n",\r
++                                   notmuch_path, incompat_features));\r
+           notmuch->mode = NOTMUCH_DATABASE_MODE_READ_ONLY;\r
+           notmuch_database_destroy (notmuch);\r
+           notmuch = NULL;\r
+@@ -906,8 +951,8 @@ notmuch_database_open (const char *path,\r
+           notmuch->query_parser->add_prefix (prefix->name, prefix->prefix);\r
+       }\r
+     } catch (const Xapian::Error &error) {\r
+-      fprintf (stderr, "A Xapian exception occurred opening database: %s\n",\r
+-               error.get_msg().c_str());\r
++      IGNORE_RESULT (asprintf (&message, "A Xapian exception occurred opening database: %s\n",\r
++                               error.get_msg().c_str()));\r
+       notmuch_database_destroy (notmuch);\r
+       notmuch = NULL;\r
+       status = NOTMUCH_STATUS_XAPIAN_EXCEPTION;\r
+@@ -916,6 +961,9 @@ notmuch_database_open (const char *path,\r
+   DONE:\r
+     talloc_free (local);\r
+ \r
++    if (status_string && message)\r
++      *status_string = message;\r
++\r
+     if (database)\r
+       *database = notmuch;\r
+     else\r
+@@ -1039,13 +1087,18 @@ notmuch_database_compact (const char *path,\r
+     notmuch_database_t *notmuch = NULL;\r
+     struct stat statbuf;\r
+     notmuch_bool_t keep_backup;\r
++    char *message = NULL;\r
+ \r
+     local = talloc_new (NULL);\r
+     if (! local)\r
+       return NOTMUCH_STATUS_OUT_OF_MEMORY;\r
+ \r
+-    ret = notmuch_database_open (path, NOTMUCH_DATABASE_MODE_READ_WRITE, &notmuch);\r
++    ret = notmuch_database_open_verbose (path,\r
++                                       NOTMUCH_DATABASE_MODE_READ_WRITE,\r
++                                       &notmuch,\r
++                                       &message);\r
+     if (ret) {\r
++      if (status_cb) status_cb (message, closure);\r
+       goto DONE;\r
+     }\r
+ \r
+diff --git a/lib/notmuch.h b/lib/notmuch.h\r
+index f066245..c671d82 100644\r
+--- a/lib/notmuch.h\r
++++ b/lib/notmuch.h\r
+@@ -230,6 +230,16 @@ notmuch_status_t\r
+ notmuch_database_create (const char *path, notmuch_database_t **database);\r
+ \r
+ /**\r
++ * Like notmuch_database_create, except optionally return an error\r
++ * message. This message is allocated by malloc and should be freed by\r
++ * the caller.\r
++ */\r
++notmuch_status_t\r
++notmuch_database_create_verbose (const char *path,\r
++                               notmuch_database_t **database,\r
++                               char **error_message);\r
++\r
++/**\r
+  * Database open mode for notmuch_database_open.\r
+  */\r
+ typedef enum {\r
+@@ -279,6 +289,17 @@ notmuch_status_t\r
+ notmuch_database_open (const char *path,\r
+                      notmuch_database_mode_t mode,\r
+                      notmuch_database_t **database);\r
++/**\r
++ * Like notmuch_database_open, except optionally return an error\r
++ * message. This message is allocated by malloc and should be freed by\r
++ * the caller.\r
++ */\r
++\r
++notmuch_status_t\r
++notmuch_database_open_verbose (const char *path,\r
++                             notmuch_database_mode_t mode,\r
++                             notmuch_database_t **database,\r
++                             char **error_message);\r
+ \r
+ /**\r
+  * Commit changes and close the given notmuch database.\r
+diff --git a/notmuch-new.c b/notmuch-new.c\r
+index ddf42c1..e6c283e 100644\r
+--- a/notmuch-new.c\r
++++ b/notmuch-new.c\r
+@@ -985,9 +985,16 @@ notmuch_new_command (notmuch_config_t *config, int argc, char *argv[])\r
+           return EXIT_FAILURE;\r
+       add_files_state.total_files = count;\r
+     } else {\r
+-      if (notmuch_database_open (db_path, NOTMUCH_DATABASE_MODE_READ_WRITE,\r
+-                                 &notmuch))\r
++      char *status_string = NULL;\r
++      if (notmuch_database_open_verbose (db_path, NOTMUCH_DATABASE_MODE_READ_WRITE,\r
++                                         &notmuch, &status_string)) {\r
++          if (status_string) {\r
++              fputs (status_string, stderr);\r
++              free (status_string);\r
++          }\r
++\r
+           return EXIT_FAILURE;\r
++      }\r
+ \r
+       if (notmuch_database_needs_upgrade (notmuch)) {\r
+           time_t now = time (NULL);\r
+diff --git a/notmuch-search.c b/notmuch-search.c\r
+index a591d45..b81ac01 100644\r
+--- a/notmuch-search.c\r
++++ b/notmuch-search.c\r
+@@ -545,6 +545,7 @@ _notmuch_search_prepare (search_context_t *ctx, notmuch_config_t *config, int ar\r
+ {\r
+     char *query_str;\r
+     unsigned int i;\r
++    char *status_string = NULL;\r
+ \r
+     switch (ctx->format_sel) {\r
+     case NOTMUCH_FORMAT_TEXT:\r
+@@ -570,9 +571,17 @@ _notmuch_search_prepare (search_context_t *ctx, notmuch_config_t *config, int ar\r
+ \r
+     notmuch_exit_if_unsupported_format ();\r
+ \r
+-    if (notmuch_database_open (notmuch_config_get_database_path (config),\r
+-                             NOTMUCH_DATABASE_MODE_READ_ONLY, &ctx->notmuch))\r
++    if (notmuch_database_open_verbose (\r
++          notmuch_config_get_database_path (config),\r
++          NOTMUCH_DATABASE_MODE_READ_ONLY, &ctx->notmuch, &status_string)) {\r
++\r
++      if (status_string) {\r
++          fputs (status_string, stderr);\r
++          free (status_string);\r
++      }\r
++\r
+       return EXIT_FAILURE;\r
++    }\r
+ \r
+     query_str = query_string_from_args (ctx->notmuch, argc, argv);\r
+     if (query_str == NULL) {\r
+diff --git a/test/symbol-test.cc b/test/symbol-test.cc\r
+index 3e96c03..d979f83 100644\r
+--- a/test/symbol-test.cc\r
++++ b/test/symbol-test.cc\r
+@@ -1,11 +1,18 @@\r
+ #include <stdio.h>\r
++#include <stdlib.h>\r
+ #include <xapian.h>\r
+ #include <notmuch.h>\r
+ \r
+ \r
+ int main() {\r
+   notmuch_database_t *notmuch;\r
+-  notmuch_database_open("fakedb", NOTMUCH_DATABASE_MODE_READ_ONLY, &notmuch);\r
++  char *message = NULL;\r
++\r
++  if (notmuch_database_open_verbose  ("fakedb", NOTMUCH_DATABASE_MODE_READ_ONLY, &notmuch, &message))\r
++      if (message) {\r
++        fputs (message, stderr);\r
++        free (message);\r
++      }\r
+ \r
+   try {\r
+     (void) new Xapian::WritableDatabase("./nonexistant", Xapian::DB_OPEN);\r
+-- \r
+2.1.4\r
+\r