| Commit message (Collapse) | Author | Age |
| |
|
| |
|
|
|
|
|
|
| |
Previously, this function took an argument called "message-id", even
though it was a general query, rather than a message ID. This changes
it to "query".
|
|
|
|
|
|
|
|
|
| |
This makes more logical sense, since it makes the recursive printer
responsible for the entire reply body and lets it start at the root of
the MIME tree instead of the first child. (We could move reply header
creation in there, too, but if we ever support proper reply to
multiple messages, we'll want just one set of reply headers computed
from the entire message set and many bodies.)
|
|
|
|
|
| |
This re-arranges the default reply formatter code to use the
mime_node_t abstraction. There are no semantic changes.
|
|
|
|
|
|
|
|
|
| |
Previously, show and reply had separate implementations of decoding
and printing text parts. Now both use show's implementation, which
was more complete. Show's implementation has been extended with an
option to add reply quoting to the extracted part (this is implemented
as a named flag to avoid naked booleans, even though it's the only
flag it can take).
|
|
|
|
|
|
|
|
|
|
| |
This adds a lib function to turn a message ID into a properly escaped
message ID query and uses this function wherever we previously
hand-constructed ID queries. Wherever this new function is used,
documentation has been clarified to refer to "id: queries" instead of
"message IDs".
This fixes the broken test introduced by the previous patch.
|
|
|
|
|
| |
Currently this is broken because Emacs doesn't properly escape double
quotes in message IDs.
|
|
|
|
|
|
|
| |
Refactor to make tagging code easier to reuse in the future. No
functional changes.
Signed-off-by: Jani Nikula <jani@nikula.org>
|
|
|
|
|
|
|
| |
Refactor to make tagging code easier to reuse in the future. No
functional changes.
Signed-off-by: Jani Nikula <jani@nikula.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
To simplify code, keep all tagging operations in a single array
instead of separate add and remove arrays. Apply tag changes in the
order specified on the command line, instead of first removing and
then adding the tags.
This results in a minor functional change: If a tag is both added and
removed, the last specified operation is now used. Previously the tag
was always added. Change the relevant test to reflect the new
behaviour.
Signed-off-by: Jani Nikula <jani@nikula.org>
|
|
|
|
|
|
|
| |
The current behaviour is that regardless of the order in which the
addition and removal of a tag are specified, the tag is added.
Signed-off-by: Jani Nikula <jani@nikula.org>
|
|
|
|
|
|
|
|
| |
The function notmuch-match-content-type was comparing content types
case sensitively. Fix it so it tests case insensitively.
This fixes a bug where emacs would not include any body when replying
to a message with content-type TEXT/PLAIN.
|
| |
|
| |
|
|
|
|
|
|
| |
Some 0.12 NEWS items descriptions were indented with 3 spaces whereas
all other lines are indented with 2 spaces. Brought those
escaped lines back in line with others.
|
|\
| |
| |
| | |
notmuch Debian 0.12-1 upload (same as 0.12 + debian changelog fix)
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
This allows for testing against both versions of gmime on a single
machine, without having to mess with pkg-config paths.
This is rework of Tom Prince's patch submitted in
id:"1331402091-15663-1-git-send-email-tom.prince@ualberta.net"
|
| |
| |
| |
| |
| |
| | |
In the future, IFS value needs to be changed in a few places
in configure -- and then restored. Store the original value
to $DEFAULT_IFS for easy restoration.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Formerly the code assumed the arguments to be triples and threw an
error if this was not the case. But those arguments are only there for
compatibility with autotools and are not used within the build system,
so just dropping the code parsing these values makes the build system
more robust.
Signed-off-by: Justus Winter <4winter@informatik.uni-hamburg.de>
|
| |
| |
| |
| | |
Signed-off-by: Justus Winter <4winter@informatik.uni-hamburg.de>
|
|\|
| |
| |
| | |
notmuch 0.12 release
|
| | |
|
|\|
| |
| |
| |
| |
| |
| | |
Conflicts:
NEWS
Conflicts resolved by hand for date of 0.12 release.
|
| |
| |
| |
| | |
Include extremely terse summary of NEWS.
|
| |
| |
| |
| | |
As we discovered recently on the list, this really is new for 0.12
|
| |
| |
| |
| | |
There may be a few NEWS changes after this, but no code (hopefully).
|
| |
| |
| |
| |
| | |
Be more specific than "Current GMime 2.6". Also explain why we need
>=2.6.7
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Use the new JSON reply format to create replies in emacs. Quote HTML
parts nicely by using mm-display-part to turn them into displayable
text, then quoting them with message-cite-original. This is very
useful for users who regularly receive HTML-only email.
Use message-mode's message-cite-original function to create the
quoted body for reply messages. In order to make this act like the
existing notmuch defaults, you will need to set the following in
your emacs configuration:
message-citation-line-format "On %a, %d %b %Y, %f wrote:"
message-citation-line-function 'message-insert-formatted-citation-line
The tests have been updated to reflect the (ugly) emacs default.
|
| |
| |
| |
| |
| | |
Add tests for creating nice replies to multipart messages, including
those with HTML parts. These tests are expected to fail for now.
|
| |
| |
| |
| |
| | |
Move a few functions related to handling multipart/alternative parts
into notmuch-lib.el, so they can be used by future reply code.
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
This new JSON format for replies includes headers generated for a
reply message as well as the headers of the original message. Using
this data, a client can intelligently create a reply. For example, the
emacs client will be able to create replies with quoted HTML parts by
parsing the HTML parts.
|
| | |
|
| |
| |
| |
| |
| | |
Factor out the creation of a reply message based on an original
message so it can be shared by different reply formats.
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
People are making patches for the 0.12 NEWS section for features that
won't be there.
Let's help them out by adding a new stanza.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
In X, Emacs distinguishes the tab key, which produces a 'tab event;
from C-i, which produces a ?\t event. However, in a terminal, these
are indistinguishable and only produce a ?\t event. In order to
simplify things, Emacs automatically translates from 'tab to ?\t (see
"Function key translations" in M-x describe-bindings), so functions
only need to be bound to ?\t to work in all situations.
Previously, the search tab completion code usedq (kbd "<tab>"), which
produced the event sequence [tab], which only matched the 'tab event
and hence only worked in X. This patch changes it to (kbd "TAB"),
which matches the general ?\t event and works in all situations.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
When the exclude tags contain a tag that does not occur anywhere in
the Xapian database the exclusion fails. We modify the way the query
is constructed to `work around' this. (In fact the new code is cleaner
anyway.)
It also seems to fix another exclusion failure bug reported by
jrollins but we have not yet worked out why it helps in that case.
|
| |
| |
| |
| |
| |
| |
| | |
This adds three tests for --output=messages searches. One test is for
the case when one exclude tag does not occur in the Xapian
database. This triggers a Xapian bug in some cases and causes the
whole exclusion to fail. The next commit avoids this bug.
|
| |
| |
| |
| |
| |
| |
| | |
The tests for the exclude code in search and count use the line
notmuch config set search.exclude_tags = deleted
which actually sets the exclude tags to be "=" and "deleted". Remove
the "=" from this line.
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Allow query debugging to be enabled at run-time by setting the
NOTMUCH_DEBUG_QUERY environment variable to a non-empty string.
Previously, enabling query debugging required recompiling, but parsed
queries are often useful for tracking down bugs in situations where
recompiling is inconvenient.
|
| |
| |
| |
| |
| |
| | |
Allow NOTMUCH_OPT_BOOLEAN to take a true or false parameter. In
particular it allows the user to turn off a boolean option with
--option=false.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Before the change, messages generated by generate_message() used "Test
message #N" for default subject where N is the generated messages
counter. Since message subject is commonly present in expected
results, there is a chance of breaking other tests when a new
generate_message() call is added. The patch changes default subject
value for generated messages to subtest name if it is available. If
subtest name is not available (i.e. message is generated during test
initialization), the old default value is used (in this case it is
fine to have the counter in the subject).
Another benefit of this change is a sane default value for subject in
generated messages, which would allow to simplify code like:
test_begin_subtest "test for a cool feature"
add_message [subject]="message for test for a cool feature"
|