"Detected x file renames" for copies

2024-01-31 Thread Michael J Gruber
Hi there, I noticed the following: - have file f indexed by notmuch - copy message file to a different folder using `notmuch insert --folder=Y < f` - run notmuch new This results in "Detected x file renames" if you do that for x files. I have not checked whether this depends on folder names or

Re: BUG: Python's Message.header fails for empty headers

2024-01-09 Thread Michael J Gruber
Am Di., 9. Jan. 2024 um 13:38 Uhr schrieb David Bremner : > > Michael J Gruber writes: > > >> > >> I agree the bindings documentation does not make much sense. I suspect > >> that the bindings should follow the underlying library and return "" if &g

Re: BUG: Python's Message.header fails for empty headers

2024-01-09 Thread Michael J Gruber
Am Di., 9. Jan. 2024 um 00:09 Uhr schrieb David Bremner : > > Vojtěch Káně writes: > > > At first, this sounds reasonable: the subject is empty, so it is > > effectively missing. That would indicate a bug in Lieer itself and would > > be fixed by a try-catch block. Notmuch's source for

Re: Difficulty understanding maildir.synchronize_flags behavior

2023-12-12 Thread Michael J Gruber
Am Di., 12. Dez. 2023 um 10:58 Uhr schrieb Andrew Todd : > Hello, > > I'm trying to set up notmuch, and I think that the behavior I want > should be possible (and is desirable), but I can't seem to make it work. > I also can't find any reference online suggesting that it *shouldn't* > work, and I

Re: Advanced search with wildcard using notmuch for mutt

2023-11-30 Thread Michael J Gruber
Am Do., 30. Nov. 2023 um 12:37 Uhr schrieb io : > > > > i have an html email with this sentence 'xycfe11cg64d_2501034012' within > the body of the message. > no result found when i search for '2501034012' > i have even tried using '*2501034012*' (wildcard) > > Notmuch doesn't support regular

Re: [PATCH 0/4] test: T380 rework

2023-11-24 Thread Michael J Gruber
So, with the key-value pairs sorted by both, I resumed testing for Python 3.1.13 and encountered failing T380 which gave me some a deja-vue due to its confusing messages: ``` T380-atomicity: Testing atomicity cat: outcount: No such file or directory

Re: T610 failing on Fedora rawhide

2023-11-24 Thread Michael J Gruber
Am Fr., 24. Nov. 2023 um 14:57 Uhr schrieb David Bremner : > Michael J Gruber writes: > > > Hi there, > > > > during my first tests for Python 3.13 (hooray...) I noticed that some > tests > > in T610 started to fail independent of that. It seems that with not

T610 failing on Fedora rawhide

2023-11-23 Thread Michael J Gruber
Hi there, during my first tests for Python 3.13 (hooray...) I noticed that some tests in T610 started to fail independent of that. It seems that with notmuch 0.38.1 on current Fedora rawhide, `notmuch_message_get_properties()` returns properties in a different order, while the tests expect a

Re: [PATCH 26/27] emacs: avoid binding unnamed commands in keymaps

2023-10-28 Thread Michael J Gruber
Am Sa., 28. Okt. 2023 um 06:22 Uhr schrieb Ryan Tate : > > Jonas Bernoulli writes: > > > - -(defun notmuch-tree-close-message-pane-and (func) - "Close > > message pane and execute FUNC. > > I am confused why a function used in config files and documented > on the notmuch website (to this

Re: v2 Image toggle fixes for emacs 29.1

2023-10-01 Thread Michael J Gruber
Am So., 1. Okt. 2023 um 13:13 Uhr schrieb David Bremner : > > I have applied this series to release and master (and uploaded a > pre-release for 0.38.1) Is "pre" the new "rc", or how is this supposed to sort? Michael ___ notmuch mailing list --

Re: Fixed Message-ID trouble

2023-09-25 Thread Michael J Gruber
Am Mo., 25. Sept. 2023 um 12:53 Uhr schrieb Teemu Likonen : > > * 2023-09-25 11:54:07+0300, Teemu Likonen wrote: > > > Some person on debian-user mailing list seems to be sending messages > > with fixed Message-ID field: the same ID in different messages. In > > Notmuch it is creating trouble

Re: notmuch breaks on \. in config file with upgrade from glib2 2.76.1 to 2.76.5

2023-09-06 Thread Michael J Gruber
Hi there [snip] > Last night, I filed > https://bugzilla.redhat.com/show_bug.cgi?id=2237562. Later, I found > this about glib 2.77 introducing regressions: > https://bugzilla.redhat.com/show_bug.cgi?id=2225257; looks like Fedora > backported enough of that into 2.76.5 to cause similar issues in

Re: T460: new sporadic failures with emacs 29

2023-09-01 Thread Michael J Gruber
Am Do., 31. Aug. 2023 um 17:17 Uhr schrieb David Bremner : > > Michael J Gruber writes: > > > > > I still get those issues. OTOH, skipping T460.14 did not show any > > adverse side effects. So I'll do that for emacs29. > > I might be nice to mark some tes

Re: T460: new sporadic failures with emacs 29

2023-08-31 Thread Michael J Gruber
Am Do., 31. Aug. 2023 um 15:16 Uhr schrieb David Bremner : > > Michael J Gruber writes: > > > Am Sa., 26. Aug. 2023 um 16:41 Uhr schrieb David Bremner > > : > >> > >> Michael J Gruber writes: > >> > >> > > >> > I tried t

Re: T460: new sporadic failures with emacs 29

2023-08-26 Thread Michael J Gruber
Am Sa., 26. Aug. 2023 um 16:41 Uhr schrieb David Bremner : > > Michael J Gruber writes: > > > > > I tried the current 0.38rc1 on COPR, and unfortunately I get the same > > T460 failure (fedora-eln-aarch64 and fedora-rawhide-x86_64 this time, > > out of 35 buildr

Re: T460: new sporadic failures with emacs 29

2023-08-26 Thread Michael J Gruber
Am Sa., 26. Aug. 2023 um 00:28 Uhr schrieb David Bremner : > > Michael J Gruber writes: > > > It took more runs to get some fails now, and archs vary, so I still > > think its a time out. And no way to get it locally so far. > > I can duplicate it locally about once ev

Re: T460: new sporadic failures with emacs 29

2023-08-24 Thread Michael J Gruber
Am Do., 24. Aug. 2023 um 17:10 Uhr schrieb David Bremner : > > David Bremner writes: > > > I just saw this when running in debian's "sbuild" isolated build > > environment. So my current guess is that this has to do with HOME > > pointing somewhere nonexistent. Is that also the case in COPR? > >

Re: T460: new sporadic failures with emacs 29

2023-08-24 Thread Michael J Gruber
Am Do., 24. Aug. 2023 um 16:01 Uhr schrieb David Bremner : > > David Bremner writes: > > > Michael J Gruber writes: > > > >> -notmuch@notmuchmail.org > >> -http://notmuchmail.org/mailman/listinfo/notmuch > >> *ERROR*: Opening output file: Permissi

T460: new sporadic failures with emacs 29

2023-08-24 Thread Michael J Gruber
Hi there, I'm sorry to report that I'm getting new sporadic (some archs sometimes) failures on Fedora 39+ only, i.e. with emacs 29, on COPR. ``` T460-emacs-tree: Testing emacs tree view interface PASS Basic notmuch-tree view in emacs ... PASS Tree view of a single thread (from show) FAIL

Re: plans for releasing 0.38

2023-08-20 Thread Michael J Gruber
Am So., 20. Aug. 2023 um 19:54 Uhr schrieb David Bremner : > > > I just posted the last patch needed to unbreak the test suite for me > with Emac 29.x and glib 2.77.x. As soon as these (or something similar) > are applied to master, I plan to feature freeze 0.38. There is less new > stuff than a

Re: [PATCH 1/2] test: showcase thread-unsafe s-expression query parser

2023-07-23 Thread Michael J Gruber
Am Sa., 22. Juli 2023 um 20:48 Uhr schrieb David Bremner : > > Kevin Boulain writes: > > > The test fails quite reliably for me: > > T810-tsan: Testing run code with TSan enabled against the library > >PASS create > >PASS query > >FAIL sexp query > > I can't get this test to

Re: T040 fails on Fedora 39 rebuild

2023-07-20 Thread Michael J Gruber
Am Fr., 21. Juli 2023 um 03:40 Uhr schrieb David Bremner : > > Michael J Gruber writes: > > > I'm not sure we should or can work around 2.77.0's behaviour. (It > > might get triggered by the add/remove trick.) > > Will FC39 definitely ship with 2.77.0 ? There is stil

Re: T040 fails on Fedora 39 rebuild

2023-07-20 Thread Michael J Gruber
Am Do., 20. Juli 2023 um 20:54 Uhr schrieb Michael J Gruber : > > Hi there, > > funny failures again during the mass rebuild for the (not quite yet) > upcoming Fedora rebuild. This test passed just a few weeks ago, so the > failure is related to something else changing.

T040 fails on Fedora 39 rebuild

2023-07-20 Thread Michael J Gruber
Hi there, funny failures again during the mass rebuild for the (not quite yet) upcoming Fedora rebuild. This test passed just a few weeks ago, so the failure is related to something else changing. But still it shows that the order in which `notmuch config` produces new config can depend on

Re: [PATCH] python: adjust legacy bindings to py 3.12

2023-07-09 Thread Michael J Gruber
Am So., 9. Juli 2023 um 16:57 Uhr schrieb David Bremner : > > michaeljgruber+grubix+...@gmail.com writes: > > > From: Michael J Gruber > > > > Py 3.12 finally pulled the plug on the `SafeConfigParser` class which > > has been deprecated since py 3.2. > > A

Re: "notmuch compact" questions

2023-06-18 Thread Michael J Gruber
Am Sa., 17. Juni 2023 um 23:49 Uhr schrieb Andy Smith : > > Hi, > > I'm using v0.31.4 on Debian 11. I have ~3.9 million messages in my > archive and the notmuch database currently takes up 85GiB (though > actually "only" 51GiB due to btrfs zstd:1 compression). Wow ;) > I did remove a few hundred

Re: T350-crypto T357-index-decryption: possible race condition?

2023-06-03 Thread Michael J Gruber
Am Di., 30. Mai 2023 um 18:57 Uhr schrieb Michael J Gruber : > > Am Sa., 27. Mai 2023 um 14:31 Uhr schrieb David Bremner : > > > > Michael J Gruber writes: > > > > > > Are all gpg related tests emacs based? Either gpg or emacs is the red > > > herring

Re: T350-crypto T357-index-decryption: possible race condition?

2023-05-30 Thread Michael J Gruber
Am Sa., 27. Mai 2023 um 14:31 Uhr schrieb David Bremner : > > Michael J Gruber writes: > > > > Are all gpg related tests emacs based? Either gpg or emacs is the red > > herring here, or both ... > > The issue (at least on rawhide) seems to be the interaction between

Re: T350-crypto T357-index-decryption: possible race condition?

2023-05-30 Thread Michael J Gruber
Am Sa., 27. Mai 2023 um 14:59 Uhr schrieb David Bremner : > > David Bremner writes: > > > > > I'm not sure if this is the main issue, but in commit bf8aa34324cc91a > > that key was replaced by an ed255519 key > > 9A3AFE6C60065A148FD4B58A7E6ABE924645CC60 > > > > Did the fedora sources get out of

Re: T350-crypto T357-index-decryption: possible race condition?

2023-05-29 Thread Michael J Gruber
David Bremner venit, vidit, dixit 2023-05-12 21:17:45: > Michael J Gruber writes: > > > oh well, attachments ... > > > > Can you encrypt to the key 6D92612D94E46381 interactively using an > approriately simplified version of that command? Took me a while, sorry. I

Re: T350-crypto T357-index-decryption: possible race condition?

2023-05-25 Thread Michael J Gruber
[now from the subscribed address, sorry] David Bremner venit, vidit, dixit 2023-05-12 21:17:45: > Michael J Gruber writes: > > > oh well, attachments ... > > > > Can you encrypt to the key 6D92612D94E46381 interactively using an > approriately simplified versio

Re: T350-crypto T357-index-decryption: possible race condition?

2023-05-12 Thread Michael J Gruber
oh well, attachments ... Am Fr., 12. Mai 2023 um 19:42 Uhr schrieb Michael J Gruber : > > Am Do., 11. Mai 2023 um 22:49 Uhr schrieb David Bremner : > > > > Michael J Gruber writes: > > > ... > > > T350-crypto: Testing PGP/MIME signature verification and decr

Re: T350-crypto T357-index-decryption: possible race condition?

2023-05-12 Thread Michael J Gruber
Am Do., 11. Mai 2023 um 22:49 Uhr schrieb David Bremner : > > Michael J Gruber writes: > ... > > T350-crypto: Testing PGP/MIME signature verification and decryption ... > > PASS signature verification with signer key unavailable > > ``` > > There the suite &quo

T350-crypto T357-index-decryption: possible race condition?

2023-05-11 Thread Michael J Gruber
Hi there, my regular notmuch test builds recently started to fail, more concretely: the test suite fails because some subtests are KILLed. Building notmuch 0.37 with my usual spec-file as a rawhide-mock build (a local chroot for the development "version" of Fedora which will become Fedora 39) I

Re: Recommended method to pair Notmuch with Neovim?

2023-04-19 Thread Michael J Gruber
James Cook venit, vidit, dixit 2023-04-19 04:19:25: > On Wed, Apr 19, 2023 at 12:47:24AM +0200, Ralph Seichter wrote: > > Jon Fineman wrote: > > > > > [database] > > > path=/home/jjf/Maildir > > > > > > > and is your Notmuch DB directory /full/path/to/your/Maildir/.notmuch , > > > > i.e. nested

Re: inbox-update: new competition of notmuch-lore

2023-04-17 Thread Michael J Gruber
Hi Felipe > I'm moving from mbsync to public-inbox and I find there aren't many tools to > make it work with notmuch. Looking at that, too. > I gave a try to notmuch-lore [1] but I found it too slow and had a couple of > issues. > > So I wrote my own script to convert public-inbox mailing lists

Re: Data loss

2023-04-04 Thread Michael J Gruber
Am Di., 4. Apr. 2023 um 00:54 Uhr schrieb Carl Worth : > > Ouch. > Yes, we're feeling with you, Fulvio :| > It's really unfortunate if notmuch-mutt makes it that easy to throw away > your email. > > That sounds like a nasty bug that should be fixed in that program. To be fair to notmuch-mutt:

Re: Data loss

2023-04-03 Thread Michael J Gruber
Am Mo., 3. Apr. 2023 um 20:17 Uhr schrieb Fulvio Pizzigoni : > > > > > > > Hi Carl e thanks for your prompt answer. > > As you suggested, I add notmuch@notmuchmail.org email as well. > > This is what I did: > fulvio@linux:~$ notmuch setup > Your full name [fulvio]: > Your primary email address [my

Re: Reimagining notmuch-git/nmbug

2023-03-29 Thread Michael J Gruber
Am Mi., 29. März 2023 um 10:41 Uhr schrieb Felipe Contreras : > > Hi, > > I noticed you promoted notmuch-git as a user tool to toy around with it. > > Very quickly I realized that most of what it does is something I've > been working on for at least 10 years: making git work with other > tools. >

Re: [PATCH] emacs/tree: use two argument form of setq-local

2023-02-21 Thread Michael J Gruber
Am Di., 21. Feb. 2023 um 12:49 Uhr schrieb David Bremner : > > Apparently the macro setq-local only takes two arguments in Emacs 26.1 > --- > emacs/notmuch-tree.el | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/emacs/notmuch-tree.el b/emacs/notmuch-tree.el >

Test breakage on EPEL 8 with tree-outline-mode

2023-02-20 Thread Michael J Gruber
Hi there, the test suite started to fail on EPEL 8 with the new tree-outline-mode: ``` EMACS emacs/notmuch-tree.elc In notmuch-tree-mode: emacs/notmuch-tree.el:1088:9:Warning: reference to free variable `notmuch-tree-outline-enabled' In notmuch-tree-outline--set-visibility:

Re: Proof of concept for counting messages in thread

2023-02-18 Thread Michael J Gruber
Am Di., 14. Feb. 2023 um 02:47 Uhr schrieb David Bremner : > > Michael J Gruber writes: > > > That is really weird: > > ``` > > xapian-delve -t G00021229 . > > Posting List for term 'G00021229' (termfreq 115, collfreq 0, > > wdf_max 0):

Re: Proof of concept for counting messages in thread

2023-02-13 Thread Michael J Gruber
Am Mo., 13. Feb. 2023 um 21:23 Uhr schrieb David Bremner : > > Michael J Gruber writes: > > > > It has 5, as confirmed by the search output and that of `notmuch > > count`. But it is matched by `count 115`. > > `xapian-check` is happy. (There used to be some is

Re: Proof of concept for counting messages in thread

2023-02-13 Thread Michael J Gruber
Am Mo., 13. Feb. 2023 um 17:32 Uhr schrieb David Bremner : > > Michael J Gruber writes: > > > I am getting a few surprising matches, e.g. > > ``` > > notmuch search --query=sexp '(thread (count 115)))' > > thread:000000021229 2021-05-17 [5/5] Michael J Grub

Re: Proof of concept for counting messages in thread

2023-02-13 Thread Michael J Gruber
115)))' thread:00021229 2021-05-17 [5/5] Michael J Gruber ... redacted notmuch count --exclude=false thread:00021229 5 ``` It could be some database issues, of course. Or me misunderstanding something :) Patch 1/2 is crlf garbled, by the way. Applies cleanly after removing the extr

Re: ## error when running notmuch from emacsclient, but not from standalone emacs instance

2023-01-18 Thread Michael J Gruber
Am Mi., 18. Jan. 2023 um 15:40 Uhr schrieb David Bremner : > > ricardomart...@riseup.net writes: > > > > > When I try to run notmuch from the emacsclient instance (M-x notmuch > > ), I get an error > > "notmuch count --batch failed > > Please check that the notmuch CLI is new enough to support

Re: Test failure in Ubuntu 22.04 and 22.10 (new test)

2022-10-06 Thread Michael J Gruber
Am Do., 6. Okt. 2022 um 18:34 Uhr schrieb David Bremner : > > Michael J Gruber writes: > > > > > Yes, lto-wrapper calls make. > > > > Are we compiling test functions on the fly during the test? In that > > case we need to make sure that each test dep

Re: [PATCH] test: replace aging OpenPGP key used in the test suite

2022-09-22 Thread Michael J Gruber
Am Do., 22. Sept. 2022 um 12:14 Uhr schrieb Justus Winter : > > Michael J Gruber writes: > > > Am Do., 22. Sept. 2022 um 10:47 Uhr schrieb Justus Winter > > : > >> > >> This replaces the old OpenPGPv4 key that is used in the test suite > >> with

Re: [PATCH] test: replace aging OpenPGP key used in the test suite

2022-09-22 Thread Michael J Gruber
Am Do., 22. Sept. 2022 um 10:47 Uhr schrieb Justus Winter : > > This replaces the old OpenPGPv4 key that is used in the test suite > with a more modern OpenPGPv4 key. All cryptographic artifacts in the Both v4? Only one key file is named v4. > @@ -6,7 +6,7 @@ Message-ID: > MIME-Version: 1.0

Re: best practices for keeping a notmuch tag in sync with maildir folder

2022-09-05 Thread Michael J Gruber
Am Mo., 5. Sept. 2022 um 13:33 Uhr schrieb David Bremner : > > Bence Ferdinandy writes: > > > are there any best practices for keeping notmuch tags synced with an imap > > folder? It seems to me that simply moving a mail file in the shell from one > > folder to the other is not something that

Re: notmuch release 0.37 now available

2022-08-22 Thread Michael J Gruber
Thanks for another fine release, and for being a pleasant upstream to work with! For Fedora Linux/RHEL/etc folks: notmuch 0.37 is available in rawhide and f37 (branched), as well as in updates-testing for the current release f36, all with sfsexp support. It will not go to any EPEL branch (per

Re: nmweb HTML injection

2022-08-22 Thread Michael J Gruber
Am Mo., 22. Aug. 2022 um 09:22 Uhr schrieb Jakub Wilk : > > See: https://nmbug.notmuchmail.org/nmweb/search/markup%20where%20appropriate > > and from the mail subject was dumped without escaping into HTML. > Interesting :) The body is htmlescape()ed, but the subject header is used as is. I

Re: [PATCH 4/4] CLI/git: replace calls to notmuch-search with database access

2022-07-09 Thread Michael J Gruber
Am Do., 7. Juli 2022 um 17:59 Uhr schrieb David Bremner : > > Tomi Ollila writes: > > > On Sun, Jul 03 2022, David Bremner wrote: > > > > I've trying to think if there were a way to somehow run only one notmuch > > command instead of notmuch search on all maeby-deleted files -- or > >

Re: [PATCH] doc: replace symlink with duplicate page for nmbug

2022-06-30 Thread Michael J Gruber
Am Do., 30. Juni 2022 um 13:20 Uhr schrieb David Bremner : > > This automatically propagates to the info version of the pages, which > saves having maintain two sets of symlinks. > --- > > What do you think about this as an alternative approach? It seems > slightly more maintainable to me,

Re: sexp and strings

2022-06-13 Thread Michael J Gruber
Am Mo., 13. Juni 2022 um 11:31 Uhr schrieb erik colson : > > Michael J Gruber writes: > > > That search works without the macro, but not as a macro: notmuch > > computes an empty `Query()` for this (as per `NOTMUCH_DEBUG_QUERY=1`). > > I'm not sure whether this

Re: sexp and strings

2022-06-13 Thread Michael J Gruber
Am Mo., 13. Juni 2022 um 01:20 Uhr schrieb erik colson : > > Hi, > > I would like to define a squery in my notmuch configuration which would > ease a query I often use. The query is > > tag:/d/ > > where d are decimal numbers. > Now I would like to shorten this to > > D d > >

Re: contact instead of (to or from)

2022-06-10 Thread Michael J Gruber
[rearranged for bottom post style] Am Fr., 10. Juni 2022 um 05:51 Uhr schrieb 杨令 : > On Fri 2022-06-10 00:35:12, erik colson wrote: > > Date: Fri, 10 Jun 2022 00:35:12 +0200 > > From: erik colson > > To: notmuch@notmuchmail.org > > Subject: contact instead of (to or from) > > > > Hi, > >> > >

Re: [PATCH v2 2/2] lib: add sexp: prefix to Xapian (infix) query parser.

2022-04-15 Thread Michael J Gruber
This breaks builds if you don't have sfexp, unfortunately. (And since there is no sfsexp release with the recent fixes, there is no sfsexp in Fedora, for example.) Not declaring _notmuch_sexp_string_to_xapian_query() twice in lib/database-private.h (once unconditionally, once depending on

Re: [PATCH 2/2] test/smime: fix signature verification test with newer gmime.

2022-04-11 Thread Michael J Gruber
Am Mo., 11. Apr. 2022 um 02:36 Uhr schrieb Daniel Kahn Gillmor < d...@debian.org>: > Thanks, Bremner! > > This series looks reasonable to me. nice clever hack to reuse the > gmime embedded .c source for the test. > Just so that others don't have to be wondering, too: notmuch does not embed

Re: [PATCH 2/4] test: due not pass T380.1 for the wrong reasons

2022-02-20 Thread Michael J Gruber
David Bremner venit, vidit, dixit 2022-02-20 00:02:40: > Tomi Ollila writes: > > > > > Wat? afaik echo is builtin in every modern bourne shell derivative... > > > > (I tested: > > $ bash -c 'builtin echo foo' > > foo > > $ bash -c 'export PATH=/tmp; echo foo; ls' > > foo > > bash: ls:

Re: [PATCH] test: allow to use --full-sync

2022-02-16 Thread Michael J Gruber
David Bremner venit, vidit, dixit 2022-02-16 14:04:17: > Michael J Gruber writes: > > > Some build infrastructure appears to habe problems with mtime/stat, > > have Huh, this looks as if my mother tongue slipped in, while it's just key proximity. > > leading to sp

[PATCH] test: allow to use --full-sync

2022-02-16 Thread Michael J Gruber
Some build infrastructure appears to habe problems with mtime/stat, leading to spurious failures (noticed on s390x and aarch64 with epel8). Allow the test suite to be run with --full-sync so that release builds can use the test suite while avoiding spurious failures. Signed-off-by: Michael J

Re: Test suite timing issues?

2022-02-15 Thread Michael J Gruber
> > > BTW, I have so far run the test suite 68 times in a row without failures > > > on a Debian s390x host. The file system is ext4, mounted relatime. It > > > would be interesting to know what file system is yielding the failures > > > Michael is seeing. > ... > So I'll try out a bit more, and

Re: Test suite timing issues?

2022-02-15 Thread Michael J Gruber
Tomi Ollila venit, vidit, dixit 2022-02-14 23:53:54: > On Mon, Feb 14 2022, David Bremner wrote: > > > Tomi Ollila writes: > > > >> > >> Looked notmuch-new.c -- time_t (seconds since epoch) is used as timestamp > >> comparisons (which would indicate the subsecond resolution most fs' provide > >>

Re: Test suite timing issues?

2022-02-12 Thread Michael J Gruber
Am Sa., 12. Feb. 2022 um 22:10 Uhr schrieb David Bremner : > Tomi Ollila writes: > > > > > Does such a change hide "buggy" functionality ? > > We mostly don't use add_message, call notmuch new via NOTMUCH_NEW in > T050-new.sh. So I think it would mostly not hide bugs in notmuch > new. OTOH, I'm

Re: Test suite timing issues?

2022-02-12 Thread Michael J Gruber
Am Sa., 12. Feb. 2022 um 21:45 Uhr schrieb Tomi Ollila : > On Sat, Feb 12 2022, David Bremner wrote: > > > Tomi Ollila writes: > > > >> On Sat, Feb 12 2022, Michael J. Gruber wrote: > >> > >> Only thing that came into mind are directory timestamps..

Re: Test suite timing issues?

2022-02-12 Thread Michael J Gruber
David Bremner venit, vidit, dixit 2022-02-12 01:03:00: > Michael J Gruber writes: > > > Hi there, > > > > I'm trying to package notmuch for Redhat's enterprise linux and clones > > (EPEL, extra packages for enterprise linux). > > > > This look

Re: Test suite timing issues?

2022-02-11 Thread Michael J Gruber
Michael J Gruber venit, vidit, dixit 2022-02-11 14:55:32: > Hi there, > > I'm trying to package notmuch for Redhat's enterprise linux and clones > (EPEL, extra packages for enterprise linux). > > This looks mostly fine, including the tests, except for intermittent > fai

Test suite timing issues?

2022-02-11 Thread Michael J Gruber
Hi there, I'm trying to package notmuch for Redhat's enterprise linux and clones (EPEL, extra packages for enterprise linux). This looks mostly fine, including the tests, except for intermittent failures on epel-8-s390x. They look like the below, or in tests following those, and apparantly all

Re: Python binding SIGABRT/SIGSEGV

2022-02-11 Thread Michael J Gruber
Austin Lund venit, vidit, dixit 2022-02-10 23:21:58: > On Thu, Feb 10, 2022 at 01:12:47PM +0100, Michael J Gruber wrote: > > Austin Lund venit, vidit, dixit 2022-02-10 06:56:12: > > > I'm clearly doing this python code wrong by not using the iterator > > > corre

Re: Python binding SIGABRT/SIGSEGV

2022-02-10 Thread Michael J Gruber
Austin Lund venit, vidit, dixit 2022-02-10 06:56:12: > I'm clearly doing this python code wrong by not using the iterator correctly: > > > import notmuch2 > > > > d = notmuch2.Database() > > m = list(d.messages("since:today")) > > p = m[0].path > > print(p) > > But I seem to be getting a

Re: [PATCH 2/4] test: due not pass T380.1 for the wrong reasons

2022-02-10 Thread Michael J Gruber
David Bremner venit, vidit, dixit 2022-02-09 23:12:24: > Michael J Gruber writes: > > > If gdb is missing then some files are never written to so that the > > comparisons of non-existing files succeeds for the wrong reason, > > claiming that `notmch new` is idempotent w

Re: [PATCH 4/4] test: set up the outcount file for T380.1

2022-02-10 Thread Michael J Gruber
Tomi Ollila venit, vidit, dixit 2022-02-09 21:50:35: > On Wed, Feb 09 2022, Michael J. Gruber wrote: > > > If gdb is present but for some reason `atomicity.py` fails to write to > > the output file then the test fails with some ugly bash errors in the > > wrong plac

Re: [PATCH 2/4] test: due not pass T380.1 for the wrong reasons

2022-02-10 Thread Michael J Gruber
Tomi Ollila venit, vidit, dixit 2022-02-09 21:49:18: > On Wed, Feb 09 2022, Michael J. Gruber wrote: > > > If gdb is missing then some files are never written to so that the > > comparisons of non-existing files succeeds for the wrong reason, > > claiming that `notmch

Re: [PATCH 1/4] test: correct comparison order in T380

2022-02-10 Thread Michael J Gruber
Tomi Ollila venit, vidit, dixit 2022-02-09 21:47:46: > On Wed, Feb 09 2022, Michael J. Gruber wrote: > > > Specifying test comparisons as "expected actual" gives a better readable > > diff since the "-" indicates missing, "+" additional items comp

[PATCH 1/4] test: correct comparison order in T380

2022-02-09 Thread Michael J Gruber
Specifying test comparisons as "expected actual" gives a better readable diff since the "-" indicates missing, "+" additional items compared to the expectations. Signed-off-by: Michael J Gruber --- test/T380-atomicity.sh | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH 3/4] test: reword T380.2 to be clearer

2022-02-09 Thread Michael J Gruber
he actual expectation and what we test for. The failing test line still gives the actual number. Signed-off-by: Michael J Gruber --- test/T380-atomicity.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/T380-atomicity.sh b/test/T380-atomicity.sh index 7f618062..49df5c38 100755 -

[PATCH 2/4] test: due not pass T380.1 for the wrong reasons

2022-02-09 Thread Michael J Gruber
. Signed-off-by: Michael J Gruber --- test/T380-atomicity.sh | 4 1 file changed, 4 insertions(+) diff --git a/test/T380-atomicity.sh b/test/T380-atomicity.sh index a6f1e037..7f618062 100755 --- a/test/T380-atomicity.sh +++ b/test/T380-atomicity.sh @@ -90,6 +90,10

[PATCH 4/4] test: set up the outcount file for T380.1

2022-02-09 Thread Michael J Gruber
a clearer reason. Background: We noticed this with arch armhfp emulated on x86_64 in Fedora's COPR test build environment. Signed-off-by: Michael J Gruber --- test/T380-atomicity.sh | 1 + 1 file changed, 1 insertion(+) diff --git a/test/T380-atomicity.sh b/test/T380-atomicity.sh index 49df5c38

[PATCH 0/4] test: T380 rework

2022-02-09 Thread Michael J Gruber
messages longer than the patch diff (in good old git.git fashion), to make it readable and also partially-pickable if you prefer. Michael J Gruber (4): test: correct comparison order in T380 test: due not pass T380.1 for the wrong reasons test: reword T380.2 to be clearer test: set up

Re: Statistics about senders with notmuch possible?

2021-12-28 Thread Michael J Gruber
signmeup1...@cock.li venit, vidit, dixit 2021-12-27 23:57:18: > I would like to create some kind of statistics to see from which sender > I've got the most emails, having something like a TOP 10 list. > > In the end of every year I delete all my emails, which are not marked as > archive and

Re: [WIP PATCH 0/1] autodoc failed to import notmuch2

2021-10-21 Thread Michael J Gruber
David Bremner venit, vidit, dixit 2021-10-21 14:20:11: > Michael J Gruber writes: > > > Hi there > > > > During Fedora package builds I noticed that autodoc fails to import the > > notmuch2 module: > > > > ``` > > WARNING: autodoc: failed to impor

[WIP PATCH 0/1] autodoc failed to import notmuch2

2021-10-21 Thread Michael J Gruber
least with 0.33 but - being just a warning - was unnoticed. Cheers Michael Michael J Gruber (1): autodoc: fix search path for notmuch2 module doc/conf.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.33.0.1146.g7ac282475c ___

[WIP PATCH 1/1] autodoc: fix search path for notmuch2 module

2021-10-21 Thread Michael J Gruber
g the doc against. Fix the path so that the build tree module comes first. Signed-off-by: Michael J Gruber --- doc/conf.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/conf.py b/doc/conf.py index 1fbd102b..c7fd8f5a 100644 --- a/doc/conf.

Re: [RFC 0/2] add --new-tags= to notmuch-new(1)

2021-09-16 Thread Michael J Gruber
andr...@rammhold.de venit, vidit, dixit 2021-09-16 14:24:14: > On 14:03 16.09.21, Michael J Gruber wrote: > > I very much sympathize with your setup. But I think the real solution > > would be one of these options: > > > > - use a lock file to prevent your script

Re: [RFC 0/2] add --new-tags= to notmuch-new(1)

2021-09-16 Thread Michael J Gruber
andr...@rammhold.de venit, vidit, dixit 2021-09-16 12:25:15: > I've recently became a little annoyed with a race condition in my current > notmuch setup that originates from only having a single set of new tags for > all notmuch-new(1) invocations. In the past I've mentioned this a couple of >

Re: configure checks for python3-notmuch2 module

2021-07-12 Thread Michael J Gruber
David Bremner venit, vidit, dixit 2021-07-11 23:29:57: > > It turns out we need pyconfig.h to build the (new) python bindings. I > don't think this is adequately checked for by the configure script, > since I had a Fedora user report a build failure (during make install, > which I found odd)

[PATCH] test: deduplicate T590

2021-06-29 Thread Michael J Gruber
Test numbers are a concise way to communicate about tests and to remeber them. Currently, there is one pait of duplicates: T590-libconfig.sh T590-thread-breakage.sh Renumber the latter one to 592 since this keeps the alphabetic order and leaves room in between. Signed-off-by: Michael J Gruber

Re: [PATCH] lib: consider all instances of Delivered-To header

2021-06-23 Thread Michael J Gruber
Hannu Hartikainen venit, vidit, dixit 2021-06-23 12:29:06: > When using notmuch-reply and guessing the From: address from > Delivered-To headers, I had the wrong address chosen today. This was > because the messages from the notmuch list contain these headers in this > order: > > Delivered-To:

Re: Test failures with notmuch 0.32 and 0.32.1 on openSUSE

2021-05-20 Thread Michael J Gruber
Dan Čermák venit, vidit, dixit 2021-05-20 16:30:40: > Michael J Gruber writes: > > > > > So, I ran the container using buildah on Fedora 33 (without the make > > corpus_download). Again, I'm not a container guy, so I don't know what > > the influence of the

Re: Test failures with notmuch 0.32 and 0.32.1 on openSUSE

2021-05-20 Thread Michael J Gruber
David Bremner venit, vidit, dixit 2021-05-20 11:59:50: > Dan Čermák writes: > > > > > Right, I mistakenly thought this would grab the test database. Is that > > one no longer necessary? > > > > Ah, that's a different make target, which is indeed no longer necessary > (we don't do the relevant

Re: Test failures with notmuch 0.32 and 0.32.1 on openSUSE

2021-05-19 Thread Michael J Gruber
Dan Čermák venit, vidit, dixit 2021-05-19 10:16:39: > Hi list, > > notmuch 0.32 recently stopped building on openSUSE Leap and Tumbleweed > due to failures in the test suite. It now consistently fails in > T356-protected-headers: > > FAIL verify signed PKCS#7 subject (multipart-signed) signer

Re: [PATCH] test: add known broken test for duplicate thread-id terms

2021-05-17 Thread Michael J Gruber
David Bremner venit, vidit, dixit 2021-05-15 15:05:07: > According to my bijection, this bug has been present since commit > 411675a6ce in 2017. It is apparently harmless for regular use, but > does make notmuch crash when compiled with -DDEBUG_DATABASE_SANITY. If a message "belongs" to 2

Re: [PATCH] lib/notmuch_database_reopen: reload some database metada

2021-05-12 Thread Michael J Gruber
David Bremner venit, vidit, dixit 2021-05-12 14:07:33: > David Bremner writes: > > > David Bremner writes: > > > >> In some uses of reopen, new documents and threads maybe have been > >> added, and e.g. compaction may have changed the uuid. > > > > It's quite possible there is more cached data

Re: Is there a reason why the trashed flag is not synced?

2021-05-12 Thread Michael J Gruber
Reto venit, vidit, dixit 2021-05-12 07:10:13: > On Tue, May 11, 2021 at 08:48:45AM +0200, Michael J Gruber wrote: > > If you sync gmail labels to exact copies in different folders then you're > > not > > holding it right, sorry ;) > > > Gmail is not an IMAP ser

Re: [PATCH] test: change database from within pre-new hook

2021-05-12 Thread Michael J Gruber
David Bremner venit, vidit, dixit 2021-05-12 00:34:33: > Michael J Gruber writes: > > > Due to the change in the config system, notmuch keeps a notmuch database > > open when it would not do so before. Consequently, it can miss changes > > to the database which are

[PATCH] test: change database from within pre-new hook

2021-05-11 Thread Michael J Gruber
that it uses wrong assumptions about the last used doc id etc. Demonstrate this by triggering an assertion. (This new test succeeds with notmuch 0.31.4.) Signed-off-by: Michael J Gruber --- test/T400-hooks.sh | 21 + 1 file changed, 21 insertions(+) diff --git a/test/T400-hooks.sh b

Re: Possible threading issues in nm 0.32

2021-05-11 Thread Michael J Gruber
... just a guess: Could it be that a9f74aee ("CLI/new: drop the write lock to run the pre-new hook.", 2021-03-18) was not enough? notmuch_database_reopen() only reopens the xapian db but does not update other members in notmuch_database_t *notmuch, such as the last doc id and thread id. If a

Re: Possible threading issues in nm 0.32

2021-05-11 Thread Michael J Gruber
Alexander Adolf venit, vidit, dixit 2021-05-11 16:32:22: > Alexander Adolf writes: > > > Michael J Gruber writes: > > > >> [...] > >> So it seems: > >> > >> - The mis-threading happens during `notmuch new`, not with `notmuch > >>

Re: Is there a reason why the trashed flag is not synced?

2021-05-10 Thread Michael J Gruber
David Bremner venit, vidit, dixit 2021-05-10 14:04:27: > Felipe Contreras writes: > > > > > It makes sense that by default synchronize_flags=true doesn't sync the > > trash flag, but what's wrong with a new configuration > > synchronize_flags=all that does? > > > > That seems plausible to me,

  1   2   >