Date: Fri, 27 Nov 2009 15:18:50 +0100
From: Marten Veldthuis <marten@veldthuis.com>
To: notmuch@notmuchmail.org
Message-ID: <20091127141850.GA1198@socpsy35>
References: <1259091680-29598-1-git-send-email-chris@chris-wilson.co.uk>
	<878wdsqdvr.fsf@yoom.home.cworth.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Disposition: inline
In-Reply-To: <878wdsqdvr.fsf@yoom.home.cworth.org>
User-Agent: Mutt/1.5.20 (2009-06-14)
Subject: Re: [notmuch] [PATCH] notmuch-new: Remove the tiresome joke from
 the output.
Precedence: list

On Fri, Nov 27, 2009 at 05:46:32AM -0800, Carl Worth wrote:
>> -    } else {
>> -	printf ("No new mail---and that's not much.\n");
>>      }
>
>Shouldn't we still print *something* here, though ("No new mail")?
>Imagine the new user trying to ensure "notmuch new" is working---and
>let's say it's not, (due to the current symlink bug or so). A silent
>success here won't make it clear whether "notmuch new" has actually done
>anything or not.

Yes, at first I thought I'd read the patch wrong... I think you definately 
want some output, just not the joke. Don't worry, it *was* funny the first 
time.

-- 
- Marten