9 years agotag: Automatically limit to messages whose tags will actually change.
Austin Clements [Wed, 9 Nov 2011 13:44:35 +0000 (08:44 -0500)]
tag: Automatically limit to messages whose tags will actually change.

This optimizes the user's tagging query to exclude messages that won't
be affected by the tagging operation, saving computation and IO for
redundant tagging operations.

For example,
  notmuch tag +notmuch
will now use the query
  ( ) and (not tag:"notmuch")

In the past, we've often suggested that people do this exact
transformation by hand for slow tagging operations.  This makes that

9 years agotest: fix "Stashing in notmuch-search" test when emacs is not available
Dmitry Kurochkin [Sun, 27 Nov 2011 18:36:19 +0000 (22:36 +0400)]
test: fix "Stashing in notmuch-search" test when emacs is not available

If emacs is not available, test_expect_equal would be called with only
one argument.  The patch fixes this by quoting the (possibly empty)
$(cat OUTPUT) argument.

9 years agotest: check if emacs is available in the beginning of test_emacs
Dmitry Kurochkin [Sun, 27 Nov 2011 18:36:18 +0000 (22:36 +0400)]
test: check if emacs is available in the beginning of test_emacs

Unfortunately, this is needed to avoid the emacs waiting loop.

9 years agotest: add function to explicitly check for external dependencies
Dmitry Kurochkin [Sun, 27 Nov 2011 18:36:17 +0000 (22:36 +0400)]
test: add function to explicitly check for external dependencies

Useful when binary is called indirectly (e.g. from emacs).

9 years agotest: declare external dependencies for the tests
Dmitry Kurochkin [Sun, 27 Nov 2011 18:36:16 +0000 (22:36 +0400)]
test: declare external dependencies for the tests

That are: dtach(1), emacs(1), emacsclient(1), gdb(1) and gpg(1).

9 years agotest: skip all subtests if external dependencies are missing during init
Dmitry Kurochkin [Sun, 27 Nov 2011 18:36:15 +0000 (22:36 +0400)]
test: skip all subtests if external dependencies are missing during init

Some tests (e.g. crypto) do a common initialization required for all
subtests.  The patch adds a check for missing external dependencies
during this initialization.  If any prerequisites are missing, all
subtests are skipped.

The check is run on the first call of test_reset_state_ function, so
no changes for the tests are needed.

9 years agotest: fix "skipping test" verbose output
Dmitry Kurochkin [Sun, 27 Nov 2011 18:36:14 +0000 (22:36 +0400)]
test: fix "skipping test" verbose output

9 years agotest: add support for external executable dependencies
Dmitry Kurochkin [Sun, 27 Nov 2011 18:36:13 +0000 (22:36 +0400)]
test: add support for external executable dependencies

There is existing support for general prerequisites in the test suite.
But it is not very convenient to use: every test case has to keep
track for it's dependencies and they have to be explicitly listed.

The patch aims to add better support for a particular type of external
dependencies: external executables.  The main idea is to replace
missing external binaries with shell functions that have the same
name.  These functions always fail and keep track of missing
dependencies for a subtest.  The result reporting functions later can
check that an external binaries are missing and correctly report SKIP
result instead of FAIL.  The primary benefit is that the test cases do
not need to declare their dependencies or be changed in any way.

9 years agotest: add test state reset to test_expect_* functions that did not have it
Dmitry Kurochkin [Thu, 17 Nov 2011 13:05:56 +0000 (17:05 +0400)]
test: add test state reset to test_expect_* functions that did not have it

9 years agotest: set EMACS_SERVER variable only after dtach(1) was successfully started
Dmitry Kurochkin [Thu, 17 Nov 2011 13:05:55 +0000 (17:05 +0400)]
test: set EMACS_SERVER variable only after dtach(1) was successfully started

Otherwise, we can set the EMACS_SERVER and return with an error.  And
subsequent calls to test_emacs would assume that emacs server is running.

9 years agotest: move subtest variables reset into a dedicated function
Dmitry Kurochkin [Thu, 17 Nov 2011 13:05:54 +0000 (17:05 +0400)]
test: move subtest variables reset into a dedicated function

Currently, there is only one such variable test_subtest_known_broken_.
But more will be added in the future.

9 years agoBuild symbol-test with make instead of hardcoding in symbol-hiding.
Amadeusz Żołnowski [Sat, 26 Nov 2011 21:14:20 +0000 (22:14 +0100)]
Build symbol-test with make instead of hardcoding in symbol-hiding.

If symbol-test is built in symbol-hiding with hardcoded g++ invokation,
it's not so easy to pass $(srcdir) which is required to find notmuch.h
when srcdir and builddir are separate directories.

9 years agoMerge branch 'release'
David Bremner [Sun, 27 Nov 2011 05:15:20 +0000 (21:15 -0800)]
Merge branch 'release'

9 years agobuild system: clean up object files in ./test and ./util
David Bremner [Sat, 26 Nov 2011 17:55:07 +0000 (09:55 -0800)]
build system: clean up object files in ./test and ./util

9 years agoWhitespaces cleanup.
Amadeusz Żołnowski [Fri, 25 Nov 2011 15:34:48 +0000 (16:34 +0100)]
Whitespaces cleanup.

9 years agoFix warnings for test/
Amadeusz Żołnowski [Fri, 25 Nov 2011 15:34:47 +0000 (16:34 +0100)]
Fix warnings for test/

9 years agotest: cleanup basic tests
Dmitry Kurochkin [Sat, 26 Nov 2011 06:12:26 +0000 (10:12 +0400)]
test: cleanup basic tests

Basic test 'Ensure that all available tests will be run by
notmuch-test' compares all tests that are run with listing of test/
directory.  There is a growing list of exceptions for files and
directories which located in the test/ directory but are not tests.
Moreover some (probably buggy) tests do create files in the the test/
directory which may be left behind in case of failure.  This makes the
basic test fail.

The patch changes the test to look only for regular executable files.
This makes the exception list much smaller.  And since no tests should
create executables in the test/ directory (if there are, they should
be fixed), the basic test should not be affected by failed or
interrupted tests.

9 years agotest: fix sed error in basic tests
Dmitry Kurochkin [Fri, 18 Nov 2011 15:21:00 +0000 (19:21 +0400)]
test: fix sed error in basic tests

The error is easy to miss, because the test passes and stderr is not
printed.  But if you run basic tests in verbose mode (./basic
--verbose), you get:

  sed: can't read notmuch-test: No such file or directory

The issue is that sed command is given two files: notmuch-test and
$TEST_DIRECTORY/notmuch-test.  And there is no notmuch-test file in
the current directory (test/tmp.basic/).  The patch just removes the
non-existing file from the sed command.

9 years agotest: remove executable permissions from
Dmitry Kurochkin [Sat, 26 Nov 2011 06:12:25 +0000 (10:12 +0400)]
test: remove executable permissions from

It is not supposed to be run, only sourced.

9 years agoemacs: test notmuch show with fourfold message indentation
Gregor Zattler [Thu, 24 Nov 2011 22:03:24 +0000 (23:03 +0100)]
emacs: test notmuch show with fourfold message indentation

Previous behaviour was to indent messages in a thread according
to depth by one space per level.  This commit tests if setting
notmuch-indent-messages-width to `4' provides a message thread
with four spaces of indentation thread depth.

9 years agoemacs: test: notmuch show without indentation
Gregor Zattler [Thu, 24 Nov 2011 22:03:23 +0000 (23:03 +0100)]
emacs: test: notmuch show without indentation

Previous behaviour was to indent messages in a thread according
to depth by one space per level.  In Felix
Geller proposed a patch in order to turn indentation off.  This
commit tests if instead setting notmuch-indent-messages-width to
`0' does turn off indentation.

9 years agoemacs: test notmuch-indent-messages-width default
Gregor Zattler [Thu, 24 Nov 2011 22:03:22 +0000 (23:03 +0100)]
emacs: test notmuch-indent-messages-width default

Previous behaviour was to indent messages in a thread according
to depth by one space per level.  This is still the case with
notmuch-indent-messages-width default value `1'.  This test
succeeds if output with default value is same as in "Basic
notmuch-show view in emacs".

9 years agoemacs: make message indentation width customisable
Gregor Zattler [Thu, 24 Nov 2011 22:03:21 +0000 (23:03 +0100)]
emacs: make message indentation width customisable

Till now Emacs UI indents messages according to their respecive
depth of neting in the thread.  The actual width of indentation
per level is hardcoded to `1' space.
This patch makes message indentation customisable by introducing
a variable `notmuch-indent-messages-width' which defaults to `1',
which is the same as before.  Felix could set this variable to
`0' in order to disable indentation, I tested it with a value of
`4' for a clearer separation of messages in a thread.

9 years agotest: add simple tests for online help 0.10.1 debian/0.10.1-1
David Bremner [Fri, 25 Nov 2011 17:28:11 +0000 (12:28 -0500)]
test: add simple tests for online help

Nothing fancy, but we can at least detect segmentation faults.

9 years agoNEWS: add NEWS stanza for 0.10.1
David Bremner [Fri, 25 Nov 2011 17:17:35 +0000 (12:17 -0500)]
NEWS: add NEWS stanza for 0.10.1

Explain the bug fix in slightly less technical language than in the
Debian changelog.

9 years agodebian: changelog stanza for 0.10.1
David Bremner [Fri, 25 Nov 2011 17:13:01 +0000 (12:13 -0500)]
debian: changelog stanza for 0.10.1

9 years agoversion: update to 0.10.1
David Bremner [Fri, 25 Nov 2011 17:11:04 +0000 (12:11 -0500)]
version: update to 0.10.1

9 years agoCLI: update call to notmuch_help_command for new calling conventions.
David Bremner [Thu, 24 Nov 2011 20:02:41 +0000 (16:02 -0400)]
CLI: update call to notmuch_help_command for new calling conventions.

When I changed the calling convention to pass in all but the zero-th
argument to subcommands, I missed this one call, resulting in a
segmentation fault.

As a bonus, the syntax "notmuch --help foo" is now equivalent to
"notmuch help foo".

9 years agomake release: use sed to check debian version
Tomi Ollila [Mon, 21 Nov 2011 16:34:24 +0000 (18:34 +0200)]
make release: use sed to check debian version

Use common sed tool instead of dpkg-parsechangelog (which is usually
available on debian systems only) to verify that debian version
information is consistent with version file.

9 years agomake release: added goal verify-version-manpage
Tomi Ollila [Mon, 21 Nov 2011 16:27:09 +0000 (18:27 +0200)]
make release: added goal verify-version-manpage

Check that the version mentioned in notmuch manual page
is consistent with the version file.

9 years agomake release: verify-version-*: change comparison logic
Tomi Ollila [Mon, 21 Nov 2011 15:55:20 +0000 (17:55 +0200)]
make release: verify-version-*: change comparison logic

verfy-version-debian, verify-version-python and verify-version-components
checked noneqality of the comparison strings and if got "positive"
answer then made that goal fail. But in case of the test ([ ])
execution failed it never got to the 'then' part of the line (and
the 'if [ ... ] then ... fi ' construct doesn't make the script line
fail in case of problems inside [ ].
This commit inverses the "logic", so that only if the comparison for
equality succeeds the script line will exit with 0 and execution
can continue past the failure case to the next line (executed by another
shell) with '@echo done'

9 years agoemacs: Avoid unnecessary markers.
Austin Clements [Mon, 21 Nov 2011 01:07:02 +0000 (20:07 -0500)]
emacs: Avoid unnecessary markers.

This is just cleanup.  These markers are all immediately resolved to
points by Emacs, so using markers here is just unncessary overhead.

9 years agoemacs: Don't record undo information for search or show buffers.
Austin Clements [Mon, 21 Nov 2011 01:00:55 +0000 (20:00 -0500)]
emacs: Don't record undo information for search or show buffers.

There's no reason to record undo information for read-only,
programmatically-constructed buffers.  The undo list just chews up
memory keeping track of our calls to insert.

9 years agodebian/control: require dtach version >= 0.8 in Build-Depends
Tomi Ollila [Mon, 21 Nov 2011 15:18:00 +0000 (17:18 +0200)]
debian/control: require dtach version >= 0.8 in Build-Depends

Since version 0.8 of dtach -n does no longer require controlling
tty to be present when executed. Currently controlling tty is not
always (if ever) present when tests are executed.

9 years agoMerge branch 'release'
David Bremner [Wed, 23 Nov 2011 20:02:47 +0000 (16:02 -0400)]
Merge branch 'release'

9 years agodebian: changelog stanza for 0.10 0.10
David Bremner [Wed, 23 Nov 2011 11:46:39 +0000 (07:46 -0400)]
debian: changelog stanza for 0.10

Give a brief summary of (and pointer to) NEWS

9 years agoNEWS: set (approximate) date for 0.10 release
David Bremner [Wed, 23 Nov 2011 11:43:01 +0000 (07:43 -0400)]
NEWS: set (approximate) date for 0.10 release

It doesn't really matter if we are off by a day.

9 years agoversion: update to 0.10
David Bremner [Wed, 23 Nov 2011 11:37:02 +0000 (07:37 -0400)]
version: update to 0.10

9 years agoNEWS: item for silent killing of search buffers.
David Bremner [Wed, 23 Nov 2011 11:34:22 +0000 (07:34 -0400)]
NEWS: item for silent killing of search buffers.

This is maybe borderline for inclusion in NEWS, but maybe the NEWS
item helps someone who thinks there is a bug.

9 years agocontrib/notmuch-deliver: don't export from git
David Bremner [Wed, 23 Nov 2011 11:25:24 +0000 (07:25 -0400)]
contrib/notmuch-deliver: don't export from git

This prevents it from being part of the release tarballs. Hopefully
this is only temporary while we sort out the GPL2 versus GPL3

9 years agoemacs: Make saving new saved searches append, not prepend
Jani Nikula [Fri, 23 Sep 2011 18:57:38 +0000 (21:57 +0300)]
emacs: Make saving new saved searches append, not prepend

Append new saved searches at the end of saved searches rather than insert
in front.

Signed-off-by: Jani Nikula <>
9 years agoemacs: Add new customization option to sort saved searches
Jani Nikula [Fri, 23 Sep 2011 18:57:37 +0000 (21:57 +0300)]
emacs: Add new customization option to sort saved searches

Add new customization option notmuch-saved-search-sort-function to sort
saved searches in user-defined order. Provide a sort function to sort the
saved searches in alphabetical order. Setting the search function to nil
causes the saved searches not to be sorted, as before. This also remains
the default. The function only affects display of the saved searches, not
the order in which they are stored by custom.

Signed-off-by: Jani Nikula <>
9 years agotags_to_maildir_flags: Cleanup double assignement
Louis Rilling [Wed, 14 Sep 2011 22:23:19 +0000 (00:23 +0200)]
tags_to_maildir_flags: Cleanup double assignement

The for loop right after already does the job.

Signed-off-by: Louis Rilling <>
9 years agolib: Kill last usage of C++ type bool
Louis Rilling [Wed, 14 Sep 2011 22:23:18 +0000 (00:23 +0200)]
lib: Kill last usage of C++ type bool

Signed-off-by: Louis Rilling <>
9 years agoMerge branch 'release'
David Bremner [Mon, 21 Nov 2011 11:26:11 +0000 (07:26 -0400)]
Merge branch 'release'

9 years agofix a double // to single /
Florian Friesdorf [Sun, 20 Nov 2011 21:15:24 +0000 (13:15 -0800)]
fix a double // to single /

9 years agoemacs: breakout notmuch-show-advance functionality from notmuch-show-advance-and...
Jameson Graef Rollins [Sun, 13 Nov 2011 21:16:48 +0000 (13:16 -0800)]
emacs: breakout notmuch-show-advance functionality from notmuch-show-advance-and-archive

This patch breaks out much of the functionality of
notmuch-show-advance-and-archive into a new function:
notmuch-show-advance.  This new function does all the advancing
through a show buffer that notmuch-show-advance-and-archive did,
without all the invasive thread archiving.  The return value of
notmuch-show-advance is nil if the bottom of the thread is not
reached, and t if it is.

notmuch-show-advance-and-archive is modified to just call
notmuch-show-advance, and then call notmuch-show-archive-thread if the
return value is true.  In this way the previous functionality of
notmuch-show-advance-and-archive is preserved.

This provides a way for people to rebind the space bar to a more sane
function if they don't like the default behavior.

9 years agoDon't link libnotmuch if libutil isn't linked in properly.
Tom Prince [Sun, 13 Nov 2011 16:34:42 +0000 (11:34 -0500)]
Don't link libnotmuch if libutil isn't linked in properly.

For some reason, on my machine, the link is picking up
/usr/lib/ instead of util/libutil.a. This causes there to be
undefined symbols in libnotmuch, making it unuseable. This patch causes
the link to fail instead.

9 years agoRELEASING: document the semi-automated version propagation.
David Bremner [Fri, 18 Nov 2011 02:26:26 +0000 (22:26 -0400)]
RELEASING: document the semi-automated version propagation.

The instructions are purposely a bit coy about what files are updated,
so we don't have to update immediately when something else is plugged
into the make recipe.

9 years agobuild system: add target update-versions to propagate version
David Bremner [Fri, 18 Nov 2011 02:26:25 +0000 (22:26 -0400)]
build system: add target update-versions to propagate version

The version from file "version" is propagated to the man page and the
python bindings via sed. Note that the git version is ignored because
of the check for MAKECMDGOALS.

9 years agobuild system: use $(filter ...) to test MAKECMDGOALS
David Bremner [Fri, 18 Nov 2011 02:26:24 +0000 (22:26 -0400)]
build system: use $(filter ...) to test MAKECMDGOALS

This makes the test easier to extend to more targets. It also corrects
a bug where "special" targets were only detected when given alone.

9 years agoversion: update version info for 0.10~rc2 0.10_rc2 debian/0.10_rc2-1
David Bremner [Sat, 19 Nov 2011 12:23:40 +0000 (08:23 -0400)]
version: update version info for 0.10~rc2

Arguably editing debian/changelog violates the "do one thing at a
time" rule, but all of these versions need to be kept in sync.

9 years agotest: make all tests terminable with Ctrl-c
Tomi Ollila [Tue, 8 Nov 2011 16:02:25 +0000 (18:02 +0200)]
test: make all tests terminable with Ctrl-c

Some tests don't break when HUP signal is sent tho those (by
pressing ctrl-c on the terminal). Therefore, the top-level
test script catches the HUP and sends TERM signal to the
started test script.

9 years agotest: emacs: tidy up "Stashing in notmuch-show" test
Pieter Praet [Wed, 16 Nov 2011 11:38:19 +0000 (12:38 +0100)]
test: emacs: tidy up "Stashing in notmuch-show" test

Merge expected output into the actual test, so we can verify the stashed
filename using ${gen_msg_filename} instead of doing sed tricks.

9 years agotest: attempt to send QUIT to smtp-dummy in case mail send failed
Tomi Ollila [Wed, 16 Nov 2011 21:25:19 +0000 (23:25 +0200)]
test: attempt to send QUIT to smtp-dummy in case mail send failed

If mail sending from emacs fails before it has chance to connect
to the smtp-dummy mail server, the opportunistic QUIT message
sending makes smtp-dummy to exit.

9 years agotest: create dtach's session socket in $TEST_TMPDIR
Tomi Ollila [Wed, 16 Nov 2011 20:50:06 +0000 (22:50 +0200)]
test: create dtach's session socket in $TEST_TMPDIR

Due to 108-character limit in unix domain socket path this change
is required; it is more probable that length of ${TMPDIR:-/tmp} is
shorter than length of path to the current directory of notmuch test
source directory. One can expect to create reasonable-length unix
domain sockets wherever $TMPDIR points to.

9 years agotest: create TEST_TMPDIR for holding temporary files
Tomi Ollila [Wed, 16 Nov 2011 19:09:00 +0000 (21:09 +0200)]
test: create TEST_TMPDIR for holding temporary files

The TEST_TMPDIR if first needed to hold dtach's socket (due
to 108-character limit in socket file names). Later it can be
used to hold other temporary files; directory deleted at exit.

9 years agobuild system: tweak VERSION so that debian-snapshot works for N.NN~rcN
David Bremner [Thu, 17 Nov 2011 16:40:50 +0000 (12:40 -0400)]
build system: tweak VERSION so that debian-snapshot works for N.NN~rcN

The problem was that the version is recovered from the git tag, which
has the ~ replaced by _. This broke the sequencing of version numbers.

9 years agoNEWS: fix some old typos and trailing whitespace
Pieter Praet [Wed, 16 Nov 2011 20:37:50 +0000 (21:37 +0100)]
NEWS: fix some old typos and trailing whitespace

(with a further M-x whitespace-cleanup by db)

9 years agoNEWS: add entries for stashing-related keybinding and tests
Pieter Praet [Wed, 16 Nov 2011 20:37:49 +0000 (21:37 +0100)]
NEWS: add entries for stashing-related keybinding and tests

Add news entries for commits:

9 years agoNEWS entry for id-links
Daniel Schoepe [Wed, 16 Nov 2011 17:53:13 +0000 (18:53 +0100)]
NEWS entry for id-links

This adds a NEWS entry for commit 4a4ada73b751b1916c5dc4d408a8056411566e38

(second try, with whitespace fix)

9 years agoRevert "NEWS entry for id-links"
David Bremner [Wed, 16 Nov 2011 18:14:08 +0000 (14:14 -0400)]
Revert "NEWS entry for id-links"

This reverts commit e93bf1f842d4d06b2468d1cdbd5e4e25877c7a44.

reverting to fix whitespace

9 years agoNEWS entry for id-links
Daniel Schoepe [Wed, 16 Nov 2011 17:53:13 +0000 (18:53 +0100)]
NEWS entry for id-links

This adds a NEWS entry for commit 4a4ada73b751b1916c5dc4d408a8056411566e38

9 years agoNEWS entry for adding tab-completion to notmuch-search
Daniel Schoepe [Wed, 16 Nov 2011 17:53:12 +0000 (18:53 +0100)]
NEWS entry for adding tab-completion to notmuch-search

Add a news entry for commit6a280088e6769015ade7758b9790384997a21ff3.

9 years agoNEWS: mention improved search buffer performance, change Optimizations to Performance
David Bremner [Wed, 16 Nov 2011 17:05:19 +0000 (13:05 -0400)]
NEWS: mention improved search buffer performance, change Optimizations to Performance

via email: id:""

9 years agonotmuch.1: bump version number
David Bremner [Wed, 16 Nov 2011 11:44:00 +0000 (07:44 -0400)]
notmuch.1: bump version number

More of a leap than a bump.  This is a bit silly keeping 3 files
syncronized. At least for this file, I would prefer a solution that
generates notmuch.1 from some template at build time.

9 years agoRELEASING: update description of "make release".
David Bremner [Wed, 16 Nov 2011 11:41:44 +0000 (07:41 -0400)]
RELEASING: update description of "make release".

The additional "safety feature" documented here is motivated by the
fact that I use gpg-agent and I don't always get the GPG prompt that
Carl was relying on as an abort point. The new version also allows
more to be done in "dry run" mode.

9 years agoRELEASING: update discussion of version handling
David Bremner [Wed, 16 Nov 2011 11:25:57 +0000 (07:25 -0400)]
RELEASING: update discussion of version handling

This is definitely reaching the point where it should be automated.

9 years agoRELEASING: update symbol handling description for current practice.
David Bremner [Wed, 16 Nov 2011 11:21:35 +0000 (07:21 -0400)]
RELEASING: update symbol handling description for current practice.

We really did bump SONAME, and we probably will again, but not just
for a simple symbol addition.

Debian versions generally need to be removed from symbols file; this
wasn't a problem before because there was no Debian versions

9 years agoNEWS: discuss contrib and nmbug
David Bremner [Wed, 16 Nov 2011 03:06:38 +0000 (23:06 -0400)]
NEWS: discuss contrib and nmbug

I mention the possibility of a seperate license here because currently
notmuch-deliver is licensed GPL v2 only.

9 years agodebian/libnotmuch2.symbols: add notmuch_query_count_threads 0.10_rc1 debian/0.10_rc1-1
David Bremner [Wed, 16 Nov 2011 00:14:59 +0000 (20:14 -0400)]
debian/libnotmuch2.symbols: add notmuch_query_count_threads

Since this is only an added symbol, no soname bump required.

9 years agodebian: new changelog stanza for 0.10~rc1-1
David Bremner [Tue, 15 Nov 2011 23:49:50 +0000 (19:49 -0400)]
debian: new changelog stanza for 0.10~rc1-1

The changelog is a bit minimalist, but we'll do better for the real

9 years agoversion: update to 0.10~rc1
David Bremner [Tue, 15 Nov 2011 23:46:08 +0000 (19:46 -0400)]
version: update to 0.10~rc1

and the usual dance with the python bindings version.

9 years agotest: add tests for notmuch search --offset and --limit
Jani Nikula [Tue, 15 Nov 2011 20:08:53 +0000 (22:08 +0200)]
test: add tests for notmuch search --offset and --limit

Signed-off-by: Jani Nikula <>
9 years agotest: add tests for notmuch count
Jani Nikula [Tue, 15 Nov 2011 20:08:52 +0000 (22:08 +0200)]
test: add tests for notmuch count

Signed-off-by: Jani Nikula <>
9 years agocli: add support for --output parameter in notmuch count
Jani Nikula [Tue, 15 Nov 2011 20:08:51 +0000 (22:08 +0200)]
cli: add support for --output parameter in notmuch count

Add support for --output=messages (which remains the default) and
--output=threads to notmuch count.

Signed-off-by: Jani Nikula <>
9 years agocli: drop unused code from notmuch count
Jani Nikula [Tue, 15 Nov 2011 20:08:50 +0000 (22:08 +0200)]
cli: drop unused code from notmuch count

Remove unused code within #if 0 blocks from notmuch count.

Signed-off-by: Jani Nikula <>
9 years agocli: add options --offset and --limit to notmuch search
Jani Nikula [Tue, 15 Nov 2011 20:08:49 +0000 (22:08 +0200)]
cli: add options --offset and --limit to notmuch search

Add options --offset=[-]N and --limit=M to notmuch search to determine the
first result and maximum number of results to display.

Option --limit=M limits the maximum number of results to display to M.

Option --offset=[-]N skips the first N results; with the leading '-' skip
until the Nth result from the end.

Note that --offset with a negative N for thread or summary output requires
counting the number of matching threads in advance.

Signed-off-by: Jani Nikula <>
9 years agolib: add function to get the number of threads matching a search
Jani Nikula [Tue, 15 Nov 2011 20:08:48 +0000 (22:08 +0200)]
lib: add function to get the number of threads matching a search

Add function notmuch_query_count_threads() to get the number of threads
matching a search. This is done by performing a search and figuring out the
number of unique thread IDs in the matching messages, a significantly
heavier operation than notmuch_query_count_messages().

Signed-off-by: Jani Nikula <>
9 years agotest: Add a test script for "notmuch tag"
Austin Clements [Tue, 8 Nov 2011 03:57:11 +0000 (22:57 -0500)]
test: Add a test script for "notmuch tag"

9 years agonews: Store "from" and "subject" headers in the database.
Austin Clements [Tue, 15 Nov 2011 01:15:33 +0000 (20:15 -0500)]
news: Store "from" and "subject" headers in the database.

9 years agoStore "from" and "subject" headers in the database.
Austin Clements [Sun, 6 Nov 2011 17:17:36 +0000 (12:17 -0500)]
Store "from" and "subject" headers in the database.

This is a rebase and cleanup of Istvan Marko's patch from

Search retrieves these headers for every message in the search
results.  Previously, this required opening and parsing every message
file.  Storing them directly in the database significantly reduces IO
and computation, speeding up search by between 50% and 10X.

Taking full advantage of this requires a database rebuild, but it will
fall back to the old behavior for messages that do not have headers
stored in the database.

9 years agotest/atomicity: change shebang to '#!/usr/bin/env bash'
Tomi Ollila [Sun, 13 Nov 2011 08:52:35 +0000 (10:52 +0200)]
test/atomicity: change shebang to '#!/usr/bin/env bash'

9 years agoemacs: add notmuch-show-worker function for specifying crypto processing directly
Jameson Graef Rollins [Sun, 13 Nov 2011 19:15:21 +0000 (11:15 -0800)]
emacs: add notmuch-show-worker function for specifying crypto processing directly

The main reason to introduce this new unexposed function is to allow
the buffer redisplay crypto switch to behaving in a more expected way.
The prefix to notmuch-show-redisplay buffer now switches the crypto
processing of the current show buffer, as opposed to switching the
logic of the notmuch-crypto-process-mime customization variable.  This
behavior is more intuitive.

9 years agoemacs: add documentation for notmuch-show crypto-switch option
Jameson Graef Rollins [Sun, 13 Nov 2011 19:15:20 +0000 (11:15 -0800)]
emacs: add documentation for notmuch-show crypto-switch option

9 years agoLink libutil using filenmae, rather than using -l.
Tom Prince [Sun, 13 Nov 2011 17:05:03 +0000 (12:05 -0500)]
Link libutil using filenmae, rather than using -l.

glibc includes a libutil, so if the wrong -L options get passed, we
will pick up glibc's version, rather than our own.

9 years agotest: make smtp-dummy work with Emacs 24
Thomas Jost [Mon, 3 Oct 2011 16:47:27 +0000 (18:47 +0200)]
test: make smtp-dummy work with Emacs 24

In Emacs 24, a space is expected after a SMTP response code. If we don't respect
that, smtpmail-send-it will wait forever.

9 years agotest: do not hide test_emacs errors
Dmitry Kurochkin [Fri, 1 Jul 2011 02:23:48 +0000 (06:23 +0400)]
test: do not hide test_emacs errors

Do not redirect test_emacs stderr to /dev/null.  Test_emacs uses
emacsclient(1) now and it does not print unwanted messages (like
those from `message') to stderr.  But it does print useful
errors, e.g. when emacs server connection fails, given expression
is not valid or undefined function is called.

9 years agoexec emacs at the end of run_emacs script
Tomi Ollila [Thu, 10 Nov 2011 23:43:03 +0000 (01:43 +0200)]
exec emacs at the end of run_emacs script

In the last line of run_emacs, exec the emacs process.
With one fork less the process list is (also) neater.

9 years agocontrib/nmbug: new script for sharing tags with a given prefix.
David Bremner [Sun, 13 Nov 2011 01:24:25 +0000 (21:24 -0400)]
contrib/nmbug: new script for sharing tags with a given prefix.

The main idea is consider the notmuch database as analogous to the
work-tree. A bare git repo is maintained in the users home directory,
with a tree of the form tags/$message-id/$tag

Like notmuch and git, we have a set of subcommnds, mainly modelled on

Implementation wise, the heavy lifting is in the following functions.

   commit xapian -> git
   checkout git -> xapian
   merge fetched git + git -> xapian
   status find differences between xapian, git, and remote git.

The central implementation trick, from an idea I think due to
tomprince on IRC is manipulate the git index directly from the xapian
tag information.  The merge routine is still done using a temporary
checkout as I wasn't able to get it working with the index only.

There are also some convenience wrappers around git commands, like "fetch"
that essential just set GIT_DIR in the environment.

In order to encode tags (viewed as octet sequences) into filenames,
we whitelist a smallish set of characters and %hex escape anything outside.

The prefix is omitted in git, which lets one save and restore to
different prefixes (although this is only lightly tested).

Thanks to Tomi Ollila for a huge amount of feedback and patches while
putting this together.

9 years agoemacs: update notmuch-crypto-process-mime config variable documentation.
Jameson Graef Rollins [Tue, 31 May 2011 17:06:00 +0000 (10:06 -0700)]
emacs: update notmuch-crypto-process-mime config variable documentation.

This mentions the fact that prefix arguments are now used to enable to
crypto switch.

9 years agoemacs: Unbind M-RET as display of thread with crypto switch.
Jameson Graef Rollins [Sun, 29 May 2011 00:09:44 +0000 (17:09 -0700)]
emacs: Unbind M-RET as display of thread with crypto switch.

Use prefix argument instead to set switch.

9 years agotest: stashing in notmuch-{show,search}
Pieter Praet [Thu, 30 Jun 2011 08:23:23 +0000 (10:23 +0200)]
test: stashing in notmuch-{show,search}

Should provide full test coverage of the stashing feature.

Signed-off-by: Pieter Praet <>
9 years agoemacs: add keybind and function to stash Message-ID without prefix
Pieter Praet [Thu, 30 Jun 2011 08:20:50 +0000 (10:20 +0200)]
emacs: add keybind and function to stash Message-ID without prefix

Add function `notmuch-show-stash-message-id-stripped'
which stashes a Message-ID after ripping off the prefix and quotes,
add bind it to "I" key in `notmuch-show-stash-map'.

Simplifying `notmuch-show-get-message-id' instead might seem better,
but that would require concat'ing in 9 places instead of 1.

Signed-off-by: Pieter Praet <>
9 years agoNEWS: mention dtach instead of screen in the description of testing changes
David Bremner [Sat, 12 Nov 2011 15:46:13 +0000 (10:46 -0500)]
NEWS: mention dtach instead of screen in the description of testing changes

This brings NEWS back in line with the actual code.

9 years agoemacs: add invisible space after the search widget field in notmuch-hello
Dmitry Kurochkin [Mon, 4 Jul 2011 06:47:05 +0000 (10:47 +0400)]
emacs: add invisible space after the search widget field in notmuch-hello

It is very convenient when C-e (bound to `widget-end-of-line') ignores
trailing spaces inside the search widget.  But it only does so if a
widget is not followed by a newline (that is why it works in the saved
search widgets).  The patch just adds an invisible space after the
search widget to get the desirable behavior of `widget-end-of-line'.
The extra space is also added to expected results of emacs tests.

9 years agoemacs: Use a single buffer invisibility spec to fix quadratic search cost.
Austin Clements [Wed, 9 Nov 2011 02:55:28 +0000 (21:55 -0500)]
emacs: Use a single buffer invisibility spec to fix quadratic search cost.

Buffer redisplay requires traversing the buffer's invisibility spec
for every part of the display that has an 'invisible text or overlay
property.  Previously, the search buffer's invisibility spec list
contained roughly one entry for each search result.  As a result,
redisplay took O(NM) time where N is the number of visible lines and M
is the total number of results.  On a slow computer, this is enough to
make even buffer motion noticeably slow.  Worse, during a search
operation, redisplay is triggered for each search result (even if
there are no visible buffer changes), so search was quadratic
(O(NM^2)) in the number of search results.

This change switches to using a single element buffer invisibility
spec.  To un-hide authors, instead of removing an entry from the
invisibility spec, it simply removes the invisibility overlay from
those authors.

I tested using a query with 6633 results on a 9 year old machine.
Before this patch, Emacs took 70 seconds to fill the search buffer;
toward the end of the search, Emacs consumed 10-20x as much CPU as
notmuch; and moving point in the buffer took about a second.  With
this patch, the same query takes 40 seconds, Emacs consumes ~3x the
CPU of notmuch by the end, and there's no noticeable lag to moving
point.  (There's still some source of non-linearity, because Emacs and
notmuch consume roughly the same amount of CPU early in the search.)

9 years agosmtp-dummy: clear sockaddr_in structure before use in bind()
Tomi Ollila [Sat, 12 Nov 2011 09:57:22 +0000 (11:57 +0200)]
smtp-dummy: clear sockaddr_in structure before use in bind()

Any junk bytes in sockaddr_in structure before passing that
to bind() system call may cause problems.

9 years agodebian: update build dependency on dtach instead of screen
Jameson Graef Rollins [Fri, 11 Nov 2011 00:17:57 +0000 (16:17 -0800)]
debian: update build dependency on dtach instead of screen

This reflects a modification to the test suite to use dtach instead of

9 years agodebian: clean up Uploaders and Build-Depends fields in debian/control
Jameson Graef Rollins [Fri, 11 Nov 2011 00:17:56 +0000 (16:17 -0800)]
debian: clean up Uploaders and Build-Depends fields in debian/control

No functional change, but this will make for cleaner diffs down the