On Wed 2020-04-29 23:02:19 +0300, Tomi Ollila wrote: > On Tue, Apr 28 2020, Daniel Kahn Gillmor wrote: > >> Without this fix, we couldn't run both add_gnupg_home and >> add_gpgsm_home in the same test script. >> >> Signed-off-by: Daniel Kahn Gillmor >> --- >> test/test-lib.sh | 8 ++++---- >> 1 file changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/test/test-lib.sh b/test/test-lib.sh >> index ac1b9315..d9997b27 100644 >> --- a/test/test-lib.sh >> +++ b/test/test-lib.sh >> @@ -110,10 +110,10 @@ unset ALTERNATE_EDITOR >> add_gnupg_home () >> { >> local output >> - [ -d ${GNUPGHOME} ] && return >> + [ -e ${GNUPGHOME}/gpg.conf ] && return > > So far so good (except perhaps David's comment not "url" not found) > > But here this change could include "consistently quoted" variable > (or/and have it done in that one commit earlier...) agreed, quoting this would be good. --dkg