Re: [PULL svn-fe] vcs-svn: simplifications, error handling improvements

From: Junio C Hamano <gitster@pobox.com>
Date: Mar 23, 2011, 06:03 UTC
Message-ID: <7vbp12gyqk.fsf@alter.siamese.dyndns.org>
In-Reply-To: <7vsjuegzj8.fsf@alter.siamese.dyndns.org>

Junio C Hamano <gitster@pobox.com> writes:

Show 5 quoted lines

> By the way, I've been getting annoyed by these three "sleep 100" getting
> stuck in t0081 and spending their sweet timeout while running my tests
> (prove is on, "make test </dev/null" to forbid it from reading my stdin).
> I see attempts to kill them early with "kill $!" but apparently they are
> not working. Can you take a look at it?

A bit more datapoint. The first one works just fine, while the second one gets stuck:

    $ cd t && make T=t0081-line-buffer.sh test
    $ cd t && make T=t0081-line-buffer.sh prove

I notice that [sh] spawned from prove is orphaned.

21643 pts/9 Ss 0:00 \_ bash 17149 pts/9 S+ 0:00 | \_ make T=t0081-line-buffer.sh prove 17155 pts/9 S+ 0:00 | \_ /bin/sh -c echo "*** prove ***"; GIT_CONFIG=.git/ 17156 pts/9 S+ 0:00 | \_ /usr/bin/perl /usr/bin/prove --exec /bin/sh t 17157 pts/9 Z+ 0:00 | \_ [sh] <defunct>

Is this a known bug in prove perhaps?

    $ /usr/bin/prove --version
    TAP::Harness v3.17 and Perl v5.10.1