unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH] test/crypto: hard code fingerprint
@ 2016-09-08 12:12 David Bremner
  2016-09-08 14:27 ` Tomi Ollila
  2016-09-09  0:23 ` David Bremner
  0 siblings, 2 replies; 3+ messages in thread
From: David Bremner @ 2016-09-08 12:12 UTC (permalink / raw)
  To: notmuch

Originally the intent was to make the test more robust against changing
test keys. It turns out that (unscientifically) gpg --with-colons output
changes more often than our test key. Rather than making the script more
complex, just hard code the fingerprint.

This fixes Debian bug #847013; I expect similar test failures as other
distros adopt gnupg 2.1.15
---

I plan to release 0.22.2 with this fix, and cf8aabdd375 (already
pushed to debian, but not debian specific).

test/T350-crypto.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/test/T350-crypto.sh b/test/T350-crypto.sh
index 3656cce..96349fa 100755
--- a/test/T350-crypto.sh
+++ b/test/T350-crypto.sh
@@ -25,8 +25,8 @@ add_gnupg_home ()
 ##################################################
 
 add_gnupg_home
-# get key fingerprint
-FINGERPRINT=$(gpg --no-tty --list-secret-keys --with-colons --fingerprint | grep '^fpr:' | cut -d: -f10)
+# Change this if we ship a new test key
+FINGERPRINT="5AEAB11F5E33DCE875DDB75B6D92612D94E46381"
 
 test_expect_success 'emacs delivery of signed message' \
 'emacs_fcc_message \
-- 
2.9.3

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] test/crypto: hard code fingerprint
  2016-09-08 12:12 [PATCH] test/crypto: hard code fingerprint David Bremner
@ 2016-09-08 14:27 ` Tomi Ollila
  2016-09-09  0:23 ` David Bremner
  1 sibling, 0 replies; 3+ messages in thread
From: Tomi Ollila @ 2016-09-08 14:27 UTC (permalink / raw)
  To: David Bremner, notmuch

On Thu, Sep 08 2016, David Bremner <david@tethera.net> wrote:

> Originally the intent was to make the test more robust against changing
> test keys. It turns out that (unscientifically) gpg --with-colons output
> changes more often than our test key. Rather than making the script more
> complex, just hard code the fingerprint.
>
> This fixes Debian bug #847013; I expect similar test failures as other
> distros adopt gnupg 2.1.15

looks tolerable to me...

> ---
>
> I plan to release 0.22.2 with this fix, and cf8aabdd375 (already
> pushed to debian, but not debian specific).
>
> test/T350-crypto.sh | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/test/T350-crypto.sh b/test/T350-crypto.sh
> index 3656cce..96349fa 100755
> --- a/test/T350-crypto.sh
> +++ b/test/T350-crypto.sh
> @@ -25,8 +25,8 @@ add_gnupg_home ()
>  ##################################################
>  
>  add_gnupg_home
> -# get key fingerprint
> -FINGERPRINT=$(gpg --no-tty --list-secret-keys --with-colons --fingerprint | grep '^fpr:' | cut -d: -f10)
> +# Change this if we ship a new test key
> +FINGERPRINT="5AEAB11F5E33DCE875DDB75B6D92612D94E46381"
>  
>  test_expect_success 'emacs delivery of signed message' \
>  'emacs_fcc_message \
> -- 
> 2.9.3
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> https://notmuchmail.org/mailman/listinfo/notmuch

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] test/crypto: hard code fingerprint
  2016-09-08 12:12 [PATCH] test/crypto: hard code fingerprint David Bremner
  2016-09-08 14:27 ` Tomi Ollila
@ 2016-09-09  0:23 ` David Bremner
  1 sibling, 0 replies; 3+ messages in thread
From: David Bremner @ 2016-09-09  0:23 UTC (permalink / raw)
  To: notmuch

David Bremner <david@tethera.net> writes:

> Originally the intent was to make the test more robust against changing
> test keys. It turns out that (unscientifically) gpg --with-colons output
> changes more often than our test key. Rather than making the script more
> complex, just hard code the fingerprint.

pushed

d

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-09-09  0:23 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-08 12:12 [PATCH] test/crypto: hard code fingerprint David Bremner
2016-09-08 14:27 ` Tomi Ollila
2016-09-09  0:23 ` David Bremner

Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).