Re: CVS commit: src/external/bsd/atf/dist/atf-sh

[Julio Merino <julio%meroh.net@localhost>]

On Fri, Jan 10, 2014 at 1:39 AM, Christos Zoulas 
<christos%netbsd.org@localhost> wrote:
> Module Name:    src
> Committed By:   christos
> Date:           Fri Jan 10 01:39:32 UTC 2014
>
> Modified Files:
>         src/external/bsd/atf/dist/atf-sh: libatf-sh.subr
>
> Log Message:
> Undo previous; unfortunately the cleanup routine gets called in a different
> shell so it can't cleanup stuff set in the environment of the first shell.

What are you trying to fix?

> There are 2 problems:
>         - calling the test routine directly does not cleanup (not using
>           atf-run)
>         - when using atf-run, the cleanup routine needs state from the
>           running code which can only be stored in files. This will never
>           allow us to run tests in parallel since we need to keep a known
>           place to pass state.

I don't see how this prevents running tests in parallel. You just need
to maintain one work directory for every test body/cleanup pair and
delete the directory only after cleanup... and you cannot share work
directories among tests anyway so you have to do that one way or
another.

-- 
Julio Merino / @jmmv