[PATCH] test: do not hide test_emacs errors

Dmitry Kurochkin dmitry.kurochkin at gmail.com
Sat Nov 12 23:06:43 PST 2011


On Sat, 12 Nov 2011 09:57:53 -0500, David Bremner <david at tethera.net> wrote:
> On Fri,  1 Jul 2011 06:23:48 +0400, Dmitry Kurochkin <dmitry.kurochkin at gmail.com> wrote:
> > Do not redirect test_emacs stderr to /dev/null.  Test_emacs uses
> > emacsclient(1) now and it does not print unwanted messages (like
> > those from `message') to stderr.  But it does print useful
> > errors, e.g. when emacs server connection fails, given expression
> > is not valid or undefined function is called.
> 
> Is this patch still relevant now that we use dtach to run the tests?
> 

It is still relevant.  If test_emacs prints any errors, we should see
them.  If it does not, we should remove the redirection.

But we need to check for other similar cases.  New test cases may have
been added with test_emacs stderr redirect.

> I didn't notice any difference, but maybe I was not supposed to.
> 

I think that is expected.

Regards,
  Dmitry

> d


More information about the notmuch mailing list