[PATCH 0/9] test: (hopefully) better test prerequisites
Dmitry Kurochkin
dmitry.kurochkin at gmail.com
Thu Nov 17 03:20:07 PST 2011
Hi Jameson.
On Thu, 17 Nov 2011 01:14:07 -0800, Jameson Graef Rollins <jrollins at finestructure.net> wrote:
> On Thu, 17 Nov 2011 05:56:17 +0400, Dmitry Kurochkin <dmitry.kurochkin at gmail.com> wrote:
> > The following patch series is an attempt to introduce proper
> > dependencies for external binaries in a less intrusive way than
> > [1]. The primary aim was to avoid changing every subtest that
> > uses external binaries.
>
> Hey, Dmitry. Thanks so much for reworking Pieter's original test suite
> improvements. I think this new approach is a bit less invasive and a
> little more elegant to work with.
>
> I've looked through the patchs, and they all look ok on first glance
> (barring any possibly needed modifications that I'm not seeing). I have
> a couple of comments to follow.
>
> However, when I tried to test the tests with the patch applied I ran
> into one problem. If I try to run the test suite with dtach
> uninstalled, it looks like I'm experiencing a hang on
> emacs_deliver_message call. I wonder if either emacs_deliver_message or
> test_emacs is not doing the right thing in the case of no dtach. I
> think emacs_deliver_message should somehow not require dtach, since it's
> not actually testing any display stuff, but because it's currently using
> test_emacs, it is somehow implicitly depending on it. Any thoughts on
> how to fix that?
>
I bet you stumbled upon a hanging smtp-dummy. There is a patch [1] to
fix that. Hopefully it would get pushed soon.
As for not requiring dtach for emacs_deliver_message, that is definitely
possible. Currently, there is only one way to run emacs to keep it
simple. And I am not sure if running emacs_deliver_message without
dtach costs extra code.
Regards,
Dmitry
[1] id:"yf639dnsqtc.fsf at taco2.nixu.fi"
> jamie.
More information about the notmuch
mailing list