Re: Regex negative lookahead failed

2021-04-24 Thread Michael J Gruber
David Bremner venit, vidit, dixit 2021-04-24 12:57:17: > Erwan Hingant writes: > > > Hello, > > > >  I have some troubles with regex negative lookahead. When searching > > in > > all directories but one (say inbox), I do the following query: > > > > > notmuch search folder:"/^(?!inbox)/" > >

Re: Regex negative lookahead failed

2021-04-24 Thread David Bremner
Erwan Hingant writes: > Hello, > >  I have some troubles with regex negative lookahead. When searching > in > all directories but one (say inbox), I do the following query: > > > notmuch search folder:"/^(?!inbox)/" > As far as I know, lookaheads are not supported by POSIX regex, which is

Re: [PATCH v4] compat: rename {,notmuch_}canonicalize_file_name

2021-04-24 Thread David Bremner
Đoàn Trần Công Danh writes: > > Let's move our implementation into a util library. > > Helped-by: Tomi Ollila > Signed-off-by: Đoàn Trần Công Danh I have applied this version to master. It looks a bit odd to have SPDX-License-Identifiers in just these two files, but I guess it is harmless. I

[PATCH] test: check for GNU parallel with --version options

2021-04-24 Thread Tobias Backer Dirks
The lastest versions of GNU parallel no longer make mention of GNU within their help output. This causes the test script to mistakenly use the moreutils parallel execution. In order to fix this, while maintaining compatibility with previous versions of GNU parallel, --version should be used.

[PATCH] test: add regression test for large number of deletions

2021-04-24 Thread David Bremner
Prior to 9ad19e4454f there was an unhandled Xapian exception when reindexing after a large number of deletes. This test was used for bisection, and will subsequently serve as a regression test. --- It turns out my original assessment of the situation was correct. Whatever the problem was, the

Re: [PATCH] test: check for GNU parallel with --version options

2021-04-24 Thread David Bremner
Tobias Backer Dirks writes: > The lastest versions of GNU parallel no longer make mention of GNU > within their help output. This causes the test script to mistakenly use > the moreutils parallel execution. In order to fix this, while > maintaining compatibility with previous versions of GNU

[PATCH] perf-test: add test for reindexing after removal/renaming msgs

2021-04-24 Thread David Bremner
At this point it is a bit tricky to measure the performance increase from the new message deletion code, since the same commit (9ad19e4) that improved the performance also seems to have fixed a bug with an uncaught Xapian exception triggered by this test. --- performance-test/T03-reindex.sh | 28

Re: Release process for 0.32

2021-04-24 Thread David Bremner
David Bremner writes: > Now that the config changes have landed, it is time to consider a > release. I have the following schedule in mind: > > April 24: Feature Freeze. > > May 1: Release. > I have pushed 0.32~rc1 [0] to - git (tag 0.32_rc1) -

Re: Regex negative lookahead failed

2021-04-24 Thread Erwan Hingant
Hello, Thanks a lot, this makes all clear and notmuch-search-terms(7) point the regex POSIX version you mention. My mistake. By the way, you completely solve my problem that of course no need regular expression has I thought... Bests, Erwan. David Bremner writes: > Erwan Hingant writes:

Re: [PATCH] perf-test: add test for reindexing after removal/renaming msgs

2021-04-24 Thread David Bremner
David Bremner writes: > At this point it is a bit tricky to measure the performance increase > from the new message deletion code, since the same commit (9ad19e4) > that improved the performance also seems to have fixed a bug with an > uncaught Xapian exception triggered by this test. Actually

Re: Regex negative lookahead failed

2021-04-24 Thread David Bremner
Michael J Gruber writes: > But, assertions are really for the case where you want to base a match > result (true/false) on parts of the expression that you do not want to > be part of the match itself. As far as I understand, matching in xapian > is solely about the true/false result, so I'm