Hi Eric, * Eric Blake wrote on Wed, Jul 08, 2009 at 06:57:17AM CEST: > --- a/lib/autotest/general.m4 > +++ b/lib/autotest/general.m4
> @@ -1362,7 +1361,7 @@ dnl kill -13 $$ > at_fn_group_prepare > if cd "$at_group_dir" && > at_fn_test $at_group && > - . "$at_test_source" # AT_JOB_FIFO_FD<&- > + . "$at_test_source" # AT_JOB_FIFO_FD>&- > then :; else > AS_WARN([unable to parse test group: $at_group]) > at_failed=: This hunk is actually wrong, and I don't remember seeing it in the first iteration of your patch. Of course, since it only changes a comment, it is not relevant. However, you cannot close the fd for writing here, because a PIPE signal (that could happen within a test group for completely unrelated reasons) will cause the process to try to use it. It's only the reading end that would be unused here (but of course the notation would close the fd for either use). I think it's best to remove the comment, or change the trap to output to $at_job_fifo rather than the fd. I haven't tried that, though. Cheers, Ralf
