]> git.notmuchmail.org Git - notmuch/commitdiff
test: emacs: fix a couple of shellcheck complaints
authorFelipe Contreras <felipe.contreras@gmail.com>
Sat, 15 May 2021 20:47:43 +0000 (15:47 -0500)
committerDavid Bremner <david@tethera.net>
Mon, 17 May 2021 10:28:44 +0000 (07:28 -0300)
In test-lib-emacs.sh line 20:
    test_require_external_prereq ${TEST_EMACS} || ret=1
                                 ^-----------^ SC2086: Double quote to prevent globbing and word splitting.

Did you mean:
    test_require_external_prereq "${TEST_EMACS}" || ret=1

In test-lib-emacs.sh line 21:
    test_require_external_prereq ${TEST_EMACSCLIENT} || ret=1
                                 ^-----------------^ SC2086: Double quote to prevent globbing and word splitting.

Did you mean:
    test_require_external_prereq "${TEST_EMACSCLIENT}" || ret=1

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
test/test-lib.sh

index 5c56bc22a56fed916ebe18430fe0ca8d66231f3b..756e5baf33078203a5c16f159a7604c76fd9f234 100644 (file)
@@ -116,8 +116,8 @@ unset NAME
 
 test_require_emacs () {
     local ret=0
-    test_require_external_prereq ${TEST_EMACS} || ret=1
-    test_require_external_prereq ${TEST_EMACSCLIENT} || 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
 }