notmuch
7 years agolib/cli: pass GMIME_ENABLE_RFC2047_WORKAROUNDS to g_mime_init()
Jani Nikula [Wed, 11 Sep 2013 17:36:43 +0000 (20:36 +0300)]
lib/cli: pass GMIME_ENABLE_RFC2047_WORKAROUNDS to g_mime_init()

As explained by Jeffrey Stedfast, the author of GMime, quoted in [1]:

> Passing the GMIME_ENABLE_RFC2047_WORKAROUNDS flag to g_mime_init()
> *should* solve the decoding problem mentioned in the thread. This
> flag should be safe to pass into g_mime_init() without any bad side
> effects and my unit tests do test that code-path.

The thread being referred to is [2].

[1] id:87bo56viyo.fsf@nikula.org
[2] id:08cb1dcd-c5db-4e33-8b09-7730cb3d59a2@gmail.com

7 years agotest: add known broken tests for known broken RFC 2047 encodings
Jani Nikula [Wed, 11 Sep 2013 17:36:42 +0000 (20:36 +0300)]
test: add known broken tests for known broken RFC 2047 encodings

Some common broken RFC 2047 encodings that we currently let gmime
parse strictly. We could tell gmime to be forgiving in what it accepts
as RFC 2047 encoding, making these tests pass.

7 years agoemacs: Move ?, q, s, m, =, and G to the common keymap
Austin Clements [Tue, 3 Sep 2013 21:45:25 +0000 (17:45 -0400)]
emacs: Move ?, q, s, m, =, and G to the common keymap

The only user-visible effect of this should be that "G" now works in
show mode (previously it was unbound for no apparent reason).

This shared keymap gives us one place to put global commands, which
both forces us to think about what commands should be global, and
ensures their bindings can't diverge (like the missing "G" in show).

7 years agoemacs: Define a common shared keymap for all of notmuch
Austin Clements [Tue, 3 Sep 2013 21:45:24 +0000 (17:45 -0400)]
emacs: Define a common shared keymap for all of notmuch

This defines a single, currently empty keymap that all other notmuch
mode maps inherit from.

7 years agoemacs: Make notmuch-help work with arbitrary keymaps
Austin Clements [Tue, 3 Sep 2013 21:45:23 +0000 (17:45 -0400)]
emacs: Make notmuch-help work with arbitrary keymaps

This converts notmuch-help to use map-keymap for all keymap traversal.
This generally cleans up and simplifies construction of keymap
documentation, and also makes notmuch-help support anything that can
be in a keymap, including more esoteric stuff like multiple
inheritance.

7 years agoemacs: Add unified refresh-this-buffer function
Austin Clements [Tue, 3 Sep 2013 21:45:22 +0000 (17:45 -0400)]
emacs: Add unified refresh-this-buffer function

This unifies the various refresh and poll-and-refresh functions we
have for different modes.  Now all modes bind "=" and "G" (except
show, which doesn't bind "G" for some reason) to
`notmuch-refresh-this-buffer' and
`notmuch-poll-and-refresh-this-buffer', respectively.

7 years agoemacs: Move `notmuch-poll' to notmuch-lib
Austin Clements [Tue, 3 Sep 2013 21:45:21 +0000 (17:45 -0400)]
emacs: Move `notmuch-poll' to notmuch-lib

7 years agoemacs: Remove notmuch-search quit continuation
Austin Clements [Tue, 3 Sep 2013 21:45:20 +0000 (17:45 -0400)]
emacs: Remove notmuch-search quit continuation

Since notmuch-hello doesn't need this any more, we can remove this
hack.  This also eliminates `notmuch-search-quit', so now all modes
bind "q" to `notmuch-kill-this-buffer'.

7 years agoemacs: Bind "s" to notmuch-search in hello-mode
Austin Clements [Tue, 3 Sep 2013 21:45:19 +0000 (17:45 -0400)]
emacs: Bind "s" to notmuch-search in hello-mode

Since there is now no difference between notmuch-hello-search and
notmuch-search when called interactively, bind "s" to notmuch-search
in notmuch-hello-mode-map.  Now all modes bind "s" this way.

7 years agoemacs: Refresh hello whenever the user switches to the buffer
Austin Clements [Tue, 3 Sep 2013 21:45:18 +0000 (17:45 -0400)]
emacs: Refresh hello whenever the user switches to the buffer

Previously, we refreshed hello when the user quit a search that was
started from hello.  This is fine assuming purely stack-oriented
buffer use, but is quite fragile and requires hacks to search.

This replaces that logic with a new approach that refreshes hello
whenever the user switches to the hello buffer, regardless of how this
happens.

7 years agoemacs: Consistently use configured sort order
Austin Clements [Tue, 3 Sep 2013 21:45:17 +0000 (17:45 -0400)]
emacs: Consistently use configured sort order

Previously, if `notmuch-search' was called interactively (bound to "s"
in search and show, but not hello), it would always use newest-first.
However, `notmuch-hello-search' (bound to "s" in hello) and
`notmuch-hello-widget-search` would call it with the user-configured
sort order.  This inconsistency seems unintentional, so change
`notmuch-search' to use the user-configured sort order when called
interactively.

7 years agoemacs: add buttons for all multipart/related parts
Istvan Marko [Fri, 16 Aug 2013 20:24:23 +0000 (13:24 -0700)]
emacs: add buttons for all multipart/related parts

When text/html parts include images as multipart/related and the
text/plain alternative is used these images can be completely hidden
with no easy way to access them or even find out that they are there.

Make notmuch-show-insert-part-multipart/related add buttons for all
parts, the first one visible the rest hidden.

7 years agoemacs: show: lazy part handling bugfix
Mark Walters [Fri, 6 Sep 2013 23:28:45 +0000 (00:28 +0100)]
emacs: show: lazy part handling bugfix

The lazy part handler had a bug that it allowed the button to be
toggled to be specified. During toggling it needs to save and restore
the text-properties for the button but it actually saved the text
properties at point rather than from the button.

In almost all cases this didn't matter as as point had the same text
properties as the button. However, it is a bug and did cause incorrect
behaviour in some cases: see id:87txhz14z6.fsf@qmul.ac.uk for details.

7 years agonotmuch-mutt: Fix tagging issues
Kevin J. McCarthy [Sat, 7 Sep 2013 17:07:57 +0000 (10:07 -0700)]
notmuch-mutt: Fix tagging issues

This patch fixes three issues with "notmuch-mutt tag":

1. The message_id was not shell quoted.
   Thanks to Jason Miller for the bug report and patch.

2. The tags passed into tag_action() were not being properly quoted.
   The "join before shell_quote" was combining multiple tags into a
   single argument to notmuch tag: '+one -two -three' instead of
   '+one' '-two' '-three'.

3. A "--" was added between the tags and search-term as shown in the
   current notmuch-tag man page.

Thanks to Tomi Ollila for suggesting the simple fix of using
the list form of system(), which bypasses the shell.

7 years agotest: unset 'xpg_echo' bash shell option
Tomi Ollila [Tue, 20 Aug 2013 16:16:43 +0000 (19:16 +0300)]
test: unset 'xpg_echo' bash shell option

When 'xpg_echo' bash shell option is unset (usually the default)
echo builtin does not expand backslash-escape sequences by default
(i.e. '\n' is echoed as '\n' instead of newline). Not all bash
installations have this feature we depend on activated by default.

Note that the feature is bash (and GNU /bin/echo) specific. It is used
as it is convenient. If portability is needed (elsewhere) use printf(1)
(also often available as a shell builtin).

7 years agodebian: remove unneeded notmuch-mutt dependencies
Kevin McCarthy [Thu, 5 Sep 2013 02:05:51 +0000 (19:05 -0700)]
debian: remove unneeded notmuch-mutt dependencies

Switching away from fdupes removes the dependency on libfile-which-perl
and the need to recommend fdupes.

7 years agonotmuch-mutt: use notmuch --duplicate flag
Kevin McCarthy [Thu, 5 Sep 2013 02:05:50 +0000 (19:05 -0700)]
notmuch-mutt: use notmuch --duplicate flag

Change notmuch-mutt to use the new --duplicate=1 flag for duplicate
removal.  This will remove duplicates based on message-id at the
notmuch level.  Previously we were using fdupes or generating sha sums
after the search.

This version will be faster, but will enable the possibility of hiding
search results due to accidental/malicious duplicate message-ids.

7 years agoemacs: fix notmuch-mua-reply point placement when signature involved
Tomi Ollila [Fri, 6 Sep 2013 14:32:18 +0000 (17:32 +0300)]
emacs: fix notmuch-mua-reply point placement when signature involved

When composing a reply, notmuch-mua-reply attempts to  cite the
the original message by inserting it before the user signature, if
one is present. The existing method used to search the signature
separator backward from the end of the buffer and then move one
line up. In case of variable `message-signature-insert-empty-line'
being nil this caused point to go to the beginning of
'--text follows this line--'
separator line, and citation was inserted there.
This change checks the value of `message-signature-insert-empty-line'
and doesn't move point if that is nil. Additional narrowing to
the body region ensures that point never goes to the separator line
(or beyond).
`message-signature-setup-hook' or `message-setup-hook' may already have
added some other content to the message body, therefore using simply
(message-goto-body) to move point to the beginning of body might lead
to unexpected results.

Original patch from "Geoffrey H. Ferrari", continued with iterations
from Jani and Mark.

7 years agotest: exit with nonzero value when not all tests completed successfully
Tomi Ollila [Sun, 8 Sep 2013 15:53:30 +0000 (18:53 +0300)]
test: exit with nonzero value when not all tests completed successfully

If any of the tests in our test system is not passing the execution
of the test suite completes with nonzero exit value.

It is better to rely on the exit value of the test system instead
of some arbitrary strings in test output (or use both).

7 years agoemacs: insert quotable parts in reply as they are displayed in show view
Jani Nikula [Sun, 1 Sep 2013 17:59:53 +0000 (20:59 +0300)]
emacs: insert quotable parts in reply as they are displayed in show view

In reply, insert quotable parts using notmuch-show-insert-bodypart
instead of calling notmuch-mm-display-part-inline directly to render
the quoted parts as they are rendered in show view.

We use a temp buffer to not leak text properties from the show
renderer into the reply. This way we also don't need to worry about
narrowing or point placement. Credits to Mark Walters
<markwalters1009@gmail.com> and Austin Clements <amdragon@MIT.EDU> for
getting this part straight.

The notable change is that replies to text/calendar parts quote the
pretty printed output of icalendar-import-buffer rather than the ugly
raw vcalendar.

7 years agocontrib: pick: test: buffer movement
Mark Walters [Tue, 3 Sep 2013 14:05:00 +0000 (15:05 +0100)]
contrib: pick: test: buffer movement

7 years agocontrib: pick: test: stash message id
Mark Walters [Tue, 3 Sep 2013 14:04:59 +0000 (15:04 +0100)]
contrib: pick: test: stash message id

This is important to test as lots of subsequent tests will fail if this fails.

7 years agocontrib: pick: test: tag/untag whole thread
Mark Walters [Tue, 3 Sep 2013 14:04:58 +0000 (15:04 +0100)]
contrib: pick: test: tag/untag whole thread

7 years agocontrib: pick: test: tag/untag single message
Mark Walters [Tue, 3 Sep 2013 14:04:57 +0000 (15:04 +0100)]
contrib: pick: test: tag/untag single message

7 years agotest: improve insert test reliability by checking message-id instead of count
Jani Nikula [Tue, 3 Sep 2013 05:19:03 +0000 (08:19 +0300)]
test: improve insert test reliability by checking message-id instead of count

There isn't a reported issue this would fix. Spotted by reading the
test.

7 years agotest: add more maildir flag syncing related tests to insert
Jani Nikula [Tue, 3 Sep 2013 05:19:02 +0000 (08:19 +0300)]
test: add more maildir flag syncing related tests to insert

Specifically test maildir flag syncing with insert.

7 years agotags_to_maildir_flags: Don't rename if no flags change
Louis Rilling [Wed, 19 Dec 2012 21:32:27 +0000 (22:32 +0100)]
tags_to_maildir_flags: Don't rename if no flags change

notmuch_message_tags_to_maildir_flags() unconditionally moves messages from
maildir directory "new/" to maildir directory "cur/", which makes messages lose
their "new" status in the MUA. However some users want to keep this "new"
status after, for instance, an auto-tagging of new messages.

However, as Austin mentioned and according to the maildir specification,
messages living in "new/" are not allowed to have flags, even if mutt allows it
to happen. For this reason, this patch prevents moving messages from "new/" to
"cur/", only if no flags have to be changed. It's hopefully enough to satisfy
mutt (and maybe other MUAs showing the "new" status) users checking the "new"
status.

Changelog:
* v2: Fix bool type as well as NULL returned despite having no errors (Austin
      Clements)
* v4: Tag the related test (contributed by Michal Sojka) as working

Signed-off-by: Louis Rilling <l.rilling@av7.net>
[Condition for keeping messages in new/ was extended to satisfy all
 tests from the previous patch. -Michal Sojka]

[Added by David Bremner, to keep the tests passing at each commit]

update insert tests for new maildir synchronization rules

As of id:1355952747-27350-4-git-send-email-sojkam1@fel.cvut.cz
we are more conservative about moving messages from ./new to ./cur.
This updates the insert tests to match

7 years agotest: Add some missing maildir synchronization tests
Michal Sojka [Wed, 19 Dec 2012 21:32:26 +0000 (22:32 +0100)]
test: Add some missing maildir synchronization tests

As mentioned by Jani Nikula in id:87vcccp4y3.fsf@nikula.org, some cases
of maildir synchronization are not covered by our tests. Let's add the
missing tests.

7 years agotest: Adding non-maildir tags does not move message from new to cur
Michal Sojka [Wed, 19 Dec 2012 21:32:25 +0000 (22:32 +0100)]
test: Adding non-maildir tags does not move message from new to cur

Some MUA's like mutt show the difference between "new" emails living in maildir
directory new/, and "old" emails living in maildir directory cur/. However
notmuch tag unconditionally moves selected messages from new/ to cur/, even if
no maildir synchronized tag is changed.

While maildir specification forbids messages with tags living in new/, there is
no need to move messages to cur/ when no maildir synchronized tag is changed.
Thus notmuch can remain transparent with respect to other MUA's.

[ Edited commit log to better describe the intended changes, and tag the
  test as broken until the actual changes are implemented -- Louis Rilling ]

Signed-off-by: Louis Rilling <l.rilling@av7.net>
[ Converted to use test_subtest_known_broken, David Bremner ]

7 years agocontrib: pick: test: refresh view
Mark Walters [Mon, 2 Sep 2013 20:15:14 +0000 (21:15 +0100)]
contrib: pick: test: refresh view

A recent proposed patch was buggy when refreshing the view. Add a test
for refresh so that this does not reoccur.

7 years agocontrib: pick: add docstring for notmuch-pick-worker
Mark Walters [Mon, 2 Sep 2013 20:15:13 +0000 (21:15 +0100)]
contrib: pick: add docstring for notmuch-pick-worker

7 years agocontrib: pick: remove unneeded variable notmuch-pick-buffer-name
Mark Walters [Mon, 2 Sep 2013 20:15:12 +0000 (21:15 +0100)]
contrib: pick: remove unneeded variable notmuch-pick-buffer-name

This variable is essentially unused: it was only used for making sure
it itself got reset after a refresh of the buffer.

It did this by passing an unnecessary argument to notmuch-pick-worker
so remove that too.

7 years agocontrib: pick: bind M-p and M-n to prev/next thread
Mark Walters [Sun, 25 Aug 2013 19:55:34 +0000 (20:55 +0100)]
contrib: pick: bind M-p and M-n to prev/next thread

This adds functions to go to the previous or next thread to
pick. Prev-thread behaves similarly to prev-message in show: if you
are on the top line of a thread it will go to the top of the previous
thread, otherwise it will go to the top of the current thread. Next
thread will always go to the top of the next thread (or the end of
buffer). These are bound to "M-p" and "M-n" by default (matching the
bindings in show).

7 years agocontrib: pick: thread tagging (including archiving) implemented
Mark Walters [Sun, 25 Aug 2013 19:55:33 +0000 (20:55 +0100)]
contrib: pick: thread tagging (including archiving) implemented

Previously pick had no actions based on the entire thread: this adds
some. Note in this version '*' is bound to `tag thread' which is not
consistent with search or show. However it still might be the most
natural thing (as it is similar to running * in the show pane).

7 years agocontrib: pick: add thread based utility functions
Mark Walters [Sun, 25 Aug 2013 19:55:32 +0000 (20:55 +0100)]
contrib: pick: add thread based utility functions

Previously notmuch-pick had no thread based functionality. This adds a
macro to iterate through all messages in a thread. To simplify this it
adds a text-property marker to the first message of each thread.

7 years agolib: fix clang build
Jani Nikula [Sat, 17 Aug 2013 21:30:01 +0000 (00:30 +0300)]
lib: fix clang build

Long story short, fix build on recent (3.2+) clang.

The long story for posterity follows.

gcc 4.6 added new warnings about structs with greater visibility than
their fields. The warnings were silenced by adjusting visibility in

commit d5523ead90b6be2b07d4af745b8ed9b980a6b9f1
Author: Carl Worth <cworth@cworth.org>
Date:   Wed May 11 13:23:13 2011 -0700

    Mark some structures in the library interface with visibility=default attribute.

Later on,

commit 3b76adf9e2c026dd03b820f4c6eab50e25444113
Author: Austin Clements <amdragon@MIT.EDU>
Date:   Sat Jan 14 19:17:33 2012 -0500

    lib: Add support for automatically excluding tags from queries

changed visibility of struct _notmuch_string_list for the same reason, and

commit 1a53f9f116fa7c460cda3df532be921baaafb082
Author: Mark Walters <markwalters1009@gmail.com>
Date:   Thu Mar 1 22:30:38 2012 +0000

    lib: Add the exclude flag to notmuch_query_search_threads

split the struct _notmuch_string_list and its typedef
notmuch_string_list_t as a way to make a forward declaration for
_notmuch_thread_create().

The subtle difference was that the struct definition now had 'visible'
in it, while the typedef didn't, and it was within the #pragma GCC
visibility push(hidden) block. This went unnoticed, as the then common
versions of clang didn't care about this.

A later change in clang (I did not dig into when this change was
introduced) caused the following error:

CXX  -O2 lib/database.o
In file included from lib/database.cc:21:
In file included from ./lib/database-private.h:33:
./lib/notmuch-private.h:479:8: error: visibility does not match previous declaration
struct visible _notmuch_string_list {
       ^
./lib/notmuch-private.h:67:33: note: expanded from macro 'visible'
                                ^
./lib/notmuch-private.h:52:13: note: previous attribute is here
            ^
1 error generated.
make: *** [lib/database.o] Error 1

This is slightly misleading due to the reference to the #pragma. The
real culprit is the typedef within the #pragma.

We could just add 'visible' to the typedef, or move the typedef
outside of the #pragma, and be done with it, but juggle the
declarations a bit to accommodate moving the typedef back with the
struct, and keep the visibility attribute in one place.

The problem was originally reported by Simonas Kazlauskas
<s@kazlauskas.me> in id:20130418102507.GA23688@godbox but I was only
able to reproduce and investigate now that I upgraded clang.

7 years agocontrib: pick: remove sync tests
Mark Walters [Mon, 26 Aug 2013 19:57:23 +0000 (20:57 +0100)]
contrib: pick: remove sync tests

Since the previous commit removed the sync parser we may as well
remove its tests too.

7 years agocontrib: pick: remove sync parser
Mark Walters [Mon, 26 Aug 2013 19:57:22 +0000 (20:57 +0100)]
contrib: pick: remove sync parser

Previously pick had the option of using an async parser like search or
a sync parser like show. The async parser has always been the default
and it seems fine so we can remove the sync one and the corresponding
defcustom.

7 years agotest: test notmuch show --include-html option
Tomi Ollila [Sat, 24 Aug 2013 15:29:30 +0000 (18:29 +0300)]
test: test notmuch show --include-html option

Test new --include-html option added to notmuch show command with
json output message parts containing text in latin1 and utf8 format.

7 years agocli: add --include-html option to notmuch show
John Lenz [Tue, 2 Jul 2013 00:19:42 +0000 (19:19 -0500)]
cli: add --include-html option to notmuch show

For my client, the largest bottleneck for displaying large threads is
exporting each html part individually since by default notmuch will not
show the json parts.  For large threads there can be quite a few parts and
each must be exported and decoded one by one.  Also, I then have to deal
with all the crazy charsets which I can do through a library but is a
pain.

Therefore, this patch adds an --include-html option that causes the
text/html parts to be included as part of the output of show.

diff man/man1/notmuch-show.1

7 years agoemacs: removed 3 duplicate functions from notmuch-show.el
Tomi Ollila [Mon, 26 Aug 2013 20:21:57 +0000 (23:21 +0300)]
emacs: removed 3 duplicate functions from notmuch-show.el

notmuch-show.el and notmuch.el had 3 duplicate, identical functions:
notmuch-foreach-mime-part, notmuch-count-attachments and
notmuch-save-attachments. Now these functions in notmuch-show.el
are replaced with declare-functions pointing to "notmuch"(.el).

7 years agoman: clarify folder: prefix regarding duplicate files
Jani Nikula [Sat, 17 Aug 2013 12:11:33 +0000 (15:11 +0300)]
man: clarify folder: prefix regarding duplicate files

The results for folder: prefix are a source of recurring confusion.

7 years agoNEWS: cli options for improved duplicate message handling
Jani Nikula [Sun, 18 Aug 2013 12:52:28 +0000 (15:52 +0300)]
NEWS: cli options for improved duplicate message handling

7 years agoman: document notmuch count --output=files
Jani Nikula [Sat, 17 Aug 2013 12:11:31 +0000 (15:11 +0300)]
man: document notmuch count --output=files

Document the notmuch count --output=files option.

7 years agotest: test notmuch count --output=files
Jani Nikula [Sat, 17 Aug 2013 12:11:30 +0000 (15:11 +0300)]
test: test notmuch count --output=files

Add tests for notmuch count --output=files option.

7 years agocli: add --output=files option to notmuch count
Jani Nikula [Sat, 17 Aug 2013 12:11:29 +0000 (15:11 +0300)]
cli: add --output=files option to notmuch count

Add support for querying the total number of files associated with the
messages matching the search. This is mostly useful with an
id:<message-id> query for a single message.

7 years agoman: document notmuch search --duplicate=N
Jani Nikula [Sat, 17 Aug 2013 12:11:28 +0000 (15:11 +0300)]
man: document notmuch search --duplicate=N

Document the notmuch search --duplicate=N option.

7 years agotest: test notmuch search --duplicate=N
Jani Nikula [Sat, 17 Aug 2013 12:11:27 +0000 (15:11 +0300)]
test: test notmuch search --duplicate=N

Add test for notmuch search --duplicate=N option.

7 years agocli: add --duplicate=N option to notmuch search
Jani Nikula [Sat, 17 Aug 2013 12:11:26 +0000 (15:11 +0300)]
cli: add --duplicate=N option to notmuch search

Effective with --output=files, output the Nth filename associated with
each message matching the query (N is 1-based). If N is greater than
the number of files associated with the message, don't print anything.

7 years agonotmuch-config: use strchr(), not index() (Solaris support)
Blake Jones [Wed, 14 Nov 2012 02:10:03 +0000 (18:10 -0800)]
notmuch-config: use strchr(), not index() (Solaris support)

notmuch-config.c has the only use of the function named "index()" in the
notmuch source.  Several other places use the equivalent function
"strchr()"; this patch just fixes notmuch-config.c to use strchr()
instead.  (Solaris needs to include <strings.h> to get the prototype for
index(), and notmuch-config.c was failing to include that header, so it
wasn't compiling as-is.)

7 years agocontrib: pick: use close-message-pane for reply etc
Mark Walters [Thu, 22 Aug 2013 17:10:26 +0000 (18:10 +0100)]
contrib: pick: use close-message-pane for reply etc

We can save some code duplication by using the new close-message-pane
functionality for reply, forward, and new mail.

7 years agocontrib: pick: add in binding to view raw message
Mark Walters [Thu, 22 Aug 2013 17:10:25 +0000 (18:10 +0100)]
contrib: pick: add in binding to view raw message

Note this does rely on the fact that we have over-ridden notmuch-show-get-properties

7 years agocontrib: pick: make help close the message pane first
Mark Walters [Thu, 22 Aug 2013 17:10:24 +0000 (18:10 +0100)]
contrib: pick: make help close the message pane first

Previously pressing "?" for help when the message pane was open meant
the help window was very small. Close the message pane before
displaying help.

7 years agocontrib: pick: close window function
Mark Walters [Thu, 22 Aug 2013 17:10:23 +0000 (18:10 +0100)]
contrib: pick: close window function

7 years agocontrib: pick: pass tab through to the message pane
Mark Walters [Thu, 22 Aug 2013 17:10:22 +0000 (18:10 +0100)]
contrib: pick: pass tab through to the message pane

This makes tab move to next button in the message pane and binds
button activate (in message pane) to "e". This means that is easy to
toggle hidden parts or hidden citations etc in the message pane.

7 years agocontrib: pick: add button press helper
Mark Walters [Thu, 22 Aug 2013 17:10:21 +0000 (18:10 +0100)]
contrib: pick: add button press helper

We will want to be able to activate buttons not in the current
buffer (ie in the message pane) so it is helpful to have a way of
activating a button without signalling error if there is no button.

7 years agocontrib: pick: add in to-message-window function
Mark Walters [Thu, 22 Aug 2013 17:10:20 +0000 (18:10 +0100)]
contrib: pick: add in to-message-window function

7 years agocontrib: pick: Link in stash map straight from notmuch-show
Mark Walters [Thu, 22 Aug 2013 17:10:19 +0000 (18:10 +0100)]
contrib: pick: Link in stash map straight from notmuch-show

These functions all now work straight from their notmuch-show
implementation so link them in.

Stash functionality was one of the key missing things in notmuch-pick.

7 years agocontrib: pick: Link in attachment functions straight from notmuch-show
Mark Walters [Thu, 22 Aug 2013 17:10:18 +0000 (18:10 +0100)]
contrib: pick: Link in attachment functions straight from notmuch-show

We can use the attachment functions straight from
notmuch-show. notmuch-show-view-all-mime-parts might be deprecated so
we either want to undeprecate it or not have this binding.

7 years agocontrib: pick: Link in notmuch-show-pipe-message
Mark Walters [Thu, 22 Aug 2013 17:10:17 +0000 (18:10 +0100)]
contrib: pick: Link in notmuch-show-pipe-message

Since we can now use show functions directly in pick we can drop pick-pipe-message.

7 years agocontrib: pick: override notmuch-show-get-prop
Mark Walters [Thu, 22 Aug 2013 17:10:16 +0000 (18:10 +0100)]
contrib: pick: override notmuch-show-get-prop

We override notmuch-show-get-prop so that many of the show functions
can be used in notmuch-pick without modification. The main use is that
it means notmuch-show-get-message-id `works' in pick. Thus we get all
the stash functions and several other `for free' in pick.

7 years agotimegm: add portable implementation (Solaris support)
Blake Jones [Wed, 21 Aug 2013 11:09:54 +0000 (13:09 +0200)]
timegm: add portable implementation (Solaris support)

The timegm(3) function is a non-standard extension to libc which is
available in GNU libc and on some BSDs.  Although SunOS had this
function in its libc, Solaris (unfortunately) removed it.  This patch
implements a very simple version of timegm() which is good enough for
parse-time-string.c.

Signed-off-by: Vladimir Marek <vlmarek@volny.cz>
7 years agostrsep: check for availability (Solaris support)
Blake Jones [Fri, 16 Aug 2013 14:38:16 +0000 (16:38 +0200)]
strsep: check for availability (Solaris support)

Solaris does not ship a version of the strsep() function.  This change
adds a check to "configure" to see whether notmuch needs to provide its
own implementation, and if so, it uses the new version in
"compat/strsep.c" (which was copied from Mutt, and apparently before
that from glibc).

Signed-off-by: Vladimir Marek <vlmarek@volny.cz>
7 years agoasctime: check for standards compliance (Solaris support)
Vladimir Marek [Fri, 16 Aug 2013 14:38:15 +0000 (16:38 +0200)]
asctime: check for standards compliance (Solaris support)

Add checks to "configure" to see whether _POSIX_PTHREAD_SEMANTICS needs
to be defined to get the right number of arguments in the prototypes for
asctime_r().  Solaris' default implementation conforms to POSIX.1c
Draft 6, rather than the final POSIX.1c spec.  The standards-compliant
version can be used by defining _POSIX_PTHREAD_SEMANTICS.

This change also adds the file "compat/check_asctime.c", which
configure uses to perform its check, and modifies compat/compat.h to
define _POSIX_PTHREAD_SEMANTICS if configure detected it was needed.

Signed-off-by: Vladimir Marek <vlmarek@volny.cz>
7 years agogetpwuid: check for standards compliance (Solaris support)
Blake Jones [Fri, 16 Aug 2013 14:38:14 +0000 (16:38 +0200)]
getpwuid: check for standards compliance (Solaris support)

Add checks to "configure" to see whether _POSIX_PTHREAD_SEMANTICS needs
to be defined to get the right number of arguments in the prototypes for
getpwuid_r().  Solaris' default implementation conforms to POSIX.1c
Draft 6, rather than the final POSIX.1c spec.  The standards-compliant
version can be used by defining _POSIX_PTHREAD_SEMANTICS.

This change also adds the file "compat/check_getpwuid.c", which
configure uses to perform its check, and modifies compat/compat.h to
define _POSIX_PTHREAD_SEMANTICS if configure detected it was needed.

Signed-off-by: Vladimir Marek <vlmarek@volny.cz>
7 years agotest: Canonicalize RFC 2047 encoding and charset
Austin Clements [Mon, 19 Aug 2013 14:40:03 +0000 (10:40 -0400)]
test: Canonicalize RFC 2047 encoding and charset

RFC 2047 states that the encoding and charset in an encoded word are
case-insensitive, so force them to lower case in the reply test.  This
fixes an issue caused by GMime versions (somewhere between 2.6.10 and
2.6.16), which changed the capitalization of the encoding.

7 years agoreply: Use RFC 2822/MIME wholly for text format template
Austin Clements [Fri, 16 Aug 2013 15:35:43 +0000 (11:35 -0400)]
reply: Use RFC 2822/MIME wholly for text format template

Previously, reply's default text format used an odd mix of RFC 2045
MIME encoding for the reply template's body and some made-up RFC
2822-like UTF-8 format for the headers.  The intent was to present the
headers to the user in a nice, un-encoded format, but this assumed
that whatever ultimately sent the email would RFC 2047-encode the
headers, while at the same time the body was already RFC 2045 encoded,
so it assumed that whatever sent the email would *not* re-encode the
body.

This can be fixed by either producing a fully decoded UTF-8 reply
template, or a fully encoded MIME-compliant RFC 2822 message.  This
patch does the latter because it is

a) Well-defined by RFC 2822 and MIME (while any UTF-8 format would be
   ad hoc).

b) Ready to be piped to sendmail.  The point of the text format is to
   be minimal, so a user should be able to pop up the template in
   whatever editor they want, edit it, and push it to sendmail.

c) Consistent with frontend capabilities.  If a frontend has the
   smarts to RFC 2047 encode the headers before sending the mail, it
   probably has the smarts to RFC 2047 decode them before presenting
   the template to a user for editing.

Also, as far as I know, nothing automated consumes the reply text
format, so changing this should not cause serious problems.  (And if
anything does still consume this format, it probably gets these
encoding issues wrong anyway.)

7 years agoreply: Remove extraneous space from generated References
Austin Clements [Fri, 16 Aug 2013 15:35:42 +0000 (11:35 -0400)]
reply: Remove extraneous space from generated References

Previously, the References header code seemed to assume
notmuch_message_get_header would return NULL if the header was not
present, but it actually returns "".  As a result of this, it was
inserting an unnecessary space when concatenating an empty or missing
original references header with the new reference.

This shows up in only two tests because the text reply format later
passes the whole reply template through g_mime_filter_headers, which
has the side effect of stripping out this extra space.

7 years agoreply: Test replying to messages with RFC 2047-encoded headers
Austin Clements [Fri, 16 Aug 2013 15:35:41 +0000 (11:35 -0400)]
reply: Test replying to messages with RFC 2047-encoded headers

7 years agotest: Make symbol-test depend on libnotmuch.so
Austin Clements [Tue, 13 Aug 2013 19:52:17 +0000 (15:52 -0400)]
test: Make symbol-test depend on libnotmuch.so

Without this
$ make -j test
intermittently fails and
$ make clean; make test/symbol-test
always fails (not that anybody would do the latter).

7 years agoreply: Document the reason for g_mime_filter_headers
Austin Clements [Mon, 12 Aug 2013 18:40:36 +0000 (14:40 -0400)]
reply: Document the reason for g_mime_filter_headers

Given how long it took me to figure out why we pass the reply headers
through g_mime_filter_headers, it's worth a comment.

7 years agolib: Document which strings are returned in UTF-8
Austin Clements [Mon, 12 Aug 2013 18:40:35 +0000 (14:40 -0400)]
lib: Document which strings are returned in UTF-8

Any string that ultimately comes from notmuch_message_file_get_header
is in UTF-8.

7 years agolib: Correct out-of-date doc comment
Austin Clements [Mon, 12 Aug 2013 18:40:34 +0000 (14:40 -0400)]
lib: Correct out-of-date doc comment

notmuch_message_get_header started returning some headers straight
from the database in 567bcbc, but this comment explicitly claimed all
headers were read from the message file.

7 years agorelease: synchronize dates 0.16
David Bremner [Sat, 3 Aug 2013 11:29:40 +0000 (08:29 -0300)]
release: synchronize dates

man pages, NEWS, and debian changelog all hardcode date.  Make them
hardcode the same date.

7 years agoemacs: bugfix attachment content-type as mime-type handling 0.16_rc2
Mark Walters [Wed, 31 Jul 2013 18:39:05 +0000 (19:39 +0100)]
emacs: bugfix attachment content-type as mime-type handling

Notmuch puts attachments in as declared content-type except when the
content-type is application/octet-stream it tries to guess the type
from the filename/extension. This means that viewing a pdf (for
example) which is sent as application/octet-strem invokes the pdf
viewer rather than just offering to save the part.

Recent changes to the attachment handling (commit 1546387d) changed
(broke) this. This patch stores the calculated mime-type with the part
and changes the attachment part handlers can use it instead.

7 years agodebian: rename notmuch-ruby to ruby-notmuch 0.16_rc1
David Bremner [Sun, 28 Jul 2013 17:08:00 +0000 (14:08 -0300)]
debian: rename notmuch-ruby to ruby-notmuch

This seems more in line with

http://wiki.debian.org/Teams/Ruby/Packaging#Guidelines_for_Ruby_packaging

7 years agodebian: build notmuch-vim again
David Bremner [Sun, 28 Jul 2013 16:48:02 +0000 (13:48 -0300)]
debian: build notmuch-vim again

Based on id:1370220299-14722-1-git-send-email-felipe.contreras@gmail.com

Hacked rather extensively by db. The most important changes:

       - bring back notmuch.yaml for the (debian specific?) vim-addons
         tool.
       - depend on vim-ruby, so we get a version of vim with ruby installed.

7 years agodebian: update symbols for libnotmuch
David Bremner [Sun, 28 Jul 2013 14:08:50 +0000 (11:08 -0300)]
debian: update symbols for libnotmuch

Adding a new symbol should require an SONAME bump.

7 years agodebian: add alot to recommends
David Bremner [Sun, 28 Jul 2013 13:57:45 +0000 (10:57 -0300)]
debian: add alot to recommends

Since this is in a disjunction, this should not force new packages to
be installed, but rather let people with auto-install-recommends (the
default) on install notmuch without emacs.

7 years agodebian: note that we now provide a notmuch-dbg package
David Bremner [Sun, 28 Jul 2013 13:54:30 +0000 (10:54 -0300)]
debian: note that we now provide a notmuch-dbg package

This is already in upstream for a bit, but we lacked a changelog entry
closing the bug.

7 years agodebian: close bug requesting packaging of notmuch-deliver
David Bremner [Sun, 28 Jul 2013 13:49:49 +0000 (10:49 -0300)]
debian: close bug requesting packaging of notmuch-deliver

notmuch insert is pretty much equivalent, so no need to support both.

7 years agodebian: start new changelog entry
David Bremner [Sun, 28 Jul 2013 13:22:52 +0000 (10:22 -0300)]
debian: start new changelog entry

Since Debian uses the changelog as metadata, we need an empty stanza
to build the new version.

7 years agoversion: set new version and release date
David Bremner [Sun, 28 Jul 2013 14:18:48 +0000 (11:18 -0300)]
version: set new version and release date

These are manually set in version and NEWS, and propagate to the other files via
"make update-versions"

7 years agoPut 0.16 NEWS in a reasonable order
Austin Clements [Sun, 28 Jul 2013 04:16:01 +0000 (00:16 -0400)]
Put 0.16 NEWS in a reasonable order

The 0.16 NEWS grew chronologically during development, and as a result
wound up in a particularly odd order.  This rearranges it to put the
most user-visible news first.  Roughly: new features, modified
behavior, bug fixes, then deprecation, with related items grouped.

This does not modify the text of any of the news.

7 years agoemacs: hello: make --batch error gracefully
Mark Walters [Thu, 4 Jul 2013 22:18:19 +0000 (23:18 +0100)]
emacs: hello: make --batch error gracefully

Recently notmuch-hello was converted to use batch count. However, it
seems that several people run different versions of notmuch-emacs and
notmuch-cli so this batch makes emacs fail with an error message if
--batch is not available in the CLI.

Amended by: db

7 years agoemacs: News for S-expressions everywhere
Austin Clements [Mon, 22 Jul 2013 02:52:00 +0000 (22:52 -0400)]
emacs: News for S-expressions everywhere

Update the news about search using S-expression support to also say
that show is also now faster.

7 years agocontrib: pick: use notmuch-start-notmuch
Mark Walters [Thu, 4 Jul 2013 07:09:48 +0000 (08:09 +0100)]
contrib: pick: use notmuch-start-notmuch

This uses the new notmuch-start-notmuch function which should give
better handling of stderr and errors generally.

7 years agocrypto: return NULL cryptoctx if protocol string is empty.
Jameson Graef Rollins [Fri, 19 Jul 2013 15:36:12 +0000 (08:36 -0700)]
crypto: return NULL cryptoctx if protocol string is empty.

Badly formed messages that don't specify a protocol in
signed/encrypted parts, end up with a protocol of NULL.  strcasecmp in
notmuch_crypto_get_context then segfaults when trying to check it
against known protocols.  If the protocol is NULL, just return an
empty context immediately (with appropriate message.)

7 years agoNEWS: Python bindings: Fix loading of libnotmuch on OS X (Darwin)
Tomi Ollila [Sat, 13 Jul 2013 08:46:23 +0000 (11:46 +0300)]
NEWS: Python bindings: Fix loading of libnotmuch on OS X (Darwin)

7 years agoemacs: dropped rest of now-unused JSON functionality
Tomi Ollila [Sat, 6 Jul 2013 12:49:51 +0000 (15:49 +0300)]
emacs: dropped rest of now-unused JSON functionality

Notmuch cli provides all structured data previously provided
in json format now in s-expression format, rendering all current
json functionality obsolete.

7 years agoNEWS: announce insert command
Peter Wang [Sat, 6 Jul 2013 02:36:28 +0000 (12:36 +1000)]
NEWS: announce insert command

7 years agodebian: Add package for debugging symbols (Closes: #717339)
Daniel Kahn Gillmor [Fri, 19 Jul 2013 20:50:09 +0000 (16:50 -0400)]
debian: Add package for debugging symbols (Closes: #717339)

7 years agoPython bindings: CDLL("libnotmuch.3.dylib") on Darwin
Tomi Ollila [Tue, 25 Jun 2013 14:36:56 +0000 (17:36 +0300)]
Python bindings: CDLL("libnotmuch.3.dylib") on Darwin

Use os.uname() to check for 'Darwin' and load "libnotmuch.3.dylib"
instead of "libnotmuch.so.3" if that is the case.

7 years agocontrib: pick: document several of the tree/forest insertion functions.
Mark Walters [Mon, 1 Jul 2013 20:05:43 +0000 (21:05 +0100)]
contrib: pick: document several of the tree/forest insertion functions.

7 years agopick: contrib: document notmuch-pick-message-window-kill-hook
Mark Walters [Mon, 1 Jul 2013 20:05:42 +0000 (21:05 +0100)]
pick: contrib: document notmuch-pick-message-window-kill-hook

7 years agopick: contrib: document notmuch-pick-refresh-result
Mark Walters [Mon, 1 Jul 2013 20:05:41 +0000 (21:05 +0100)]
pick: contrib: document notmuch-pick-refresh-result

7 years agopick: contrib: document notmuch-pick-show-out
Mark Walters [Mon, 1 Jul 2013 20:05:40 +0000 (21:05 +0100)]
pick: contrib: document notmuch-pick-show-out

7 years agocontrib: pick: remove unused function
Mark Walters [Mon, 1 Jul 2013 07:54:03 +0000 (08:54 +0100)]
contrib: pick: remove unused function

The function notmuch-pick-show-error was used before Austin's improved
error handling. It is now redundant so remove.

7 years agocontrib: pick: remove unused variable
Mark Walters [Mon, 1 Jul 2013 07:54:02 +0000 (08:54 +0100)]
contrib: pick: remove unused variable

This variable was needed at one point for the async parser but is not
needed anymore.

7 years agocontrib: pick: bugfix: make the right variable buffer-local
Mark Walters [Mon, 1 Jul 2013 07:54:01 +0000 (08:54 +0100)]
contrib: pick: bugfix: make the right variable buffer-local

The variable notmuch-pick-message-buffer should be buffer local but
instead notmuch-pick-message-buffer-name (a non-existent variable) was
made buffer local.