| Commit message (Collapse) | Author | Age |
|---|
| ... | |
| | | |
|
| |\|
| |
| |
| | |
debian bug fix release, disable tests
|
| | |
| |
| |
| |
| | |
Hopefully this is temporary, but I don't want to rush a fix for the
tests.
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Version string has strict format requirements in release-check.sh:
only numbers and periods (in sane order) are accepted.
Mismatch there used to halt further execution.
In this case, checking versions like '*~rc1' for (more) problems
was not possible.
This 'fatal error' is now changed buffered error message like in
following tests, and is displayed at the end of execution.
|
| |/
|
|
| |
This is just a cosmetic fix to make the "type" of ret more clear.
|
| |
|
|
| |
Bump the version in-place in NEWS.
|
| |
|
|
| |
Cribbed from NEWS.
|
| | |
|
| |
|
|
|
| |
Hopefully it is clear that more details are available in the man
pages.
|
| |
|
|
| |
Without this file, the created package is empty.
|
| |
|
|
|
| |
This is in some sense a rollback, but it makes all the automation
happier if the Debian and upstream versions match.
|
| |
|
|
| |
This was fixed a while ago in git, but not released yet.
|
| |
|
|
|
|
|
| |
This patch is due to Jakub Wilk <jwilk@debian.org>.
It does add a build depend on python3 for people using the "make
debian-snapshot" target.
|
| |
|
|
|
|
|
|
|
|
|
| |
- enable hardening
- fix dh syntax. Now that we have compat level 9, the old, wrong
syntax is no longer accepted.
- update debian/libnotmuch{3,-dev}.install for multiarch.
- update versioned dependency on debhelper.
|
| |
|
|
|
| |
If we have some DMs we want to upload, we have to enable it with a
signed command file to dak.
|
| |
|
|
|
|
| |
The distinction between extra and optional is generally not very
important, except that being extra forces anything that depends on you
to be extra.
|
| |
|
|
|
|
|
|
| |
In case last input for batch tagging was either invalid or skippable
line, notmuch command exited with non-zero value.
After this change if there is at least one invalid line, notmuch
command will exit with non-zero value. Additionally, skipped lines
(last or other) doesn't cause non-zero value to be returned.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There is a bug in the current notmuch code with w3m and invisible
parts. w3m sets a keymap, and if we have a hidden [text/html] point
at the start of the following line still gets this w3m keymap which
causes some strange effects. For example, RET gives an error "No URL
at Point" rather than hiding the message, <down> goes to the next link
rather than just down a line.
These keybinding are also inconvenient when the text/html part is
displayed so we ask w3m not to install a keymap.
This is only likely to be a problem for emacs 23 as shr is preferred
as html renderer on emacs 24 (although the user can set the renderer
to w3m even on emacs 24).
This solution was suggested by Tomi Ollila <tomi.ollila@iki.fi>
|
| | |
|
| |
|
|
|
|
|
|
|
| |
Checking and propagating tag_op_list_apply() errors is especially
important with batch tagging, as the processing of the batch input
would not stop otherwise. Additionally this sets the exit code, which
is useful in scripts.
Amended by: David Bremner
|
| |
|
|
|
| |
as usual (and doubly so since Debian is in release freeze), destined
for experimental.
|
| |
|
|
| |
"Atomically" update the python bindings and man page versions.
|
| |
|
|
|
| |
Xapian doesn't consider ":abc" to be a prefixed term. This makes
parse_boolean_term similarly reject queries with an empty prefix.
|
| |
|
|
|
|
|
|
|
| |
spaces.
This test also serves as documentation of the quoting
requirements. The comment lines are so that it exactly matches the man
page. Nothing more embarrassing than having an example in the man page
fail.
|
| | |
|
| |
|
|
|
|
| |
The (now fixed) bug that this test revealed is that unquoted
message-ids with whitespace or other control characters in them are
split into several tokens by the Xapian query parser.
|
| |
|
|
| |
We test quotes seperately because they matter to the query escaper.
|
| |
|
|
|
| |
This tests argument parsing, blank lines and comments, and basic hex
decoding functionality.
|
| |
|
|
|
|
| |
This is based on the similar test for notmuch restore, but the parser
in batch tagging mode is less tolerant of a few cases, in particular
those tested by illegal_tag.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add support for batch tagging operations through stdin to "notmuch
tag". This can be enabled with the new --batch command line option to
"notmuch tag". The input must consist of lines of the format:
+<tag>|-<tag> [...] [--] <query> [...]
Each line is interpreted similarly to "notmuch tag" command line
arguments. The delimiter is one or more spaces ' '. Any characters in
<tag> MAY be hex encoded with %NN where NN is the hexadecimal value of
the character. Any ' ' and '%' characters in <tag> and MUST be hex
encoded (using %20 and %25, respectively). For future-proofing, any
'"' characters in <tag> SHOULD be hex-encoded.
Any characters that are not part of <tag> or
MUST NOT be hex encoded.
<query> is passed verbatim to Xapian
Leading and trailing space ' ' is ignored. Empty lines and lines
beginning with '#' are ignored.
Signed-off-by: Jani Nikula <jani@nikula.org>
Hacked-like-crazy-by: David Bremner <david@tethera.net>
|
| |
|
|
|
|
|
|
|
| |
Command line parsing is factored out into a function
parse_tag_command_line in tag-util.c.
There is some duplicated code eliminated in tag_query, and a bunch of
translation from using the bare tag_op structs to using that tag-utils
API.
|
| |
|
|
|
| |
This will allow us to be consistent between batch tagging and command
line tagging as far as what is an illegal tag.
|
| |
|
|
|
|
| |
While one comment in generated .notmuch-config file looked good in the
source file notmuch-config.c, the generated output was inconsistently
wide -- even breaking the 80-column boundary.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We recently switched to popping up a buffer to report CLI errors, but
this was too intrusive, especially for transient errors and especially
since we made fewer things ignore errors. This patch changes this to
display a basic error message in the minibuffer (using Emacs' usual
error handling path) and, if there are additional details, to log
these to a separate error buffer and reference the error buffer from
the minibuffer message. This is more in line with how Emacs typically
handles errors, but makes the details available to the user without
flooding them with the details.
Given this split, we pare down the basic message and make it more
user-friendly, and also make the verbose message even more detailed
(and more debugging-oriented).
|
| |
|
|
|
| |
Describe the new batch-tag format. For notmuch-restore, rather than
half-heartedly duplicating the description, we now cite notmuch-dump.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
This switches the new batch-tag format away from using a home-grown
hex-encoding scheme for message IDs in the dump to simply using Xapian
queries with Xapian quoting syntax.
This has a variety of advantages beyond presenting a cleaner and more
consistent interface. Foremost is that it will dramatically simplify
the quoting for batch tagging, which shares the same input format.
While the hex-encoding is no better or worse for the simple ID queries
used by dump/restore, it becomes onerous for general-purpose queries
used in batch tagging. It also better handles strange cases like
"id:foo and bar", since this is no longer syntactically valid.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When we switch to using regular Xapian queries in the dump format, \n
will cause problems, so we disallow it. Specially, while Xapian can
quote and parse queries containing \n without difficultly, quoted
queries containing \n still span multiple lines, which breaks the
line-orientedness of the dump format. Strictly speaking, we could
still round-trip these, but it would significantly complicate restore
as well as scripts that deal with tag dumps. This complexity would
come at absolutely no benefit: because of the RFC 2822 unfolding
rules, no amount of standards negligence can produce a message with a
message ID containing a line break (not even Outlook can do it!).
Hence, we simply disallow it.
|
| |
|
|
|
|
|
| |
This parses the subset of Xapian's boolean term quoting rules that are
used by make_boolean_term. This is provided as a generic string
utility, but will be used shortly in notmuch restore to parse and
optimize for ID queries.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is now a generic boolean term quoting function. It performs
minimal quoting to produce user-friendly queries.
This could live in tag-util as well, but it is really nothing specific
to tags (although the conventions are specific to Xapian).
The API is changed from "caller-allocates" to "readline-like". The
scan for max tag length is pushed down into the quoting routine.
Furthermore, this now combines the term prefix with the quoted term;
arguably this is just as easy to do in the caller, but this will
nicely parallel the boolean term parsing function to be introduced
shortly.
This is an amalgamation of code written by David Bremner and myself.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Previously, restore would abort if a message ID in the dump was
missing. Furthermore, it would only report this as a warning. This
patch makes it distinguish abort-worthy lookup failures like
out-of-memory from non-fatal failure to find a message ID. The former
is reported as an error and causes restore to abort, while the latter
is reported as a warning and does not cause an abort.
This restores 0.14's non-fatal handling of missing message IDs in
restore (though 0.14 also considered serious errors non-fatal; we
retain the new and better handling of serious errors).
|
| |
|
|
|
|
|
|
|
|
| |
This patch corrects several undesirable behaviours:
1) Empty files were not detected, leading to buffer read overrun.
2) An initial blank line cause restore to silently abort
3) Initial comment line caused format detection to fail
|
| |
|
|
|
| |
Three of these are marked broken; the third is a regression test,
since it passes by virtue of batch-tag being the default input format.
|
| |
|
|
|
| |
As with the valgrind logs, we print a (very) brief summary and leave
the log for inspection.
|
| |
|
|
| |
This gives line numbers for better debugging.
|
| |
|
|
|
| |
These are intended to be simple wrappers to provide slightly better
debugging information than what talloc currently provides natively.
|
| |
|
|
|
|
|
| |
The argument handling in notmuch.c seems due for an overhaul, but
until then use an environment variable to specify a location to write
the talloc leak report to. This is only enabled for the (interesting)
case where some notmuch subcommand is invoked.
|
| |
|
|
| |
Consistently use [...]; one less space. Use singular <search-term>
|
| |
|
|
|
| |
This is essentially cosmetic, since success=0 is promised by
the comments in tag-utils.h.
|
| |
|
|
|
|
|
| |
We add a hook to the show buffer in the message window to close the
message window when that buffer quits. It checks that the
message-window is still displaying the show-message buffer and then
closes it.
|
| |
|
|
| |
Wording suggested by Austin.
|
Amadeusz Żołnowski
David Bremner
Tomi Ollila
Mark Walters
Jani Nikula
Austin Clements