Re: [PATCH] test: aggregate-results updates

Subject: Re: [PATCH] test: aggregate-results updates

Date: Tue, 03 Aug 2021 22:39:06 +0300

To: David Bremner,


From: Tomi Ollila

On Mon, Aug 02 2021, David Bremner wrote:

> Tomi Ollila <> writes:
>> -	mkdir -p "$test_results_dir"
>> +	test -d "$test_results_dir" || mkdir "$test_results_dir"
> Lately I've notice some complaints during parallel test running
> 	mkdir: cannot create directory '/home/bremner/software/upstream/notmuch/test/test-results': File exists
> It seems like this change might be the culprit. Can you explain why it
> was needed?

ah, parallel test non-atomicity is the problem; fix would be to restore -p to mkdir
(so it does not fail when another program executed mkdir between that test
check and mkdir execution)


test -d "$test_results_dir" || mkdir -p "$test_results_dir"

Checking the dir with shell builtin drops need to execute mkdir in external
process, which makes the tests run a bit snappier (in small increments).

notmuch mailing list --
To unsubscribe send an email to