Re: [PATCH 2/2] python-cffi: returned OwnedMessage objects from Message.replies

2022-01-11 Thread David Bremner
Floris Bruynooghe  writes:

> On Sun 09 Jan 2022 at 09:26 -0400, David Bremner wrote:
>> One thing I would like to think
>> about is the length of time it takes to run the pytests. It is not
>> currently the bottleneck in running the parallel tests for me, but it is
>> among the slower T*.sh. So it might be nice at some point to split into
>> several invocations for pytest, assuming multiple invocations of pytest
>> can run in parallel.
>
> There is the pytest-xdist plugin which is widely used and probably will
> work fine on the notmuch testsuite (I even contributed to the plugin at
> some point).  It will split the workload between however many processes
> you choose, running them in parallel.

It sounds like it has potential, but I'm not sure how it would interact
with the existing scheme controlled by make. I suspect we'd end up with
too many processes/threads in total.

d


___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


[PATCH] test/gpgsm: use --with-colons when calculating fingerprint.

2022-01-11 Thread David Bremner
As stressed by the gpg documentation, the non-'with-colons' output
format is subject to change, and indeed it did in 2.3.x (x<=3). This
should make the the test suite more robust against such changes.
---
 test/test-lib.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/test-lib.sh b/test/test-lib.sh
index 6bc0b723..f1275b85 100644
--- a/test/test-lib.sh
+++ b/test/test-lib.sh
@@ -145,7 +145,7 @@ add_gpgsm_home () {
 mkdir -p -m 0700 "$GNUPGHOME"
 gpgsm --batch --no-tty --no-common-certs-import --pinentry-mode=loopback 
--passphrase-fd 3 \
  --disable-dirmngr --import  >"$GNUPGHOME"/import.log 2>&1 3<<<'' 
<$NOTMUCH_SRCDIR/test/smime/0xE0972A47.p12
-fpr=$(gpgsm --batch --list-key test_su...@notmuchmail.org | sed -n 
's/.*fingerprint: //p')
+fpr=$(gpgsm --batch --with-colons --list-key test_su...@notmuchmail.org | 
awk -F: '/^fpr/ {print $10}')
 echo "$fpr S relax" >> "$GNUPGHOME/trustlist.txt"
 gpgsm --quiet --batch --no-tty --no-common-certs-import --disable-dirmngr 
--import < $NOTMUCH_SRCDIR/test/smime/ca.crt
 echo "4D:E0:FF:63:C0:E9:EC:01:29:11:C8:7A:EE:DA:3A:9A:7F:6E:C1:0D S" >> 
"$GNUPGHOME/trustlist.txt"
-- 
2.34.1

___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


Re: [PATCH 2/2] python-cffi: returned OwnedMessage objects from Message.replies

2022-01-11 Thread Floris Bruynooghe
On Sun 09 Jan 2022 at 09:26 -0400, David Bremner wrote:
> One thing I would like to think
> about is the length of time it takes to run the pytests. It is not
> currently the bottleneck in running the parallel tests for me, but it is
> among the slower T*.sh. So it might be nice at some point to split into
> several invocations for pytest, assuming multiple invocations of pytest
> can run in parallel.

There is the pytest-xdist plugin which is widely used and probably will
work fine on the notmuch testsuite (I even contributed to the plugin at
some point).  It will split the workload between however many processes
you choose, running them in parallel.
___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


Re: [PATCH] configure: Ignore more options that Fedora spec macros expect

2022-01-11 Thread David Bremner
Tomi Ollila  writes:

> On Tue, Jan 11 2022, michaeljgruber wrote:
>
>> From: Michael J Gruber 
>>
>> Signed-off-by: Michael J Gruber 
>> ---
>
> LGTM.
>
> Tomi

applied to master.

d
___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


Re: [PATCH] configure: Ignore more options that Fedora spec macros expect

2022-01-11 Thread Tomi Ollila
On Tue, Jan 11 2022, michaeljgruber wrote:

> From: Michael J Gruber 
>
> Signed-off-by: Michael J Gruber 
> ---

LGTM.

Tomi


> This allows the Fedora packager to use the standard Fedora packaging
> macros rather than a hand-tailored configure call.
>
> [Sorry for resend, my mails don't seem to get through, again.]
>
>  configure | 10 ++
>  1 file changed, 10 insertions(+)
>
> diff --git a/configure b/configure
> index e7bb7edc..9e45cbe6 100755
> --- a/configure
> +++ b/configure
> @@ -312,12 +312,22 @@ for option; do
>   true
>  elif [ "${option%%=*}" = '--host' ] ; then
>   true
> +elif [ "${option%%=*}" = '--bindir' ] ; then
> + true
> +elif [ "${option%%=*}" = '--sbindir' ] ; then
> + true
>  elif [ "${option%%=*}" = '--datadir' ] ; then
>   true
>  elif [ "${option%%=*}" = '--localstatedir' ] ; then
>   true
> +elif [ "${option%%=*}" = '--sharedstatedir' ] ; then
> + true
>  elif [ "${option%%=*}" = '--libexecdir' ] ; then
>   true
> +elif [ "${option%%=*}" = '--exec-prefix' ] ; then
> + true
> +elif [ "${option%%=*}" = '--program-prefix' ] ; then
> + true
>  elif [ "${option}" = '--disable-maintainer-mode' ] ; then
>   true
>  elif [ "${option}" = '--disable-dependency-tracking' ] ; then
> -- 
> 2.34.1.653.g3264c0b0fb
>
> ___
> notmuch mailing list -- notmuch@notmuchmail.org
> To unsubscribe send an email to notmuch-le...@notmuchmail.org
___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


Re: [PATCH] test: fix support for gpgsm in gnupg 2.3

2022-01-11 Thread David Bremner
Stig Palmquist  writes:

> gpgsm --list-keys output changed the label for fingerprints from
> "fingerprint: " to "sha[12] fpr: " breaking tests with gnupg 2.3. this
> adds support for both.
> ---
>  test/test-lib.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/test-lib.sh b/test/test-lib.sh
> index 6bc0b723..3de608f9 100644
> --- a/test/test-lib.sh
> +++ b/test/test-lib.sh
> @@ -145,7 +145,7 @@ add_gpgsm_home () {
>  mkdir -p -m 0700 "$GNUPGHOME"
>  gpgsm --batch --no-tty --no-common-certs-import --pinentry-mode=loopback 
> --passphrase-fd 3 \
> --disable-dirmngr --import  >"$GNUPGHOME"/import.log 2>&1 3<<<'' 
> <$NOTMUCH_SRCDIR/test/smime/0xE0972A47.p12
> -fpr=$(gpgsm --batch --list-key test_su...@notmuchmail.org | sed -n 
> 's/.*fingerprint: //p')
> +fpr=$(gpgsm --batch --list-key test_su...@notmuchmail.org | sed -n 
> 's/.*\(fingerprint\|sha1 fpr\): //p')

Thanks for the patch. It occurs to me that we should probably use
--with-colons to avoid future issues like this. It looks like we wabt
the row labelled "fpr"; I assume this has not changed with 2.3?

d
___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


[PATCH] test: fix support for gpgsm in gnupg 2.3

2022-01-11 Thread Stig Palmquist
gpgsm --list-keys output changed the label for fingerprints from
"fingerprint: " to "sha[12] fpr: " breaking tests with gnupg 2.3. this
adds support for both.
---
 test/test-lib.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/test-lib.sh b/test/test-lib.sh
index 6bc0b723..3de608f9 100644
--- a/test/test-lib.sh
+++ b/test/test-lib.sh
@@ -145,7 +145,7 @@ add_gpgsm_home () {
 mkdir -p -m 0700 "$GNUPGHOME"
 gpgsm --batch --no-tty --no-common-certs-import --pinentry-mode=loopback 
--passphrase-fd 3 \
  --disable-dirmngr --import  >"$GNUPGHOME"/import.log 2>&1 3<<<'' 
<$NOTMUCH_SRCDIR/test/smime/0xE0972A47.p12
-fpr=$(gpgsm --batch --list-key test_su...@notmuchmail.org | sed -n 
's/.*fingerprint: //p')
+fpr=$(gpgsm --batch --list-key test_su...@notmuchmail.org | sed -n 
's/.*\(fingerprint\|sha1 fpr\): //p')
 echo "$fpr S relax" >> "$GNUPGHOME/trustlist.txt"
 gpgsm --quiet --batch --no-tty --no-common-certs-import --disable-dirmngr 
--import < $NOTMUCH_SRCDIR/test/smime/ca.crt
 echo "4D:E0:FF:63:C0:E9:EC:01:29:11:C8:7A:EE:DA:3A:9A:7F:6E:C1:0D S" >> 
"$GNUPGHOME/trustlist.txt"
-- 
2.34.1


___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


[PATCH] configure: Ignore more options that Fedora spec macros expect

2022-01-11 Thread michaeljgruber+grubix+git
From: Michael J Gruber 

Signed-off-by: Michael J Gruber 
---
This allows the Fedora packager to use the standard Fedora packaging
macros rather than a hand-tailored configure call.

[Sorry for resend, my mails don't seem to get through, again.]

 configure | 10 ++
 1 file changed, 10 insertions(+)

diff --git a/configure b/configure
index e7bb7edc..9e45cbe6 100755
--- a/configure
+++ b/configure
@@ -312,12 +312,22 @@ for option; do
true
 elif [ "${option%%=*}" = '--host' ] ; then
true
+elif [ "${option%%=*}" = '--bindir' ] ; then
+   true
+elif [ "${option%%=*}" = '--sbindir' ] ; then
+   true
 elif [ "${option%%=*}" = '--datadir' ] ; then
true
 elif [ "${option%%=*}" = '--localstatedir' ] ; then
true
+elif [ "${option%%=*}" = '--sharedstatedir' ] ; then
+   true
 elif [ "${option%%=*}" = '--libexecdir' ] ; then
true
+elif [ "${option%%=*}" = '--exec-prefix' ] ; then
+   true
+elif [ "${option%%=*}" = '--program-prefix' ] ; then
+   true
 elif [ "${option}" = '--disable-maintainer-mode' ] ; then
true
 elif [ "${option}" = '--disable-dependency-tracking' ] ; then
-- 
2.34.1.653.g3264c0b0fb

___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org