[PATCH 4/4] test: fix error messages for missing binary dependencies

Tomi Ollila tomi.ollila at nixu.com
Tue Nov 29 06:48:55 PST 2011


LGTM


On Tue, 29 Nov 2011 02:41:07 +0400, Dmitry Kurochkin <dmitry.kurochkin at gmail.com> wrote:
> The fake missing binary functions check if the binary has already be
> added to the diagnostic message to avoid duplicates.  Unfortunately,
> this check was buggy because the message string does not have the
> trailing space.
> ---
>  test/test-lib.sh |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/test/test-lib.sh b/test/test-lib.sh
> index 2861d88..a975957 100644
> --- a/test/test-lib.sh
> +++ b/test/test-lib.sh
> @@ -556,7 +556,7 @@ test_declare_external_prereq () {
>  	hash $binary 2>/dev/null || eval "
>  	test_missing_external_prereq_${binary}_=t
>  $binary () {
> -	echo -n \"\$test_subtest_missing_external_prereqs_\" | grep -qe \" $name \" ||
> +	echo -n \"\$test_subtest_missing_external_prereqs_ \" | grep -qe \" $name \" ||
>  	test_subtest_missing_external_prereqs_=\"\$test_subtest_missing_external_prereqs_ $name\"
>  	false
>  }"
> -- 
> 1.7.7.3
> 
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
> 

-- 
Tomi Ollila !! tomi.ollila at nixu.fi !! +358 400 888 366
Nixu Oy !! http://www.nixu.fi/ !! Keilaranta 15, FI-02150 Espoo, Finland


More information about the notmuch mailing list