The extra machinery to check for the actual output format is justified by the possibility that distros may patch this newere output format into older versions of gmime. --- configure | 17 +++++++++++++++++ test/T355-smime.sh | 7 +++++-- 2 files changed, 22 insertions(+), 2 deletions(-)
diff --git a/configure b/configure index d6e1200e..922de60d 100755 --- a/configure +++ b/configure @@ -588,6 +588,11 @@ int main () { #ifdef CHECK_VALIDITY validity = g_mime_certificate_get_id_validity (cert); if (validity != GMIME_VALIDITY_FULL) return !! fprintf (stderr, "Got validity %d, expected %d\n", validity, GMIME_VALIDITY_FULL); +#endif +#ifdef CHECK_EMAIL + const char *email = g_mime_certificate_get_email (cert); + if (! email) return !! fprintf (stderr, "no email returned"); + if (email[0] == '<') return 2; #endif return 0; } @@ -622,6 +627,15 @@ EOF errors=$((errors + 1)) fi fi + printf "Checking for GMime new email format... " + if ${CC} -DCHECK_EMAIL ${CFLAGS} ${gmime_cflags} _check_gmime_cert.c ${gmime_ldflags} -o _check_email && + GNUPGHOME=${TEMP_GPG} ./_check_email; then + gmime_new_email_format=1 + printf "Yes.\n" + else + gmime_new_email_format=0 + printf "No (some tests will be skipped).\n" + fi else printf 'No.\nFailed to set up gpgsm for testing X.509 certificate validity support.\n' errors=$((errors + 1)) @@ -1559,6 +1573,9 @@ NOTMUCH_HAVE_XAPIAN_DB_RETRY_LOCK=${WITH_RETRY_LOCK} # Whether GMime can verify X.509 certificate validity NOTMUCH_GMIME_X509_CERT_VALIDITY=${gmime_x509_cert_validity} +# Whether GMime returns bare emails (without <>) +NOTMUCH_GMIME_NEW_EMAIL_FORMAT=${gmime_new_email_format} + # Whether GMime can verify signatures when decrypting with a session key: NOTMUCH_GMIME_VERIFY_WITH_SESSION_KEY=${gmime_verify_with_session_key} diff --git a/test/T355-smime.sh b/test/T355-smime.sh index 31fa4b4e..2905263b 100755 --- a/test/T355-smime.sh +++ b/test/T355-smime.sh @@ -35,6 +35,9 @@ EOF test_expect_equal_file EXPECTED OUTPUT test_begin_subtest "signature verification (notmuch CLI)" +if [ $NOTMUCH_GMIME_NEW_EMAIL_FORMAT != 1 ]; then + test_subtest_known_broken +fi output=$(notmuch show --format=json --verify subject:"test signed message 001" \ | notmuch_json_show_sanitize \ | sed -e 's|"created": [-1234567890]*|"created": 946728000|g' \ @@ -46,7 +49,7 @@ expected='[[[{"id": "XXXXX", "timestamp": 946728000, "date_relative": "2000-01-01", "tags": ["inbox","signed"], - "crypto": {"signed": {"status": [{"fingerprint": "'$FINGERPRINT'", "status": "good","userid": "CN=Notmuch Test Suite", "email": "<test_su...@notmuchmail.org>", "expires": 424242424, "created": 946728000}]}}, + "crypto": {"signed": {"status": [{"fingerprint": "'$FINGERPRINT'", "status": "good","userid": "CN=Notmuch Test Suite", "email": "test_su...@notmuchmail.org", "expires": 424242424, "created": 946728000}]}}, "headers": {"Subject": "test signed message 001", "From": "Notmuch Test Suite <test_su...@notmuchmail.org>", "To": "test_su...@notmuchmail.org", @@ -55,7 +58,7 @@ expected='[[[{"id": "XXXXX", "sigstatus": [{"fingerprint": "'$FINGERPRINT'", "status": "good", "userid": "CN=Notmuch Test Suite", - "email": "<test_su...@notmuchmail.org>", + "email": "test_su...@notmuchmail.org", "expires": 424242424, "created": 946728000}], "content-type": "multipart/signed", -- 2.35.1 _______________________________________________ notmuch mailing list -- notmuch@notmuchmail.org To unsubscribe send an email to notmuch-le...@notmuchmail.org