Re: [PATCH 3/3] compact: Provide user with more error feedback

Subject: Re: [PATCH 3/3] compact: Provide user with more error feedback

Date: Mon, 28 Oct 2013 18:12:33 -0400

To: Tomi Ollila, notmuch@notmuchmail.org

Cc:

From: Ben Gamari


Tomi Ollila <tomi.ollila@iki.fi> writes:

> On Sun, Oct 27 2013, Ben Gamari <bgamari.foss@gmail.com> wrote:
>
>> Provide instructions on what to do when we couldn't move the compacted
>> database into place.
>>
>> Signed-off-by: Ben Gamari <bgamari.foss@gmail.com>
>> ---
>>  lib/database.cc | 18 +++++++++++++++++-
>>  1 file changed, 17 insertions(+), 1 deletion(-)
>>
>> diff --git a/lib/database.cc b/lib/database.cc
>> index 57c2292..6f9fed1 100644
>> --- a/lib/database.cc
>> +++ b/lib/database.cc
>> @@ -938,7 +938,23 @@ notmuch_database_compact (const char* path,
>>      }
>>  
>>      if (rename(compact_xapian_path, xapian_path)) {
>> -	fprintf (stderr, "Error moving compacted database\n");
>> +	fprintf (stderr, "Error moving compacted database into place: %s\n", strerror(errno));
>> +	fprintf (stderr, "\n");
>> +	fprintf (stderr, "Encountered error %s while moving the compacted database,\n",
>> +		 strerror(errno));
>
> You probably missed my comments in id:m2d2mz7qv7.fsf@guru.guru-group.fi ?
>
Indeed I did. My apologies. New set coming.

Cheers,

- Ben
part-000.sig (application/pgp-signature)

Thread: