From: Jameson Graef Rollins <jrollins@finestructure.net>
Date: Sun, 22 Nov 2009 21:58:35 +0000 (-0500)
Subject: modify notmuch_setup_command to return 1 if config file write fails.
X-Git-Tag: 0.1~332
X-Git-Url: https://git.notmuchmail.org/git?p=notmuch;a=commitdiff_plain;h=34c840a23e1db4c6cf4a836b441d027fc17e9706

modify notmuch_setup_command to return 1 if config file write fails.

This fixes a small bug in notmuch_setup_command such that it returned
OK and output the setup message footer even if the config file write
step failed.
---

diff --git a/notmuch-setup.c b/notmuch-setup.c
index d06fbf80..5ec176d3 100644
--- a/notmuch-setup.c
+++ b/notmuch-setup.c
@@ -156,10 +156,11 @@ notmuch_setup_command (unused (void *ctx),
 	notmuch_config_set_database_path (config, absolute_path);
     }
 
-    notmuch_config_save (config);
-
-    if (is_new)
-	welcome_message_post_setup ();
-
-    return 0;
+    if (! notmuch_config_save (config)) {
+	if (is_new)
+	  welcome_message_post_setup ();
+	return 0;
+    } else {
+	return 1;
+    }
 }