]> git.notmuchmail.org Git - notmuch/log
notmuch
2 years agotest: rename indexing corpus
David Bremner [Sat, 20 Aug 2022 18:50:05 +0000 (11:50 -0700)]
test: rename indexing corpus

The corpus is not really suitable for general indexing test since the
sole message is ignored (and will most likely continue to be ignored)
by notmuch-new.

2 years agolib: add field processor for lastmod: prefix
David Bremner [Sun, 14 Aug 2022 15:02:59 +0000 (12:02 -0300)]
lib: add field processor for lastmod: prefix

By sharing the existing logic used by the sexp query parser, this
allows negative lastmod revisions to be interpreted as relative to the
most recent revision.

2 years agolib: factor out lastmod range handling from sexp parser.
David Bremner [Sun, 14 Aug 2022 15:02:58 +0000 (12:02 -0300)]
lib: factor out lastmod range handling from sexp parser.

This will permit the re-use of the same logic in the infix query
parser. The location of the shared code in the infix side is for
consistency with the other shared parsing logic. It will make more
sense when a Xapian field processor is added for the lastmod prefix.

2 years agolib/sexp: provide relative lastmod queries
David Bremner [Sun, 14 Aug 2022 15:02:57 +0000 (12:02 -0300)]
lib/sexp: provide relative lastmod queries

Test the relatively trivial logic changes for the sexp query parser
first before refactoring that logic to share with the infix query
parser.

2 years agolib/open: create database path in some cases
David Bremner [Fri, 29 Jul 2022 11:31:37 +0000 (08:31 -0300)]
lib/open: create database path in some cases

There is some duplication of code here, but not all of the locations
valid to find a database make sense to create. Furthermore we nead two
passes, so the control flow in _choose_database_path would get a bit
convoluted.

2 years agolib/open: return non-SUCCESS on missing database path
David Bremner [Fri, 29 Jul 2022 11:23:11 +0000 (08:23 -0300)]
lib/open: return non-SUCCESS on missing database path

This simplifies the logic of creating the directory path when it doesn't
exist.

2 years agolib/open: refactor call to mkdir into function
David Bremner [Thu, 28 Jul 2022 10:41:02 +0000 (06:41 -0400)]
lib/open: refactor call to mkdir into function

This makes the error handling available for re-use. Using
g_mkdir_with_parents also handles the case of a pre-existing
directory. This introduces new functionality, namely creating the
parent directories, which will be useful for creating directories like
'.local/share/notmuch/default'.

2 years agotest/path-config: set database.mail_root but not database.path
David Bremner [Tue, 26 Jul 2022 01:04:21 +0000 (21:04 -0400)]
test/path-config: set database.mail_root but not database.path

The failing "create database" test replicates a bug reported by Sean
Whitton [1]. The other two failures also look related to the database
being (re)created in the wrong place.

[1]: id:87y1wqkw13.fsf@athena.silentflame.com.

2 years agotest: add test for creating database in various configurations.
David Bremner [Tue, 26 Jul 2022 00:37:41 +0000 (20:37 -0400)]
test: add test for creating database in various configurations.

The existing database creation (via add_email_corpus) was always done
in the traditional configuration. The use of xapian-metadata is just
to portably ensure that there is a database created where we expect
there to be.

2 years agoemacs/show: use read-shell-command instead of read-string
Antoine Beaupré [Tue, 30 Aug 2022 14:17:45 +0000 (10:17 -0400)]
emacs/show: use read-shell-command instead of read-string

This enables auto-completion of commands, something which plain
read-string does not do. It's otherwise a drop-in
replacement. According to `C-h f`, read-shell-command was introduced
in Emacs 23.1 or earlier.

2 years agoemacs: new notmuch-tree-process-exit-functions
jao [Sun, 21 Aug 2022 03:06:18 +0000 (04:06 +0100)]
emacs: new notmuch-tree-process-exit-functions

Hook run when the tree insertion process finishes its job.

--
This patch supersedes <id:20220816214023.1523322-1-jao@gnu.org>, but
changing the new variable name.

Right now, it can be used for silly things like removing or changing
the the "End of search." hardcoded message in the tree buffer.  But
also for more sophisticated things like folding all threads in add-ons
like my outline mode for tree buffers (to be submitted).

Signed-off-by: jao <jao@gnu.org>
2 years agodebian: update changelog for new build-depends archive/debian/0.37-1 debian/0.37-1
David Bremner [Wed, 24 Aug 2022 16:12:57 +0000 (09:12 -0700)]
debian: update changelog for new build-depends

2 years agodebian: build-depend on emacs-el
David Bremner [Wed, 24 Aug 2022 16:10:02 +0000 (09:10 -0700)]
debian: build-depend on emacs-el

This is hopefully temporary, to work around some bugs related to emacs
28 in Debian.

2 years agoNEWS: set release date for 0.37 0.37
David Bremner [Sun, 21 Aug 2022 20:17:37 +0000 (13:17 -0700)]
NEWS: set release date for 0.37

2 years agodebian: changelog for 0.37-1
David Bremner [Sun, 21 Aug 2022 20:09:25 +0000 (13:09 -0700)]
debian: changelog for 0.37-1

2 years agoversion: update to 0.37
David Bremner [Sun, 21 Aug 2022 20:05:34 +0000 (13:05 -0700)]
version: update to 0.37

2 years agoNEWS: minimal entries for two emacs changes.
David Bremner [Sun, 21 Aug 2022 20:02:08 +0000 (13:02 -0700)]
NEWS: minimal entries for two emacs changes.

2 years agotest: increase cffi timeout
Michael J Gruber [Sun, 14 Aug 2022 15:22:17 +0000 (17:22 +0200)]
test: increase cffi timeout

By default, the test suite uses 2min for other tests and 5s for cffi
tests. Sporadically, this leads to test failures caused by the timeout
on slower or loaded test infrastructure (as seen on ppc64le in Fedora's
infrastructure during branch time).

Increase the cffi timeout to the same 2m=120s.

Signed-off-by: Michael J Gruber <git@grubix.eu>
2 years agochangelog for 0.37~rc0-3 archive/debian/0.37_rc0-3 debian/0.37_rc0-3
David Bremner [Sun, 14 Aug 2022 14:49:54 +0000 (11:49 -0300)]
changelog for 0.37~rc0-3

2 years agodebian: changelog for 0.37~rc0-2 archive/debian/0.37_rc0-2 debian/0.37_rc0-2
David Bremner [Sun, 14 Aug 2022 13:56:15 +0000 (10:56 -0300)]
debian: changelog for 0.37~rc0-2

2 years agodebian: upload release candidate to experimental, not unstable archive/debian/0.37_rc0-1 debian/0.37_rc0-1
David Bremner [Sun, 14 Aug 2022 10:28:40 +0000 (07:28 -0300)]
debian: upload release candidate to experimental, not unstable

Oops, tagged "upstream" version is incorrect, but it's a minor issue.

2 years agodebian: changelog stanza for 0.37~rc0-1 0.37_rc0
David Bremner [Sun, 14 Aug 2022 09:55:40 +0000 (06:55 -0300)]
debian: changelog stanza for 0.37~rc0-1

2 years agoversion: update to 0.37~rc0
David Bremner [Sun, 14 Aug 2022 09:53:41 +0000 (06:53 -0300)]
version: update to 0.37~rc0

2 years agodoc/emacs: use define-key instead of hard-coding markup
David Bremner [Sun, 31 Jul 2022 11:33:23 +0000 (08:33 -0300)]
doc/emacs: use define-key instead of hard-coding markup

Add keys to index. Allow indexing of both keys and commands. Use
standard typesetting for multiple keybindings.

2 years agodoc/emacs: replace use of emacsvar with el:defcustom
David Bremner [Sun, 31 Jul 2022 11:33:22 +0000 (08:33 -0300)]
doc/emacs: replace use of emacsvar with el:defcustom

Mainly this allows dropping the ad-hoc directive emacsvar

2 years agodoc/emacs: replace emacscmd with el:define-key
David Bremner [Sun, 31 Jul 2022 11:33:21 +0000 (08:33 -0300)]
doc/emacs: replace emacscmd with el:define-key

This allows us to eliminate the ad-hoc emacscmd directive / role, and
improve indexing.

doc/emacs: convert remaining uses of emacscmd to define-key

This allows us to remove the no-longer used object type.

2 years agodoc/emacs: split index category from label
David Bremner [Sun, 31 Jul 2022 11:33:20 +0000 (08:33 -0300)]
doc/emacs: split index category from label

A more verbose category is needed for the index, but not the
description in the manual.

2 years agodoc/emacs: drop domain label
David Bremner [Sun, 31 Jul 2022 11:33:19 +0000 (08:33 -0300)]
doc/emacs: drop domain label

This makes the generate info files somewhat less verbose, so that they
say e.g. "Key Binding" instead of "Emacs Key Binding"

2 years agodoc/elisp: replace label for Emacs commands and key bindings.
David Bremner [Sun, 31 Jul 2022 11:33:18 +0000 (08:33 -0300)]
doc/elisp: replace label for Emacs commands and key bindings.

In the notmuch doc index it is important to be clear which parts are
Emacs related.

2 years agodoc: copy elisp.py from flycheck, make available
David Bremner [Sun, 31 Jul 2022 11:33:17 +0000 (08:33 -0300)]
doc: copy elisp.py from flycheck, make available

This corresponds to flycheck commit
1702d2db3d8ba9bcb9b2bf810e791e907fcc3adc, which is apparently the last
time this file was modified (2018).

Embedding seems defensible since the file is evidently not changing
that much, and we need to change some labels to make it fit with the
not-just-emacs nature of notmuch docs.

2 years agodoc: add some documentation about duplicate files.
David Bremner [Sat, 30 Jul 2022 23:45:51 +0000 (20:45 -0300)]
doc: add some documentation about duplicate files.

This is mainly motivated by the new emacs-UI features to select
duplicates, but hopefully it will help demystify things for users of
the CLI as well.

2 years agoemacs: add docstring for notmuch-show-choose-duplicate
David Bremner [Sun, 31 Jul 2022 12:46:51 +0000 (09:46 -0300)]
emacs: add docstring for notmuch-show-choose-duplicate

It should have one in any case, but in particular it is needed for the
manual.

2 years agoemacs/show: restrict inlined mimetypes on refresh.
David Bremner [Mon, 1 Aug 2022 11:39:18 +0000 (08:39 -0300)]
emacs/show: restrict inlined mimetypes on refresh.

This fixes the bug reported by Al [1]. Essentially apply the same fix
as [2] in a different place.

[1]: id:877d41nmr1.fsf@gmail.com
[2]: 90a7c1af368a527700dcde9b0dcbd760afc7bd92

2 years agotest/emacs: add known broken test for inlining on refresh.
David Bremner [Mon, 1 Aug 2022 11:39:17 +0000 (08:39 -0300)]
test/emacs: add known broken test for inlining on refresh.

This duplicates the bug reported by Al [1]

[1]: id:877d41nmr1.fsf@gmail.com

2 years agoemacs: add duplicate support to notmuch-show-view-raw-message
David Bremner [Thu, 4 Aug 2022 11:48:08 +0000 (08:48 -0300)]
emacs: add duplicate support to notmuch-show-view-raw-message

This fixes one of the bugs reported by Gregor in [1]

[1]: id:87edxw8jp4.fsf@no.workgroup

2 years agoemacs/show: provide a more friendly function to get duplicate num
David Bremner [Thu, 4 Aug 2022 11:48:07 +0000 (08:48 -0300)]
emacs/show: provide a more friendly function to get duplicate num

There is not much code here, but at least we can fix the indexing as 1
based.

2 years agotest/emacs: add known broken test for viewing raw duplicate
David Bremner [Thu, 4 Aug 2022 11:48:06 +0000 (08:48 -0300)]
test/emacs: add known broken test for viewing raw duplicate

This duplicates one of the bugs reported by Gregor in [1].

[1]: id:87edxw8jp4.fsf@no.workgroup

2 years agoemacs: update notmuch-show-get-filename to handle duplicates
David Bremner [Thu, 4 Aug 2022 11:08:33 +0000 (08:08 -0300)]
emacs: update notmuch-show-get-filename to handle duplicates

This fixes one of the bugs reported by Gregor in [1].

[1]: id:87edxw8jp4.fsf@no.workgroup

2 years agotest/emacs: add known broken test for stashing duplicate filename
David Bremner [Thu, 4 Aug 2022 11:08:32 +0000 (08:08 -0300)]
test/emacs: add known broken test for stashing duplicate filename

This duplicates (no pun intended) one of the bugs reported by Gregor
in id:87edxw8jp4.fsf@no.workgroup

2 years agoNEWS: initial set of NEWS for 0.37.
David Bremner [Mon, 1 Aug 2022 13:14:15 +0000 (10:14 -0300)]
NEWS: initial set of NEWS for 0.37.

There are one or two more changes to apply (and potentially document)
before release, but this covers most of my commits.

2 years agodoc/notmuch: merge two sections on external commands.
David Bremner [Sat, 30 Jul 2022 23:45:49 +0000 (20:45 -0300)]
doc/notmuch: merge two sections on external commands.

When adding the description of the propagation of NOTMUCH_CONFIG, I
missed that there was already a section on external commands, with a
different title.

2 years agotests: remove dead code from T055-path-config.sh
David Bremner [Fri, 29 Jul 2022 12:19:56 +0000 (09:19 -0300)]
tests: remove dead code from T055-path-config.sh

This case statement does nothing.

2 years agonotmuch.c: add missing trailing newlines in two error messages
Tomi Ollila [Sun, 31 Jul 2022 17:32:11 +0000 (20:32 +0300)]
notmuch.c: add missing trailing newlines in two error messages

Removed duplicate error check (and the message) in 3rd case where
the same error message (w/o trailing newline) was present.

In case of test/T040-setup.sh, command substitution deletes
trailing newlines, so related test there cannot be changed
(and therefore could not notice this user experience flaw).

2 years agodoc/emacs: add docstring for notmuch-unthreaded-result-format.
David Bremner [Sat, 16 Jul 2022 21:22:28 +0000 (17:22 -0400)]
doc/emacs: add docstring for notmuch-unthreaded-result-format.

The main change is actually updating cross references.

2 years agoemacs: update defcustom for notmuch-unthreaded-result-format
David Bremner [Sat, 16 Jul 2022 21:22:27 +0000 (17:22 -0400)]
emacs: update defcustom for notmuch-unthreaded-result-format

This is essentially a copy of that for notmuch-tree-result-format,
aside from the default value.

2 years agodoc/emacs: add docstring and example for n-tree-result-format
David Bremner [Sat, 16 Jul 2022 21:22:26 +0000 (17:22 -0400)]
doc/emacs: add docstring and example for n-tree-result-format

This example is based on one originally by Jon Hurst.

2 years agoemacs: update defcustom for notmuch-search-result-format.
David Bremner [Sat, 16 Jul 2022 21:22:25 +0000 (17:22 -0400)]
emacs: update defcustom for notmuch-search-result-format.

It seems redundant to have the previous example, since the default
value is always show by describe variable.

Enforce more restrictions on the keys in the alist, since arbitrary
strings don't work as field names.

Document that functions can be used in lieu of field names.

2 years agodoc/emacs: add docstring and example for n-search-result-format
David Bremner [Sat, 16 Jul 2022 21:22:24 +0000 (17:22 -0400)]
doc/emacs: add docstring and example for n-search-result-format

When the ability to use functions was added, this example was confined
to a commit message, which is not user discoverable.

2 years agoemacs: update defcustom for notmuch-search-result-format.
David Bremner [Sat, 16 Jul 2022 21:22:23 +0000 (17:22 -0400)]
emacs: update defcustom for notmuch-search-result-format.

It seems redundant to have the previous example, since the default
value is always show by describe variable.

Enforce more restrictions on the keys in the alist, since arbitrary
strings don't work as field names.

Document that functions can be used in lieu of field names.

2 years agoemacs/reply: reply to correct duplicate
David Bremner [Fri, 1 Jul 2022 21:45:48 +0000 (18:45 -0300)]
emacs/reply: reply to correct duplicate

Essentially we just need to arrange to pass the right --duplicate
argument to notmuch reply.

As a side-effect, correct the previously unused value of EXPECTED in
T453-emacs-reply.sh.

2 years agoCLI/reply: support --duplicate argument
David Bremner [Fri, 1 Jul 2022 21:45:47 +0000 (18:45 -0300)]
CLI/reply: support --duplicate argument

We want the reply used to match that shown e.g. in the emacs
interface. As a first step provide that functionality on the command
line.

Schema does not need updating as the duplicate key was already
present (with a constant value of 1).

2 years agoemacs/show: display count of duplicates in headerline
David Bremner [Fri, 1 Jul 2022 21:45:46 +0000 (18:45 -0300)]
emacs/show: display count of duplicates in headerline

There is no real cost here, except screen real estate. Some people
might prefer hiding the duplicate count, but we leave that for a
future commit.

2 years agoemacs/show: provide notmuch-show-choose-duplicate
David Bremner [Fri, 1 Jul 2022 21:45:45 +0000 (18:45 -0300)]
emacs/show: provide notmuch-show-choose-duplicate

This new command allows the user to interactively choose a different
duplicate (file) to display for a given message in
notmuch-show-mode. Since both tree and unthreaded view use
notmuch-show-mode, this provides the same facility there.

2 years agoCLI/show: support --duplicate for structured output
David Bremner [Fri, 1 Jul 2022 21:45:44 +0000 (18:45 -0300)]
CLI/show: support --duplicate for structured output

This introduces a new mandatory key for message structures, namely
"duplicate". Per convention in devel/schemata this does _not_ increase
the format version. This means that clients are responsible for
checking that it exists, and not crashing if it does not.

The main functional change is teaching mime_node_open to understand a
'duplicate' argument.

Support for --duplicate in notmuch-reply would make sense, but we
defer it to a later commit.

2 years agoCLI/show: initial support for --duplicate for (raw output only)
David Bremner [Fri, 1 Jul 2022 21:45:43 +0000 (18:45 -0300)]
CLI/show: initial support for --duplicate for (raw output only)

Add command line argument --duplicate, analogous with that already
supported for notmuch-search.

Use of a seperate function for _get_filename is mainly a form of
documentation at this point.

md5sum is of course a weak hash, but it is good enough for
this (non-adversarial) test suite use.

2 years agoCLI/git: opportunistically use bindings to check for known messages
David Bremner [Fri, 15 Jul 2022 14:23:21 +0000 (10:23 -0400)]
CLI/git: opportunistically use bindings to check for known messages

If the bindings are installed, use them to avoid one exec of notmuch
search per message.

2 years agoCLI/git: current cache contents of index
David Bremner [Sat, 2 Jul 2022 23:45:11 +0000 (20:45 -0300)]
CLI/git: current cache contents of index

Rather than shelling out once per message to get the list of files
corresponding to tags, it is much faster (although potentially a bit
memory intensive) to read them all at once.

2 years agoperf-test: add tests notmuch-git
David Bremner [Sat, 2 Jul 2022 19:11:58 +0000 (16:11 -0300)]
perf-test: add tests notmuch-git

The main focus of these initial tests is the (currently unacceptably
slow) checkout performance.

2 years agodoc/emacs: add notmuch-show-empty-saved-searches to manual
David Bremner [Fri, 1 Jul 2022 16:14:29 +0000 (13:14 -0300)]
doc/emacs: add notmuch-show-empty-saved-searches to manual

Answering a user question, I had to dig for this variable, but I think
it is a reasonably common customization wish, particularly for users
with custom count-functions.

2 years agodoc/emacs: define and use emacsvar and emacscmd markup
David Bremner [Fri, 1 Jul 2022 10:40:39 +0000 (07:40 -0300)]
doc/emacs: define and use emacsvar and emacscmd markup

This makes it easier to navigate the merged index of all of the
notmuch documentation.

2 years agodoc: do not generate python module index
David Bremner [Fri, 1 Jul 2022 10:52:35 +0000 (07:52 -0300)]
doc: do not generate python module index

This currently seems unhelpful, and not generating it removes if from
the navigation bar, which prevents mis-clicking.

2 years agodoc: drop module index from ToC
David Bremner [Fri, 1 Jul 2022 09:52:04 +0000 (06:52 -0300)]
doc: drop module index from ToC

There is only one module listed, so the index is not useful.

2 years agodoc: retitle notmuch-emacs manual
David Bremner [Thu, 30 Jun 2022 12:27:10 +0000 (09:27 -0300)]
doc: retitle notmuch-emacs manual

This makes the HTML TOC read more naturally. Add a label to make cross
referencing easier.

2 years agodoc: reorganize sphinx toctree
David Bremner [Thu, 30 Jun 2022 12:00:43 +0000 (09:00 -0300)]
doc: reorganize sphinx toctree

This mainly affects the html output (although users generating epub or
pdf would also notice a change). The goal is twofold: make the TOC a
bit friendler and easier to navigate by introducing some hierarchy,
and allow links for nmbug and notmuch-setup.

2 years agodoc: improve config item labels in info output
David Bremner [Fri, 1 Jul 2022 11:23:24 +0000 (08:23 -0300)]
doc: improve config item labels in info output

Commit 306b7028d added the nmconfig role / directive. Unfortunately
the default of using the directive name in texinfo output is pretty
ugly, so attempt to make it more human readable by passing `objname`
to add_object_type invocation.

2 years agodoc/emacs: add discussion about notmuch-show performance.
David Bremner [Tue, 28 Jun 2022 01:02:05 +0000 (22:02 -0300)]
doc/emacs: add discussion about notmuch-show performance.

Most of the information is already in the docstrings, but hopefully
by grouping it is more discoverable.

2 years agoemacs/show: introduce notmuch-show-height-limit
David Bremner [Tue, 28 Jun 2022 01:02:04 +0000 (22:02 -0300)]
emacs/show: introduce notmuch-show-height-limit

Analogous to notmuch-show-depth-limit, it requires a bit more
calculation, but that cost seems dominated by the rendering cost.

2 years agoemacs/show: obey notmuch-show-max-text-part-size for part 0
David Bremner [Tue, 28 Jun 2022 01:02:03 +0000 (22:02 -0300)]
emacs/show: obey notmuch-show-max-text-part-size for part 0

This parameter was originally introduced to hide large attachements
that happened to be text/plain. From a performance point of view,
there is no reason not to also hide large message bodies.

2 years agoemacs/show: optionally insert bodies of "deep messages" lazily
David Bremner [Tue, 28 Jun 2022 01:02:02 +0000 (22:02 -0300)]
emacs/show: optionally insert bodies of "deep messages" lazily

This leverages the machinery already there to insert buttons for
attachments.

A potential use-case is browsing the top layers of the tree to decide
which of the lower subtrees to read.

2 years agoperf-test/emacs: test showing longish threads
David Bremner [Tue, 28 Jun 2022 01:02:01 +0000 (22:02 -0300)]
perf-test/emacs: test showing longish threads

This particular thread takes about 100 times longer to display in
emacs than on the command line.

The parameter notmuch-show-max-text-part-size exists, but is currently
ineffective for this task because it only hides attachments, not
part 0.

In future commits, the parameter notmuch-show-{depth,height}-limit
will trigger a similar kind of hiding for bodies as
n-s-m-text-part-size already does for attachments.

2 years agodebian: add git as a build-dependency, for the test suite
David Bremner [Sun, 3 Jul 2022 15:11:00 +0000 (12:11 -0300)]
debian: add git as a build-dependency, for the test suite

This is needed to run (and test) notmuch-git.

2 years agoCLI/git: add --format-version argument to init subcommand
David Bremner [Thu, 23 Jun 2022 12:30:45 +0000 (09:30 -0300)]
CLI/git: add --format-version argument to init subcommand

This is primarily intended to support testing upward compatibility
with legacy repos.

2 years agoCL/git: add format version 1
David Bremner [Thu, 23 Jun 2022 12:30:44 +0000 (09:30 -0300)]
CL/git: add format version 1

The original nmbug format (now called version 0) creates 1
subdirectory of 'tags/' per message. This causes problems for more
than (roughly) 100k messages.

Version 1 introduces 2 layers of hashed directories. This scheme was
chose to balance the number of subdirectories with the number of extra
directories (and git objects) created via hashing.

This should be upward compatible in the sense that old repositories
will continue to work with the updated notmuch-git.

2 years agoCLI/git: replace most mentions of nmbug
David Bremner [Mon, 4 Jul 2022 15:33:48 +0000 (12:33 -0300)]
CLI/git: replace most mentions of nmbug

Particularly in help messages, nmbug is confusing for users who may
have never heard of it.

2 years agotest: known broken test for top level .notmuch in split configs
David Bremner [Sun, 16 Jan 2022 15:25:10 +0000 (11:25 -0400)]
test: known broken test for top level .notmuch in split configs

In split configurations there is no special significance to a top
level directory called .notmuch in the mail root. Users should
therefore be able to have mail stored underneath it.

2 years agobindings/python-cffi: fix docstring
David Bremner [Sun, 3 Jul 2022 20:26:46 +0000 (17:26 -0300)]
bindings/python-cffi: fix docstring

the method Database.get_message does exist any more (if it ever
did). This makes the docstring unhelpful as an example.

2 years agotest: add new corpus of duplicate messages
David Bremner [Fri, 1 Jul 2022 21:45:42 +0000 (18:45 -0300)]
test: add new corpus of duplicate messages

This corpus will be used to test a new --duplicate option for notmuch-show

2 years agotest: define and use notmuch_sexp_*_sanitize functions
David Bremner [Fri, 1 Jul 2022 21:45:41 +0000 (18:45 -0300)]
test: define and use notmuch_sexp_*_sanitize functions

These are based on the equivalent functions for json. Like those,
these are pretty simple-minded, and don't really understand the
syntax.

2 years agotest: use notmuch_json_show_sanitize more places
David Bremner [Fri, 1 Jul 2022 21:45:40 +0000 (18:45 -0300)]
test: use notmuch_json_show_sanitize more places

This makes the tests more robust against changing output formats, by
allowing us to centralize fixes in the sanitization function. It is
not appropriate for all cases, in particular it is unneeded when using
test_json_nodes, and unhelpful when testing filenames.

2 years agodoc: replace symlink with copies for nmbug, notmuch-setup
David Bremner [Fri, 1 Jul 2022 01:14:20 +0000 (22:14 -0300)]
doc: replace symlink with copies for nmbug, notmuch-setup

Previously only man page aliases were being added as symlinks.  The
addition to man_pages in conf.py automatically propagates to the list
of generated info pages.

Installation of the new pages is handled by existing recipes.

2 years agoemacs: mark notmuch-query.el as obsolete
David Bremner [Tue, 28 Jun 2022 11:50:55 +0000 (08:50 -0300)]
emacs: mark notmuch-query.el as obsolete

The only functionality actually used by notmuch is the base function
notmuch-query-get-threads; the other functions in this file have
nothing to do with that (single) use.  Move that function into
notmuch-lib.el and rename to reflect use. Deprecate the other
functions in notmuch-query.el.

2 years agolib/sexp: add parameter expansion for regex and wildcard
David Bremner [Wed, 15 Jun 2022 12:14:47 +0000 (09:14 -0300)]
lib/sexp: add parameter expansion for regex and wildcard

Fix the bug reported at [1].

The parameter expansion for regex and wildcard modifiers has to be
done a bit differently, because their arguments are not s-expressions
defining complete Xapian queries.

[1]: id:87o7yxqxy6.fsf@code.pm

2 years agotest/sexp: add known broken tests for macro param inside rx/wildcard
David Bremner [Wed, 15 Jun 2022 12:14:46 +0000 (09:14 -0300)]
test/sexp: add known broken tests for macro param inside rx/wildcard

These tests replicate the problem reported by Eric Colson [1] (for the
regex case).

[1]: id:87o7yxqxy6.fsf@code.pm

2 years agomake git ignore new build products
Michael J Gruber [Wed, 29 Jun 2022 11:04:35 +0000 (13:04 +0200)]
make git ignore new build products

nmbug and notmuch-git are new build products. Make git ignore them just
like other build products.

Signed-off-by: Michael J Gruber <git@grubix.eu>
2 years agodoc/sexp: document range queries.
David Bremner [Sat, 4 Jun 2022 21:53:59 +0000 (18:53 -0300)]
doc/sexp: document range queries.

Give examples for date fields, as these are commonly useful for user
queries.

2 years agolib/sexp: allow * as alias for "" in range searches.
David Bremner [Sat, 4 Jun 2022 21:53:58 +0000 (18:53 -0300)]
lib/sexp: allow * as alias for "" in range searches.

It can be tedious to use "" inside of a string, e.g. in a shell script.

2 years agolib/sexp: special case "" as an argument in lastmod ranges.
David Bremner [Sat, 4 Jun 2022 21:53:57 +0000 (18:53 -0300)]
lib/sexp: special case "" as an argument in lastmod ranges.

Support this syntax for constincy with (data from to) ranges.

2 years agotest/sexp: one sided range searches.
David Bremner [Sat, 4 Jun 2022 21:53:56 +0000 (18:53 -0300)]
test/sexp: one sided range searches.

The date range parsing machinery already knows how to do something
appropriate with an empty string, but the lastmod parsing blindly
tries to parse each atom as a number.

2 years agolib: check for writable db in n_m_tags_maildir_flags
David Bremner [Wed, 25 May 2022 10:51:17 +0000 (07:51 -0300)]
lib: check for writable db in n_m_tags_maildir_flags

The database needs to be writable because the list of stored file
names will change in general.

2 years agolib: add NOTMUCH_STATUS_CLOSED_DATABASE, use in _n_d_ensure_writable
David Bremner [Wed, 25 May 2022 10:51:16 +0000 (07:51 -0300)]
lib: add NOTMUCH_STATUS_CLOSED_DATABASE, use in _n_d_ensure_writable

In order for a database to actually be writeable, it must be the case that it
is open, not just the correct type of Xapian object. By explicitely
checking, we are able to provide better error reporting, in particular
for the previously broken test in T566-lib-message.

2 years agolib: Add missing private status values.
David Bremner [Wed, 25 May 2022 10:51:15 +0000 (07:51 -0300)]
lib: Add missing private status values.

These were missed when the corresponding status codes were added.

2 years agolib/tag: handle NULL argument to notmuch_tags_valid
David Bremner [Wed, 25 May 2022 10:51:14 +0000 (07:51 -0300)]
lib/tag: handle NULL argument to notmuch_tags_valid

Make the behaviour when passed NULL consistent with
notmuch_filenames_valid. The library already passes the result of
notmuch_message_get_tags without checking for NULL, so it should be
handled.

2 years agotest: add known broken test for notmuch_tags_valid (NULL)
David Bremner [Wed, 25 May 2022 10:51:13 +0000 (07:51 -0300)]
test: add known broken test for notmuch_tags_valid (NULL)

This should return false, but currently segfaults.

Start a new file for tags library API related tests. This is maybe
overkill, but new C boilerplate which doesn't corrupt the database is
needed anyway.

2 years agotest: error handling for n_m_tags_to_maildir_flags
David Bremner [Wed, 25 May 2022 10:51:12 +0000 (07:51 -0300)]
test: error handling for n_m_tags_to_maildir_flags

The closed database case should fail gracefully, but currently it
segfaults.

2 years agotest: error handling in _n_message_{add,remove}_filename
David Bremner [Mon, 23 May 2022 23:39:01 +0000 (20:39 -0300)]
test: error handling in _n_message_{add,remove}_filename

Use a closed database to force throwing a Xapian exception. Leave the
check vague to accomodate a potential explicit check for open database.

2 years agolib/message: check return status from _n_m_add_{path,folder}_terms
David Bremner [Mon, 23 May 2022 23:39:00 +0000 (20:39 -0300)]
lib/message: check return status from _n_m_add_{path,folder}_terms

Mainly to propagate information about Xapian exceptions.

2 years agolib/message: check return status of _n_m_{add,remove}_term
David Bremner [Mon, 23 May 2022 23:38:59 +0000 (20:38 -0300)]
lib/message: check return status of _n_m_{add,remove}_term

Xapian exceptions are not something that can be ignored, in general.

2 years agolib: define macro NODISCARD
David Bremner [Mon, 23 May 2022 23:38:58 +0000 (20:38 -0300)]
lib: define macro NODISCARD

In either C++17 (or later) mode, or when running cppcheck, this can be
used to selectively generate warnings about discarded return values.

2 years agolib/message: drop _notmuch_message_get_thread_id_only
David Bremner [Mon, 23 May 2022 23:38:57 +0000 (20:38 -0300)]
lib/message: drop _notmuch_message_get_thread_id_only

This function has been unused since commit 4083fd8.

2 years agotest: _notmuch_message_remove_term catches exceptions.
David Bremner [Mon, 23 May 2022 23:38:56 +0000 (20:38 -0300)]
test: _notmuch_message_remove_term catches exceptions.

Unfortunately we can't differentiate between the two distinct error
conditions for Document::remove_term.