From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>
To: David Edmondson <dme@dme.org>, notmuch@notmuchmail.org
Subject: Re: [PATCH 1/3] test: Don't return the result of checking for running
	emacs to the tester.
In-Reply-To: <cunipkapf7j.fsf@hotblack-desiato.hh.sledj.net>
References: <87pqei327k.fsf@gmail.com>
	<1326809224-22673-1-git-send-email-dme@dme.org>
	<87aa5mxvfv.fsf@gmail.com>
	<cunipkapf7j.fsf@hotblack-desiato.hh.sledj.net>
User-Agent: Notmuch/0.11+73~g8a5402e (http://notmuchmail.org) Emacs/23.3.1
	(x86_64-pc-linux-gnu)
Date: Tue, 17 Jan 2012 18:51:58 +0400
Message-ID: <87wr8qwfe9.fsf@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Precedence: list

On Tue, 17 Jan 2012 14:37:52 +0000, David Edmondson <dme@dme.org> wrote:
> (And for the list...)
> 
> On Tue, 17 Jan 2012 18:20:04 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:
> > Can you please elaborate why this is needed?
> 
> This code:
> 
> 		    # wait until the emacs server is up
> 		    until test_emacs '()' 2>/dev/null; do
> 			    sleep 1
> 		    done
> 
> outputs 'nil', so the first caller to test_emacs has 'nil\n' prepended
> to their expected output.

Thanks.  Would be nice to have this explained in the commit message.  No
need to resend just because of this though.

Regards,
  Dmitry