From 34c840a23e1db4c6cf4a836b441d027fc17e9706 Mon Sep 17 00:00:00 2001 From: Jameson Graef Rollins Date: Sun, 22 Nov 2009 16:58:35 -0500 Subject: [PATCH] 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. --- notmuch-setup.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) 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; + } }