t/lib-gpg: drop redundant killing of gpg-agent
authorTodd Zullinger <tmz@pobox.com>
Fri, 8 Feb 2019 03:17:46 +0000 (22:17 -0500)
committerJunio C Hamano <gitster@pobox.com>
Fri, 8 Feb 2019 18:01:46 +0000 (10:01 -0800)
In 53fc999306 ("gpg-interface t: extend the existing GPG tests with
GPGSM", 2018-07-20), the gpgconf call which kills gpg-agent was copied
from the existing gpg setup code.

The reason for killing gpg-agent is given in 29ff1f8f74 ("t: lib-gpg:
flush gpg agent on startup", 2017-07-20):

When running gpg-relevant tests, a gpg-daemon is spawned for each
GNUPGHOME used. This daemon may stay running after the test and cache
file descriptors for the trash directories, even after the trash
directory is removed. This leads to ENOENT errors when attempting to
create files if tests are run multiple times.

Add a cleanup script to force flushing the gpg-agent for that GNUPGHOME
(if any) before setting up the GPG relevant-environment.

Killing gpg-agent once per test is sufficient.

Signed-off-by: Todd Zullinger <tmz@pobox.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/lib-gpg.sh
index d90160eb0b51aa478c2c70c9521a19484e3ffc97..041f10171aeb9c0f49a3f8ce583f41f755919031 100755 (executable)
@@ -61,7 +61,6 @@ then
                        | grep fingerprint: | cut -d" " -f4 | tr -d '\n' > \
                        ${GNUPGHOME}/trustlist.txt &&
                echo " S relax" >>"${GNUPGHOME}/trustlist.txt" &&
-               (gpgconf --kill gpg-agent >/dev/null 2>&1 || : ) &&
                echo hello | gpgsm --homedir "${GNUPGHOME}" >/dev/null \
                        -u committer@example.com -o /dev/null --sign - 2>&1 &&
                test_set_prereq GPGSM