notmuch-config: ENOENT vs generic handling when file open fails.
authorTomi Ollila <tomi.ollila@iki.fi>
Sun, 26 Feb 2017 19:12:39 +0000 (21:12 +0200)
committerDavid Bremner <david@tethera.net>
Tue, 28 Feb 2017 12:08:26 +0000 (08:08 -0400)
When opening configuration file fails, ENOENT (file not found) is
handled specially -- in setup missing file is ok (often expected),
and otherwise user can be informed to run notmuch setup.

In any other case the the reason is unknown, so there is no other
option but to print generic error message to stderr.

notmuch-config.c

index 959410cce7b81b95a29969b67f41840610dc9f12..e4aaef610173661b83491cfec95547f82703ad3d 100644 (file)
@@ -215,24 +215,23 @@ get_config_from_file (notmuch_config_t *config, notmuch_bool_t create_new)
 
     FILE *fp = fopen(config->filename, "r");
     if (fp == NULL) {
 
     FILE *fp = fopen(config->filename, "r");
     if (fp == NULL) {
-       /* If create_new is true, then the caller is prepared for a
-        * default configuration file in the case of FILE NOT FOUND.
-        */
-       if (create_new) {
-           config->is_new = TRUE;
-           ret = TRUE;
-           goto out;
-       } else if (errno == ENOENT) {
-           fprintf (stderr, "Configuration file %s not found.\n"
-                    "Try running 'notmuch setup' to create a configuration.\n",
-                    config->filename);
-           goto out;
+       if (errno == ENOENT) {
+           /* If create_new is true, then the caller is prepared for a
+            * default configuration file in the case of FILE NOT FOUND.
+            */
+           if (create_new) {
+               config->is_new = TRUE;
+               ret = TRUE;
+           } else {
+               fprintf (stderr, "Configuration file %s not found.\n"
+                        "Try running 'notmuch setup' to create a configuration.\n",
+                        config->filename);
+           }
        } else {
        } else {
-           fprintf (stderr, "Error opening config file '%s': %s\n"
-                    "Try running 'notmuch setup' to create a configuration.\n",
+           fprintf (stderr, "Error opening config file '%s': %s\n",
                     config->filename, strerror(errno));
                     config->filename, strerror(errno));
-           goto out;
        }
        }
+       goto out;
     }
 
     config_str = talloc_zero_array (config, char, config_bufsize);
     }
 
     config_str = talloc_zero_array (config, char, config_bufsize);