From: Daniel Kahn Gillmor Date: Fri, 8 Jul 2016 09:27:17 +0000 (+0200) Subject: [PATCH v4 06/16] Prefer gpg2 in the test suite if available X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=494b4faa966e5569d0df150788b7d7834683657a;p=notmuch-archives.git [PATCH v4 06/16] Prefer gpg2 in the test suite if available --- diff --git a/3e/c33a2e175e93a79d7b2dc7de7521bdc2d647de b/3e/c33a2e175e93a79d7b2dc7de7521bdc2d647de new file mode 100644 index 000000000..2295d9a6b --- /dev/null +++ b/3e/c33a2e175e93a79d7b2dc7de7521bdc2d647de @@ -0,0 +1,189 @@ +Return-Path: +X-Original-To: notmuch@notmuchmail.org +Delivered-To: notmuch@notmuchmail.org +Received: from localhost (localhost [127.0.0.1]) + by arlo.cworth.org (Postfix) with ESMTP id B53966DE02D9 + for ; Fri, 8 Jul 2016 03:13:49 -0700 (PDT) +X-Virus-Scanned: Debian amavisd-new at cworth.org +X-Spam-Flag: NO +X-Spam-Score: 0.06 +X-Spam-Level: +X-Spam-Status: No, score=0.06 tagged_above=-999 required=5 tests=[AWL=0.060] + autolearn=disabled +Received: from arlo.cworth.org ([127.0.0.1]) + by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024) + with ESMTP id jw1BAxRuWCC6 for ; + Fri, 8 Jul 2016 03:13:41 -0700 (PDT) +Received: from che.mayfirst.org (che.mayfirst.org [162.247.75.118]) + by arlo.cworth.org (Postfix) with ESMTP id 4EE826DE0350 + for ; Fri, 8 Jul 2016 03:13:12 -0700 (PDT) +Received: from fifthhorseman.net (unknown [88.128.80.54]) + by che.mayfirst.org (Postfix) with ESMTPSA id 82B97F99A + for ; Fri, 8 Jul 2016 06:13:11 -0400 (EDT) +Received: by fifthhorseman.net (Postfix, from userid 1000) + id 9EF4A211C9; Fri, 8 Jul 2016 11:27:34 +0200 (CEST) +From: Daniel Kahn Gillmor +To: Notmuch Mail +Subject: [PATCH v4 06/16] Prefer gpg2 in the test suite if available +Date: Fri, 8 Jul 2016 11:27:17 +0200 +Message-Id: <1467970047-8013-7-git-send-email-dkg@fifthhorseman.net> +X-Mailer: git-send-email 2.8.1 +In-Reply-To: <1467970047-8013-1-git-send-email-dkg@fifthhorseman.net> +References: <1467970047-8013-1-git-send-email-dkg@fifthhorseman.net> +X-BeenThere: notmuch@notmuchmail.org +X-Mailman-Version: 2.1.20 +Precedence: list +List-Id: "Use and development of the notmuch mail system." + +List-Unsubscribe: , + +List-Archive: +List-Post: +List-Help: +List-Subscribe: , + +X-List-Received-Date: Fri, 08 Jul 2016 10:13:49 -0000 + +Now that the notmuch client prefers gpg2 if available, having the test +suite use the same preference makes it more likely to validate as +expected. + +Be warned that the final test in T350-crypto.sh fails with an infinite +loop in gpg if you're using an unpatched GnuPG 2.1.10, due to an +upstream GnuPG bug: https://bugs.gnupg.org/gnupg/issue2187. In +debian, this is resolved in 2.1.10-3 +--- + test/README | 2 +- + test/T030-config.sh | 2 +- + test/T040-setup.sh | 2 +- + test/T350-crypto.sh | 16 ++++++++-------- + test/test-lib.sh | 10 +++++++++- + 5 files changed, 20 insertions(+), 12 deletions(-) + +diff --git a/test/README b/test/README +index 104a120..b4489da 100644 +--- a/test/README ++++ b/test/README +@@ -23,7 +23,7 @@ that you know if you break anything. + - emacs(1) + - emacsclient(1) + - gdb(1) +- - gpg(1) ++ - gpg(1) or gpg2(1) + - python(1) + + If your system lacks these tools or have older, non-upgreable versions +diff --git a/test/T030-config.sh b/test/T030-config.sh +index 0915abd..9eb9294 100755 +--- a/test/T030-config.sh ++++ b/test/T030-config.sh +@@ -54,7 +54,7 @@ new.tags=unread;inbox; + new.ignore= + search.exclude_tags= + maildir.synchronize_flags=true +-crypto.gpg_path=gpg ++crypto.gpg_path=$GPG + foo.string=this is another string value + foo.list=this;is another;list value; + built_with.compact=something +diff --git a/test/T040-setup.sh b/test/T040-setup.sh +index 021f2d0..afc7bc0 100755 +--- a/test/T040-setup.sh ++++ b/test/T040-setup.sh +@@ -29,7 +29,7 @@ new.tags=foo;bar; + new.ignore= + search.exclude_tags=baz; + maildir.synchronize_flags=true +-crypto.gpg_path=gpg ++crypto.gpg_path=$GPG + built_with.compact=something + built_with.field_processor=something + built_with.retry_lock=something" +diff --git a/test/T350-crypto.sh b/test/T350-crypto.sh +index 3656cce..4bc15bc 100755 +--- a/test/T350-crypto.sh ++++ b/test/T350-crypto.sh +@@ -12,11 +12,11 @@ add_gnupg_home () + local output + [ -d ${GNUPGHOME} ] && return + mkdir -m 0700 "$GNUPGHOME" +- gpg --no-tty --import <$TEST_DIRECTORY/gnupg-secret-key.asc >"$GNUPGHOME"/import.log 2>&1 ++ $GPG --no-tty --import <$TEST_DIRECTORY/gnupg-secret-key.asc >"$GNUPGHOME"/import.log 2>&1 + test_debug "cat $GNUPGHOME/import.log" +- if (gpg --quick-random --version >/dev/null 2>&1) ; then ++ if ($GPG --quick-random --version >/dev/null 2>&1) ; then + echo quick-random >> "$GNUPGHOME"/gpg.conf +- elif (gpg --debug-quick-random --version >/dev/null 2>&1) ; then ++ elif ($GPG --debug-quick-random --version >/dev/null 2>&1) ; then + echo debug-quick-random >> "$GNUPGHOME"/gpg.conf + fi + echo no-emit-version >> "$GNUPGHOME"/gpg.conf +@@ -26,7 +26,7 @@ add_gnupg_home () + + add_gnupg_home + # get key fingerprint +-FINGERPRINT=$(gpg --no-tty --list-secret-keys --with-colons --fingerprint | grep '^fpr:' | cut -d: -f10) ++FINGERPRINT=$($GPG --no-tty --list-secret-keys --with-colons --fingerprint | grep '^fpr:' | cut -d: -f10) + + test_expect_success 'emacs delivery of signed message' \ + 'emacs_fcc_message \ +@@ -67,8 +67,8 @@ test_expect_equal_json \ + + test_begin_subtest "signature verification with full owner trust" + # give the key full owner trust +-echo "${FINGERPRINT}:6:" | gpg --no-tty --import-ownertrust >>"$GNUPGHOME"/trust.log 2>&1 +-gpg --no-tty --check-trustdb >>"$GNUPGHOME"/trust.log 2>&1 ++echo "${FINGERPRINT}:6:" | $GPG --no-tty --import-ownertrust >>"$GNUPGHOME"/trust.log 2>&1 ++$GPG --no-tty --check-trustdb >>"$GNUPGHOME"/trust.log 2>&1 + output=$(notmuch show --format=json --verify subject:"test signed message 001" \ + | notmuch_json_show_sanitize \ + | sed -e 's|"created": [1234567890]*|"created": 946728000|') +@@ -325,8 +325,8 @@ Notmuch Test Suite key revocation (automated) $(date '+%F_%T%z') + y + + " \ +- | gpg --no-tty --quiet --command-fd 0 --armor --gen-revoke "0x${FINGERPRINT}!" 2>/dev/null \ +- | gpg --no-tty --quiet --import ++ | $GPG --no-tty --quiet --command-fd 0 --armor --gen-revoke "0x${FINGERPRINT}!" 2>/dev/null \ ++ | $GPG --no-tty --quiet --import + output=$(notmuch show --format=json --verify subject:"test signed message 001" \ + | notmuch_json_show_sanitize \ + | sed -e 's|"created": [1234567890]*|"created": 946728000|') +diff --git a/test/test-lib.sh b/test/test-lib.sh +index aac0343..5c14d1e 100644 +--- a/test/test-lib.sh ++++ b/test/test-lib.sh +@@ -86,6 +86,13 @@ unset GREP_OPTIONS + # For emacsclient + unset ALTERNATE_EDITOR + ++# choose the preferred GnuPG binary: ++if command -v gpg2 > /dev/null; then ++ GPG=gpg2 ++else ++ GPG=gpg ++fi ++ + # Convenience + # + # A regexp to match 5 and 40 hexdigits +@@ -1172,6 +1179,7 @@ test_emacs () { + $load_emacs_tests \ + --eval '(setq server-name \"$server_name\")' \ + --eval '(server-start)' \ ++ --eval '(setq epg-gpg-program \"$GPG\")' \ + --eval '(orphan-watchdog $$)'" || return + EMACS_SERVER="$server_name" + # wait until the emacs server is up +@@ -1368,7 +1376,7 @@ test_declare_external_prereq dtach + test_declare_external_prereq emacs + test_declare_external_prereq ${TEST_EMACSCLIENT} + test_declare_external_prereq gdb +-test_declare_external_prereq gpg ++test_declare_external_prereq gpg2 || test_declare_external_prereq gpg + test_declare_external_prereq openssl + test_declare_external_prereq gpgsm + test_declare_external_prereq ${NOTMUCH_PYTHON} +-- +2.8.1 +