Re: [PATCH 1/3] test: Don't return the result of checking for running emacs to the tester.
Can you please elaborate why this is needed? Regards, Dmitry ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH 1/3] test: Don't return the result of checking for running emacs to the tester.
(And for the list...) On Tue, 17 Jan 2012 18:20:04 +0400, Dmitry Kurochkin 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. pgpWGYEF7yq5G.pgp Description: PGP signature ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH 1/3] test: Don't return the result of checking for running emacs to the tester.
On Tue, 17 Jan 2012 14:37:52 +, David Edmondson wrote: > (And for the list...) > > On Tue, 17 Jan 2012 18:20:04 +0400, Dmitry Kurochkin > 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 ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH 1/3] test: Don't return the result of checking for running emacs to the tester.
LGTM. Regards, Dmitry ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch