[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