From: Carl Worth <cworth@cworth.org>
Date: Thu, 12 Nov 2009 01:25:42 +0000 (-0800)
Subject: notmuch setup: Add some comments when creating a .notmuch-config file.
X-Git-Tag: 0.1~535
X-Git-Url: https://git.notmuchmail.org/git?p=notmuch;a=commitdiff_plain;h=c884c30c30f97a857a0fae16d0d212632080361d

notmuch setup: Add some comments when creating a .notmuch-config file.

The "notmuch setup" command is nice and all, but we also want to allow
users to conveniently edit the configuration file directly.
---

diff --git a/notmuch-config.c b/notmuch-config.c
index 0dc2b133..f7313bfa 100644
--- a/notmuch-config.c
+++ b/notmuch-config.c
@@ -23,6 +23,36 @@
 #include <pwd.h>
 #include <netdb.h>
 
+static const char toplevel_config_comment[] =
+    " .notmuch-config - Configuration file for the notmuch mail system\n"
+    "\n"
+    " For more information about notmuch, see http://notmuchmail.org";
+
+static const char database_config_comment[] =
+    " Database configuration\n"
+    "\n"
+    " The only value supported here is 'path' which should be the top-level\n"
+    " directory where your mail currently exists and to where mail will be\n"
+    " delivered in the future. Files should be individual email messages.\n"
+    " Notmuch will store its database within a sub-directory of the path\n"
+    " configured here named \".notmuch\".\n";
+
+static const char user_config_comment[] =
+    " User configuration\n"
+    "\n"
+    " Here is where you can let notmuch know how you would like to be\n"
+    " addressed. Valid settings are\n"
+    "\n"
+    "\tname		Your full name.\n"
+    "\tprimary_email	Your primary email address.\n"
+    "\tother_email	A list (separated by ';') of other email addresses\n"
+    "\t		at which you receive email.\n"
+    "\n"
+    " Notmuch will use the various email addresses configured here when\n"
+    " formatting replies. It will avoid including your own addresses in the\n"
+    " recipient list of replies, and will set the From address based on the\n"
+    " address to which the original email was addressed.\n";
+
 struct _notmuch_config {
     char *filename;
     GKeyFile *key_file;
@@ -118,6 +148,7 @@ notmuch_config_t *
 notmuch_config_open (void *ctx, const char *filename)
 {
     GError *error = NULL;
+    int config_file_is_new = 0;
 
     notmuch_config_t *config = talloc (ctx, notmuch_config_t);
     if (config == NULL) {
@@ -156,6 +187,8 @@ notmuch_config_open (void *ctx, const char *filename)
 	    talloc_free (config);
 	    return NULL;
 	}
+
+	config_file_is_new = 1;
     }
 
     if (notmuch_config_get_database_path (config) == NULL) {
@@ -201,6 +234,17 @@ notmuch_config_open (void *ctx, const char *filename)
 	}
     }
 
+    /* When we create a new configuration file here, we  add some
+     * comments to help the user understand what can be done. */
+    if (config_file_is_new) {
+	g_key_file_set_comment (config->key_file, NULL, NULL,
+				toplevel_config_comment, NULL);
+	g_key_file_set_comment (config->key_file, "database", NULL,
+				database_config_comment, NULL);
+	g_key_file_set_comment (config->key_file, "user", NULL,
+				user_config_comment, NULL);
+    }
+
     return config;
 }