[PATCH 0/9] test: (hopefully) better test prerequisites

Jameson Graef Rollins jrollins at finestructure.net
Thu Nov 17 01:14:07 PST 2011


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?

jamie.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20111117/95a951e4/attachment.pgp>


More information about the notmuch mailing list