notmuch: Fix setup so that accepting the default mail path works.

The recent change from GIOChannel to getline, (with a semantic
change of the newline terminator now being included in the
result that setup_command sees), broke this.
This commit is contained in:
Carl Worth 2009-10-20 13:16:16 -07:00
parent 7f254fb603
commit 28fa0bc2d6

View file

@ -289,13 +289,17 @@ setup_command (int argc, char *argv[])
getline (&mail_directory, &line_size, stdin); getline (&mail_directory, &line_size, stdin);
printf ("\n"); printf ("\n");
if (mail_directory &&
mail_directory[strlen(mail_directory)-1] == '\n')
{
mail_directory[strlen(mail_directory)-1] = '\0';
}
if (mail_directory == NULL || strlen (mail_directory) == 0) { if (mail_directory == NULL || strlen (mail_directory) == 0) {
if (mail_directory) if (mail_directory)
free (mail_directory); free (mail_directory);
mail_directory = default_path; mail_directory = default_path;
} else { } else {
if (mail_directory[strlen(mail_directory)-1] == '\n')
mail_directory[strlen(mail_directory)-1] = '\0';
/* XXX: Instead of telling the user to use an environment /* XXX: Instead of telling the user to use an environment
* variable here, we should really be writing out a configuration * variable here, we should really be writing out a configuration
* file and loading that on the next run. */ * file and loading that on the next run. */