Re: [PATCH] test: allow to use --full-sync

Subject:Re: [PATCH] test: allow to use --full-sync

Date:Wed, 16 Feb 2022 09:04:17 -0400

To:Michael J Gruber ,notmuch@notmuchmail.org

Cc:

From:David Bremner


Michael J Gruber <git@grubix.eu> writes:

> Some build infrastructure appears to habe problems with mtime/stat,

have

> leading to spurious failures (noticed on s390x and aarch64 with epel8).
> Allow the test suite to be run with --full-sync so that release builds
> can use the test suite while avoiding spurious failures.
>
> Signed-off-by: Michael J Gruber <git@grubix.eu>
> ---
>  test/README             | 8 ++++++++
>  test/test-lib-common.sh | 7 ++++++-
>  test/test-lib.sh        | 2 +-
>  3 files changed, 15 insertions(+), 2 deletions(-)
>
> diff --git a/test/README b/test/README
> index 10f127cb..1de79b78 100644
> --- a/test/README
> +++ b/test/README
> @@ -110,6 +110,14 @@ printed on screen. This printing can be disabled by setting the
>  NOTMUCH_TEST_QUIET variable to a non-null value. Message on test
>  failures and skips are still printed.

typo in variable name.

How confident are you that this actually fixes your intermittent
failures?  There are quite a few literal invocations of "notmuch new" in
the test suite. Would those need to be changed as well?

d
_______________________________________________
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-leave@notmuchmail.org

Thread: