Re: [PATCH 10/13] test: emacs: check for configured emacs

Subject: Re: [PATCH 10/13] test: emacs: check for configured emacs

Date: Sat, 01 May 2021 23:20:50 +0300

To: Felipe Contreras, notmuch@notmuchmail.org

Cc: Daniel Kahn Gillmor

From: Tomi Ollila


On Sat, May 01 2021, Felipe Contreras wrote:

> Commit d59d9c81 (test: Make the emacsclient binary user-configurable,
> 2012-11-27) modified the prereq check for the configured emacsclient,
> but we probably want to do the same for emacs itself.

LGTM.

>
> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
> ---
>  test/test-lib-emacs.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/test-lib-emacs.sh b/test/test-lib-emacs.sh
> index 83f5b10b..3075fa59 100644
> --- a/test/test-lib-emacs.sh
> +++ b/test/test-lib-emacs.sh
> @@ -16,7 +16,7 @@
>  
>  test_require_emacs () {
>      local ret=0
> -    test_require_external_prereq emacs || ret=1
> +    test_require_external_prereq ${TEST_EMACS} || ret=1
>      test_require_external_prereq ${TEST_EMACSCLIENT} || ret=1
>      test_require_external_prereq dtach || ret=1
>      return $ret
> -- 
> 2.31.0
> _______________________________________________
> notmuch mailing list -- notmuch@notmuchmail.org
> To unsubscribe send an email to notmuch-leave@notmuchmail.org
_______________________________________________
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-leave@notmuchmail.org

Thread: