Re: [PATCH] Return maximum of 150 results

2011-08-19 Thread Michal Sojka
Hi James, On Fri, 19 Aug 2011, James Vasile wrote: Display a maximum of 150 results when searching for messages. This prevents client lag when searches return thousands of results you'll never look at I would not like this feature. If I search for something I really want to see everything.

Re: [PATCH] Return maximum of 150 results

2011-08-19 Thread Michal Sojka
On Fri, 19 Aug 2011, James Vasile wrote: On Fri, 19 Aug 2011 17:00:23 +0200, Michal Sojka sojk...@fel.cvut.cz wrote: Hi James, On Fri, 19 Aug 2011, James Vasile wrote: Display a maximum of 150 results when searching for messages. This prevents client lag when searches return

[PATCH] Do not query on notmuch-search exit

2011-08-22 Thread Michal Sojka
Emacs 23.2 queries by default about killing existing processes. This is annoying when one wants to interrupt long search with 'q' key. Disable this behavior for notmuch. -- I'm resending this again as jrollins requested in id:871uwhz228@servo.factory.finestructure.net. Can anyone check

Re: [PATCH] Do not query on notmuch-search exit

2011-08-22 Thread Michal Sojka
On Mon, 22 Aug 2011, Tomi Ollila wrote: On Mon 22 Aug 2011 17:28, Tomi Ollila tomi.oll...@nixu.com writes: On Mon 22 Aug 2011 16:49, Michal Sojka sojk...@fel.cvut.cz writes: id:871uwhz228@servo.factory.finestructure.net. Can anyone check whether this works with emacs earlier than 23.2

[PATCH] Do not query on notmuch-search exit

2011-08-22 Thread Michal Sojka
Emacs 23.2 queries by default about killing existing processes. This is annoying when one wants to interrupt long search with 'q' key. Disable this behavior for notmuch. --- emacs/notmuch.el |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch.el

Re: [PATCH v6 1/2] emacs: User-defined sections in notmuch-hello

2011-11-24 Thread Michal Sojka
interconnected. Is there something I can do to help with the review? I think there's also one performance improvement written by Michal Sojka, that isn't included in this version yet, Yes, that's true. I'm aware of one bug in this improvement. I usually start notmuch by emacsclient -c

Re: [PATCH v6 1/2] emacs: User-defined sections in notmuch-hello

2011-11-27 Thread Michal Sojka
On Mon, 28 Nov 2011, Dmitry Kurochkin wrote: Hi Daniel. On Thu, 24 Nov 2011 15:01:01 +0100, Daniel Schoepe dan...@schoepe.org wrote: On Thu, 24 Nov 2011 09:54:50 -0400, David Bremner da...@tethera.net wrote: On Mon, 10 Oct 2011 15:39:41 +0200, Daniel Schoepe dan...@schoepe.org wrote:

[PATCH] emacs: Fix display of highlighted line in notmuch-show

2012-02-14 Thread Michal Sojka
When notmuch-search-line-faces is used to set background color in search results, the highlight of the current line is not always displayed correctly. This patch fixes that by increasing the priority property of the highlight overlay. --- emacs/notmuch.el | 15 +++ 1 files changed,

[PATCH v2] emacs: Fix display of highlighted line in notmuch-search

2012-02-14 Thread Michal Sojka
When notmuch-search-line-faces is used to set background color in search results, the highlight of the current line is not always displayed correctly. This patch fixes that by increasing the priority property of the highlight overlay. --- emacs/notmuch.el | 15 +++ 1 files changed,

Re: [PATCH v10 0/2] emacs: User-defined sections in notmuch-hello

2012-02-18 Thread Michal Sojka
On Fri, 17 Feb 2012, Dmitry Kurochkin wrote: Found a small bug... Changes: v10: * explicitly return nil from `notmuch-hello-insert-recent-searches', otherwise `indent-rigidly' returns a marker pointing to non-existing buffer and breaks final-target-pos handling in `notmuch-hello'

[PATCH] emacs-hello: Do not calculate the count of the messages in hidden sections

2012-02-18 Thread Michal Sojka
The result is that hello screen shows much faster when some sections are hidden. --- emacs/notmuch-hello.el | 20 ++-- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/emacs/notmuch-hello.el b/emacs/notmuch-hello.el index aad373d..e9caade 100644 ---

[PATCH] emacs: Mention race condition safety in user visible documentation

2012-02-18 Thread Michal Sojka
After recent rework of a/A/x/X key bindings, the important paragraph in documentation of `notmuch-show-archive-thread' stayed hidden from users, because no key is bound to this function. This patch copies the important paragraph to the documentation of functions currently bound to keys. ---

Re: [PATCH] emacs: Mention race condition safety in user visible documentation

2012-02-18 Thread Michal Sojka
On Sun, 19 Feb 2012, Jani Nikula wrote: On Feb 19, 2012 12:44 AM, Michal Sojka sojk...@fel.cvut.cz wrote: After recent rework of a/A/x/X key bindings, the important paragraph in documentation of `notmuch-show-archive-thread' stayed hidden from users, because no key is bound

Re: [PATCH] emacs: Mention race condition safety in user visible documentation

2012-02-21 Thread Michal Sojka
On Tue, 21 Feb 2012, Mark Walters wrote: On Sun, 19 Feb 2012 08:33:00 +0100, Michal Sojka sojk...@fel.cvut.cz wrote: On Sun, 19 Feb 2012, Jani Nikula wrote: On Feb 19, 2012 12:44 AM, Michal Sojka sojk...@fel.cvut.cz wrote: After recent rework of a/A/x/X key bindings, the important

Re: Searching through different charsets

2012-02-23 Thread Michal Sojka
On Wed, 22 Feb 2012, Serge Z wrote: Hello! I've got the following problem: fetched emails can be in different encodings. And searching a term typed in one encoding (system default) does not match the same term in another encoding. The solution, as I see, can be in preprocessing each

[PATCH] test: Add test for searching of uncommonly encoded messages

2012-02-23 Thread Michal Sojka
Emails that are encoded differently than as ASCII or UTF-8 are not indexed properly by notmuch. It is not possible to search for non-ASCII words within those messages. --- test/encoding|9 + test/test-lib.sh |5 + 2 files changed, 14 insertions(+), 0 deletions(-) diff

Re: [PATCH] test: Add test for searching of uncommonly encoded messages

2012-02-23 Thread Michal Sojka
On Fri, 24 Feb 2012, Serge Z wrote: Quoting Michal Sojka (2012-02-24 04:33:15) Emails that are encoded differently than as ASCII or UTF-8 are not indexed properly by notmuch. It is not possible to search for non-ASCII words within those messages. Ok. But we can preprocess each incoming

[PATCH 2/2] test: Remove 'broken' flag from encoding test

2012-02-23 Thread Michal Sojka
--- test/encoding |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/test/encoding b/test/encoding index 3992b5c..f0d073c 100755 --- a/test/encoding +++ b/test/encoding @@ -22,7 +22,6 @@ irrelevant message} test_begin_subtest Search for ISO-8859-2 encoded message

[PATCH 1/2] Convert non-UTF-8 parts to UTF-8 before indexing them

2012-02-23 Thread Michal Sojka
This fixes a bug that didn't allow to search for non-ASCII words such parts. The code here was copied from show_text_part_content(), because the show command already does the needed conversion when showing the message. --- lib/index.cc | 15 +++ 1 files changed, 15 insertions(+), 0

Re: [PATCH] test: Add test for searching of uncommonly encoded messages

2012-02-24 Thread Michal Sojka
On Fri, 24 Feb 2012, Serge Z wrote: Quoting Michal Sojka (2012-02-24 11:00:02) I'm not big fan of adding preprocessor. First, I thing that both reasons you mention are actually bugs and it would be better to fix them for everybody than requiring each user to configure some preprocessor

Re: Double decoded text/html parts (was: [PATCH] test: Add test for searching of uncommonly encoded messages)

2012-02-26 Thread Michal Sojka
On Sat, 25 Feb 2012, Serge Z wrote: Hi! I've struck another problem: I've got an html/text email with body encoded with cp1251. Its encoding is mentioned in both Content-type: email header and html meta tag. So when the client tries to display it with external html2text converter, The

Re: [PATCH v10 1/2] emacs: User-defined sections in notmuch-hello

2012-03-01 Thread Michal Sojka
On Thu, 01 Mar 2012, David Bremner wrote: Pushed, finally. Thanks for both of your hard work on this. Hi, is anybody willing to review id:1329603149-6047-1-git-send-email-sojk...@fel.cvut.cz? I'd like to see it merged togetger with these patches as it makes my work with notmuch-hello much

JSON readtable error when replying

2012-04-30 Thread Michal Sojka
Hi list, when I try to reply within emacs to a particular message, I get JSON readtable error. This happens with with the current git master as well as with a version from about a month ago. Is this a known problem or shall I dig into it and find what wrong? Thanks, -Michal

Re: JSON readtable error when replying

2012-04-30 Thread Michal Sojka
Hi Adam, Adam Wolfe Gordon awg+notm...@xvx.ca writes: On Mon, Apr 30, 2012 at 09:00, Michal Sojka sojk...@fel.cvut.cz wrote: when I try to reply within emacs to a particular message, I get JSON readtable error. This happens with with the current git master as well as with a version from about

[PATCH] emacs: Do not pass stderr of notmuch reply to JSON parser

2012-05-01 Thread Michal Sojka
Sometimes, notmuch reply outputs something to stderr, for example: Failed to verify signed part: Cannot verify multipart/signed part: unsupported signature protocol. When this happens, replying in emacs fails, because emacs cannot parse the error message as JSON. This patch causes emacs to ignore

Re: android 3.0 tablet and notmuch?

2012-05-07 Thread Michal Sojka
David Belohrad da...@belohrad.ch writes: Dear All, i experience very weird problem: have samsung galaxy tab 10.1 without any firmware tweaks. now, when I take a picture in the tablet and send it via email to me, then I open this email in emacs, the picture is not there. Only text part of

Re: emacs complains about encoding?

2012-05-22 Thread Michal Sojka
Hello Adam, Adam Wolfe Gordon awg+notm...@xvx.ca writes: It turns out it's actually not the emacs side, but an interaction between our JSON reply format and emacs. The JSON reply (and show) code includes part content for all text/* parts except text/html. Because all JSON is required to be

Re: emacs complains about encoding?

2012-05-23 Thread Michal Sojka
Tomi Ollila tomi.oll...@iki.fi writes: Michal Sojka sojk...@fel.cvut.cz writes: Hello Adam, Adam Wolfe Gordon awg+notm...@xvx.ca writes: It turns out it's actually not the emacs side, but an interaction between our JSON reply format and emacs. The JSON reply (and show) code includes part

Re: [PATCH] emacs: use 'gnus-decoded in notmuch-mm-display-part-inline ()

2012-05-23 Thread Michal Sojka
Tomi Ollila tomi.oll...@iki.fi writes: When mail message is read from emacs, the message structure obtained may contain parts which have content included (`text/plain` for example) and other parts where content is not included (`text/html` for example). In case content is included, the

Re: Sending from multiple accounts

2012-05-23 Thread Michal Sojka
Hi Doug, Doug Penner darwinsurvi...@gmail.com writes: I am in the process of trying to migrate from alpine (which handles separate accounts *very* distinctly) to notmuch (which combines them). I have no problem with incoming e-mails being combined (actually looking forward to it) but am

[PATCH] emacs: Add customization group for replying

2012-07-26 Thread Michal Sojka
When one wants to customize the format of reply message, there nothing to tell the user how to do it. Without knowing that construction of reply buffer is actually implemented in the message package, it seems that replying cannot be customized. Because I was getting annoyed by the empty line the

Re: [PATCH 01/11] lib: new thread addresses structure

2012-08-30 Thread Michal Sojka
Hi Jameson, some comments below. On Mon, Aug 20 2012, Jameson Graef Rollins wrote: This new structure holds addresses associated with a thread, both matched and unmatched. Initially this will be used to replace the existing infrastructure for storing the addresses of thread authors. Further

Re: [PATCH] emacs: Add customization group for replying

2012-09-04 Thread Michal Sojka
On Thu, Jul 26 2012, Michal Sojka wrote: When one wants to customize the format of reply message, there nothing to tell the user how to do it. Without knowing that construction of reply buffer is actually implemented in the message package, it seems that replying cannot be customized

Re: [PATCH] emacs: Add customization group for replying

2012-09-04 Thread Michal Sojka
On Tue, Sep 04 2012, Michal Nazarewicz wrote: On Thu, Jul 26 2012, Michal Sojka wrote: Because I was getting annoyed by the empty line the between citation line and the quoted message, I figured out that changing this is fairly easy when one knows what to customize. On semi-related note

Re: [PATCH v2 0/5] emacs: notmuch-hello status message refresh hook, etc

2012-09-05 Thread Michal Sojka
On Tue, Sep 04 2012, Jani Nikula wrote: On Tue, 04 Sep 2012, Michal Sojka sojk...@fel.cvut.cz wrote: On Sun, Sep 02 2012, Jani Nikula wrote: Hi all, v2 of [1] addressing review comments: added NEWS, improved documentation, made the query customizable. [1] id

Re: [PATCH v2] emacs: Add customization group for replying

2012-09-06 Thread Michal Sojka
On Thu, Sep 06 2012, Tomi Ollila wrote: On Thu, Sep 06 2012, Michal Sojka sojk...@fel.cvut.cz wrote: When one wants to customize the format of reply message, there is nothing to tell the user how to do it. Without knowing that construction of reply buffer is actually implemented

Re: [PATCH v2] emacs: Add customization group for replying

2012-09-06 Thread Michal Sojka
On Thu, Sep 06 2012, Tomi Ollila wrote: On Thu, Sep 06 2012, Michal Sojka sojk...@fel.cvut.cz wrote: So far good, but... You forgot to handle the current group setting for notmuch-message-replied-tags -- currently notmuch-message.el has this: (defcustom notmuch-message-replied-tags

Re: [PATCH] notmuch-show: include Bcc header in json output

2012-09-06 Thread Michal Sojka
On Wed, Sep 05 2012, Michal Nazarewicz wrote: From: Michal Nazarewicz min...@mina86.com --- notmuch-show.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) Actually, I don't understand why json does not include all the headers... Hi, I tested this and haven't found any

Re: [PATCH] notmuch-show: include Bcc header in json output

2012-09-07 Thread Michal Sojka
On Thu, Sep 06 2012, Michal Nazarewicz wrote: From: Michal Nazarewicz min...@mina86.com --- notmuch-show.c |7 +++ test/json|8 test/test-lib.sh |5 + 3 files changed, 16 insertions(+), 4 deletions(-) diff --git a/notmuch-show.c b/notmuch-show.c

mm-image-fit-p: Invalid image specification

2012-09-14 Thread Michal Sojka
Hello, in emacs, I have a problem displaying an email that contain the following part (--format=json): { content-type : image/svg+xml, filename : _logo.svg, id : 4 } I see this message mm-image-fit-p: Invalid image specification and no other emails after this

[PATCH] test: Improve tests for the date/time parser module

2012-09-25 Thread Michal Sojka
This patch reworks date/time parser library test program to make it easier to to write the actual tests. It also modifies the notmuch test script and adds several new tests to it. The INPUT file for the test contains both the dates to be parsed as well as the expected results. The test program

Re: [PATCH v3 6/9] lib: add date range query support

2012-09-25 Thread Michal Sojka
On Thu, Sep 13 2012, Michal Nazarewicz wrote: On Wed, Sep 12 2012, Jani Nikula wrote: Add a custom value range processor to enable date and time searches of the form date:since..until, where since and until are expressions understood by the previously added date/time parser, to restrict the

Re: [PATCH v3 2/9] parse-time-string: add a date/time parser to notmuch

2012-10-03 Thread Michal Sojka
On Wed, Oct 03 2012, Jani Nikula wrote: On Tue, 25 Sep 2012, Michal Sojka sojk...@fel.cvut.cz wrote: Hello Jani, On Wed, Sep 12 2012, Jani Nikula wrote: Add a date/time parser to notmuch, to be used for adding date range query support for notmuch lib later on. Add the parser to a directory

Re: [PATCH] test: Improve tests for the date/time parser module

2012-10-04 Thread Michal Sojka
On Wed, Oct 03 2012, Jani Nikula wrote: On Tue, 25 Sep 2012, Michal Sojka sojk...@fel.cvut.cz wrote: This patch reworks date/time parser library test program to make it easier to to write the actual tests. It also modifies the notmuch test script and adds several new tests

Re: [PATCH v6 0/9] notmuch search date:since..until query support

2012-11-02 Thread Michal Sojka
2012-02-25 [1/8] Jani Nikula| Tomi Ollila, Mark Walters; [RFC PATCH 0/2] natural language date range search (inbox list notmuch notmuch::patch notmuch::wip unread watch) thread:5191 2011-08-15 [2/8] Jani Nikula, Michal Sojka| Jason Woofenden, Tomi Ollila; [RFC PATCH 3/3] lib: add

[PATCH 2/3] test: Add some missing maildir synchronization tests

2012-12-19 Thread Michal Sojka
As mentioned by Jani Nikula in id:87vcccp4y3@nikula.org, some cases of maildir synchronization are not covered by our tests. Let's add the missing tests. --- test/maildir-sync | 12 1 file changed, 12 insertions(+) diff --git a/test/maildir-sync b/test/maildir-sync index

[PATCH 1/3] test: Adding non-maildir tags does not move message from new to cur

2012-12-19 Thread Michal Sojka
From: Michal Sojka so...@os.inf.tu-dresden.de 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

[PATCH 3/3] tags_to_maildir_flags: Don't rename if no flags change

2012-12-19 Thread Michal Sojka
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

[PATCH 0/3] Adding non-maildir tags does not move message from new to cur

2012-12-19 Thread Michal Sojka
): tags_to_maildir_flags: Don't rename if no flags change Michal Sojka (2): test: Adding non-maildir tags does not move message from new to cur test: Add some missing maildir synchronization tests lib/message.cc| 13 - test/maildir-sync | 20 2 files changed, 32

Re: [PATCH] test: Adding non-maildir tags does not move message from new to cur

2012-12-19 Thread Michal Sojka
Hello Jani, On Sat, Dec 08 2012, Jani Nikula wrote: On Sat, 08 Dec 2012, da...@tethera.net wrote: From: Michal Sojka so...@os.inf.tu-dresden.de Some MUA's like mutt show the difference between new emails living in maildir directory new/, and old emails living in maildir directory cur

Re: mm-image-fit-p: Invalid image specification

2014-01-27 Thread Michal Sojka
On Sat, Jan 25 2014, David Bremner wrote: Michal Sojka sojk...@fel.cvut.cz writes: Hello, in emacs, I have a problem displaying an email that contain the following part (--format=json): { content-type : image/svg+xml, filename : _logo.svg, id : 4 } I

[PATCH 1/2] Add test for incorrect threading of messages

2014-05-26 Thread Michal Sojka
This happens when there is some garbage after the last Message-ID in the References header. See for example https://lkml.org/lkml/headers/2014/5/19/864. --- test/T510-thread-replies.sh | 28 1 file changed, 28 insertions(+) diff --git a/test/T510-thread-replies.sh

[PATCH 0/2] Improve message threading

2014-05-26 Thread Michal Sojka
. I think that my problem is orthogonal to what was discussed there. The patch does not break any existing text (except two in T460-emacs-tree.sh, which fail even with 0.18 on my system). Cheers, -Michal Michal Sojka (2): Add test for incorrect threading of messages Make parsing of References

Re: nevermore

2014-07-29 Thread Michal Sojka
Hi all! On Wed, Jul 16 2014, Trevor Jim wrote: If you would like to incorporate things into the main branch, that would be great. Here's what I think the status is: The asynchronous mail address harvesting works pretty well. Much better than having to install python etc. It would be better

[PATCH RFC] Emacs: Add address completion mechanism implemented in elisp

2014-07-29 Thread Michal Sojka
Notmuch currently has an address completion mechanism that requires external script to provide completion candidates. This patch adds a completion mechanism found in https://github.com/tjim/nevermore, which is implemented in elisp only. notmuch-lib.el is extended with function

Re: [PATCH RFC] Emacs: Add address completion mechanism implemented in elisp

2014-07-30 Thread Michal Sojka
On Wed, Jul 30 2014, David Bremner wrote: Michal Sojka sojk...@fel.cvut.cz writes: Notmuch currently has an address completion mechanism that requires external script to provide completion candidates. This patch adds a completion mechanism found in https://github.com/tjim/nevermore, which

Re: [PATCH RFC] Emacs: Add address completion mechanism implemented in elisp

2014-07-31 Thread Michal Sojka
On Thu, Jul 31 2014, Trevor Jim wrote: BTW I have been refactoring my async process code. You might consider it for your patch. I'll look at that. I figured out that I might also reuse notmuch-parser.el. I'll post here my findings later. -Michal ___

Re: [PATCH RFC] Emacs: Add address completion mechanism implemented in elisp

2014-08-11 Thread Michal Sojka
Hi David, On Fri, Aug 01 2014, David Bremner wrote: Michal Sojka sojk...@fel.cvut.cz writes: notmuch-company.el hooks itself into message-mode and uses company-mode to offer the completion to the user. The file is put into the contrib directory which means that the use has to install

[PATCH v2] Emacs: Add address completion mechanism implemented in elisp

2014-08-11 Thread Michal Sojka
Currently, notmuch has an address completion mechanism that requires external command to provide completion candidates. This patch adds a completion mechanism inspired by https://github.com/tjim/nevermore, which is implemented in Emacs lisp only. The core of the new mechanism is the function

Re: [PATCH v3] Emacs: Add address completion mechanism implemented in elisp

2014-08-17 Thread Michal Sojka
On 11.8.2014 17:31, Michal Sojka wrote: Currently, notmuch has an address completion mechanism that requires external command to provide completion candidates. This patch adds a completion mechanism inspired by https://github.com/tjim/nevermore, which is implemented in Emacs lisp only. The core

[PATCH v4 0/3] Emacs: Address completion implemented in elisp

2014-09-19 Thread Michal Sojka
out by others. This is not (yet) based on Jani's patch for printing the addresses by notmuch search (id:1410021689-15901-1-git-send-email-j...@nikula.org). I'll at this as a next step as it will simplify the elisp code and make the address completion faster. Michal Sojka (3): Emacs: Display

[PATCH v4 1/3] Emacs: Display a message when generating address completion candidates

2014-09-19 Thread Michal Sojka
The TAB-initiated address completion generates completion candidates synchronously, blocking the UI. Since this can take long time, it is better to let the use know what's happening. --- emacs/notmuch-address.el | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH v4 3/3] Emacs: Add address completion based on company-mode

2014-09-19 Thread Michal Sojka
+1,81 @@ +;; notmuch-company.el --- Mail address completion for notmuch via company-mode -*- no-byte-compile: t; lexical-binding: t -*- + + +;; Authors: Trevor Jim t...@mac.com +;; Michal Sojka sojk...@fel.cvut.cz +;; +;; Keywords: mail, completion + +;; This program is free software; you

[PATCH v4 2/3] Emacs: Add address completion mechanism implemented in elisp

2014-09-19 Thread Michal Sojka
Currently, notmuch has an address completion mechanism that requires external command to provide completion candidates. This patch adds a completion mechanism inspired by https://github.com/tjim/nevermore, which is implemented in Emacs lisp only. The preexisting address completion mechanism,

[PATCH 1/5] cli: Refactor option passing in the search command

2014-09-22 Thread Michal Sojka
Many functions that implement the search command need to access command line options. Instead of passing each option in a separate variable, put them in a structure and pass only this structure. This will become handy in the following patches. --- notmuch-search.c | 122

[PATCH 5/5] cli: Add tests for 'search --output=addresses' and similar

2014-09-22 Thread Michal Sojka
--- test/T090-search-output.sh | 59 +++ test/T095-search-unique.sh | 63 ++ 2 files changed, 122 insertions(+) create mode 100755 test/T095-search-unique.sh diff --git a/test/T090-search-output.sh

[PATCH 2/5] cli: Extend the search command for --output=addresses and similar

2014-09-22 Thread Michal Sojka
The new outputs allow printing senders, recipients or both of matching messages. This code based on a patch from Jani Nikula. --- completion/notmuch-completion.bash | 2 +- completion/notmuch-completion.zsh | 3 +- doc/man1/notmuch-search.rst| 22 +++- notmuch-search.c

[PATCH 0/5] notmuch search --output=addresses

2014-09-22 Thread Michal Sojka
). Another nice use case is sending patched with git: git send-email --cc-cmd='nmsenders id:' ... where nmsenders script contains: notmuch search --output=sender $(notmuch search --output=threads $1) Michal Sojka (5): cli: Refactor option passing in the search command cli: Extend

Re: [PATCH 1/5] cli: Refactor option passing in the search command

2014-09-25 Thread Michal Sojka
On Thu, Sep 25 2014, Tomi Ollila wrote: On Mon, Sep 22 2014, Michal Sojka sojk...@fel.cvut.cz wrote: Many functions that implement the search command need to access command line options. Instead of passing each option in a separate variable, put them in a structure and pass only

Re: Changing sort order in notmuch-show thread view (emacs) ?

2014-09-27 Thread Michal Sojka
Hi Olivier, On Wed, Sep 24 2014, Olivier Berger wrote: Hi. Maybe this is obvious, but I can't figure out how to change the sort order from inside a threaded view when reviewing discussions in a thread, in Emacs. Using removes thread indentation, but then I'm not sure what the sort order

Re: [RFC] database: get and set mapping of dovecot compatible maildir keywords

2014-09-27 Thread Michal Sojka
Hi David, On Sat, Sep 27 2014, David Bremner wrote: A future modification of notmuch_message_tags_to_maildir_flags and notmuch_message_maildir_flags_to_tags could allow 26 more tags to be synced via maildir. --- I'm not sure if this is worth pursuing or not, but I thought I'd toss it out

Re: [PATCH] Add default configuration values to the man page

2014-10-05 Thread Michal Sojka
On Wed, Oct 01 2014, Sergei Shilovsky wrote: --- doc/man1/notmuch-config.rst | 18 ++ 1 file changed, 18 insertions(+) diff --git a/doc/man1/notmuch-config.rst b/doc/man1/notmuch-config.rst index 3c9a568..9de5534 100644 --- a/doc/man1/notmuch-config.rst +++

[PATCH v2 0/4] notmuch search --output=addresses

2014-10-05 Thread Michal Sojka
): cli: Add support for parsing multiple keyword arguments Michal Sojka (3): cli: Refactor option passing in the search command cli: Extend the search command for --output=addresses and similar cli: Add configurable address deduplication for --output=addresses command-line-arguments.c

[PATCH v2 4/4] cli: Add configurable address deduplication for --output=addresses

2014-10-05 Thread Michal Sojka
The code here is an extended version of a patch from Jani Nikula. --- completion/notmuch-completion.bash | 6 ++- completion/notmuch-completion.zsh | 3 +- doc/man1/notmuch-search.rst| 32 notmuch-search.c | 101 ++---

[PATCH v2 3/4] cli: Add support for parsing multiple keyword arguments

2014-10-05 Thread Michal Sojka
From: Jani Nikula j...@nikula.org This allows having multiple --foo=bar --foo=baz options on the command line, with the corresponding values OR'd together. [Test added by Michal Sojka] --- command-line-arguments.c | 6 +- command-line-arguments.h | 1 + test/T410-argument

[PATCH v2 1/4] cli: Refactor option passing in the search command

2014-10-05 Thread Michal Sojka
Many functions that implement the search command need to access command line options. Instead of passing each option in a separate variable, put them in a structure and pass only this structure. This will become handy in the following patches. --- notmuch-search.c | 122

[PATCH v2 2/4] cli: Extend the search command for --output=addresses and similar

2014-10-05 Thread Michal Sojka
The new outputs allow printing senders, recipients or both of matching messages. This code based on a patch from Jani Nikula. --- completion/notmuch-completion.bash | 2 +- completion/notmuch-completion.zsh | 3 +- doc/man1/notmuch-search.rst| 22 +++- notmuch-search.c

Re: [PATCH v2 2/4] cli: Extend the search command for --output=addresses and similar

2014-10-09 Thread Michal Sojka
On Mon, Oct 06 2014, Tomi Ollila wrote: On Sun, Oct 05 2014, Michal Sojka sojk...@fel.cvut.cz wrote: The new outputs allow printing senders, recipients or both of matching messages. This code based on a patch from Jani Nikula. OK, IMO... 1/4 OK Before 2/4 add support for 'flag

[PATCH v3 0/4] notmuch search --output=sender/recipients

2014-10-12 Thread Michal Sojka
multiple keyword arguments Michal Sojka (3): cli: Refactor option passing in the search command cli: Extend the search command for --output={sender,recipients} cli: Add an option to filter our duplicate addresses command-line-arguments.c | 6 +- command-line-arguments.h

[PATCH v3 2/4] cli: Add support for parsing multiple keyword arguments

2014-10-12 Thread Michal Sojka
From: Jani Nikula j...@nikula.org This allows having multiple --foo=bar --foo=baz options on the command line, with the corresponding values OR'd together. [Test added by Michal Sojka] --- command-line-arguments.c | 6 +- command-line-arguments.h | 1 + test/T410-argument

[PATCH v3 1/4] cli: Refactor option passing in the search command

2014-10-12 Thread Michal Sojka
Many functions that implement the search command need to access command line options. Instead of passing each option in a separate variable, put them in a structure and pass only this structure. This will become handy in the following patches. --- notmuch-search.c | 122

[PATCH v3 4/4] cli: Add an option to filter our duplicate addresses

2014-10-12 Thread Michal Sojka
This adds a --filter-by option to notmuch search. It can be used to filter out duplicate addresses in --output=sender/receivers. The code here is an extended version of a patch from Jani Nikula. --- completion/notmuch-completion.bash | 6 ++- completion/notmuch-completion.zsh | 3 +-

[PATCH v3 3/4] cli: Extend the search command for --output={sender, recipients}

2014-10-12 Thread Michal Sojka
The new outputs allow printing senders, recipients or both of matching messages. The --output option is converted from keyword argument to flags argument, which means that the user can use --output=sender and --output=recipients simultaneously, to print both. Other combinations produce an error.

Re: [PATCH v3 3/4] cli: Extend the search command for --output={sender, recipients}

2014-10-24 Thread Michal Sojka
Hi Mark, I mostly agree with your points mentioned in this and other your emails. I'll prepare v4 based on that. On Wed, Oct 22 2014, Mark Walters wrote: On Sun, 12 Oct 2014, Michal Sojka sojk...@fel.cvut.cz wrote: The new outputs allow printing senders, recipients or both of matching

[PATCH v4 6/6] cli: search: Add --output=count

2014-10-27 Thread Michal Sojka
This output can be used with --output=recipients or --output=sender and in addition to the mailboxes, it prints how many times was each mailbox encountered during search. --- completion/notmuch-completion.bash | 2 +- completion/notmuch-completion.zsh | 2 +- doc/man1/notmuch-search.rst

[PATCH v4 0/6] notmuch search --output=sender/recipients

2014-10-27 Thread Michal Sojka
- There is no way to output unfiltered (duplicite) addresses. Hopefully, the introduction of --output=count is sufficient replacement for this feature. Cheers, -Michal Jani Nikula (1): cli: Add support for parsing keyword-flag arguments Michal Sojka (5): cli: search: Refactor passing of command line

[PATCH v4 4/6] cli: search: Add --output={sender,recipients}

2014-10-27 Thread Michal Sojka
The new outputs allow printing senders, recipients or both of matching messages. To print both, the user can use --output=sender and --output=recipients simultaneously. Currently, the same address can appear multiple times in the output. The next patch will change this. For this reason, the test

[PATCH v4 1/6] cli: search: Refactor passing of command line options

2014-10-27 Thread Michal Sojka
Many functions that implement the search command need to access command line options. Instead of passing each option in a separate variable, put them in a structure and pass only this structure. This will become handy in the following patches. --- notmuch-search.c | 125

[PATCH v4 3/6] cli: search: Convert --output to keyword-flag argument

2014-10-27 Thread Michal Sojka
This converts notmuch search to use the recently introduced keyword-flag argument parser. At this point, it only makes the code slightly less readable but following patches that add new --output keywords will profit from this. --- notmuch-search.c | 35 ++- 1 file

[PATCH v4 2/6] cli: Add support for parsing keyword-flag arguments

2014-10-27 Thread Michal Sojka
From: Jani Nikula j...@nikula.org This allows having multiple --foo=bar --foo=baz options on the command line, with the corresponding values OR'd together. [Test added by Michal Sojka] --- command-line-arguments.c | 6 +- command-line-arguments.h | 1 + test/T410-argument

Re: [PATCH v4 1/6] cli: search: Refactor passing of command line options

2014-10-30 Thread Michal Sojka
Hi Mark, On Thu, Oct 30 2014, Mark Walters wrote: Hi On Mon, 27 Oct 2014, Michal Sojka sojk...@fel.cvut.cz wrote: Many functions that implement the search command need to access command line options. Instead of passing each option in a separate variable, put them in a structure and pass

Re: [PATCH v4 5/6] cli: search: Add configurable way to filter out duplicate addresses

2014-10-30 Thread Michal Sojka
On Thu, Oct 30 2014, Mark Walters wrote: On Mon, 27 Oct 2014, Michal Sojka sojk...@fel.cvut.cz wrote: This adds an algorithm to filter out duplicate addresses from address outputs (sender, receivers). The algorithm can be configured with --filter-by command line option. The code here

Re: [PATCH v4 5/6] cli: search: Add configurable way to filter out duplicate addresses

2014-10-30 Thread Michal Sojka
On Thu, Oct 30 2014, Tomi Ollila wrote: On Thu, Oct 30 2014, Mark Walters markwalters1...@gmail.com wrote: On Mon, 27 Oct 2014, Michal Sojka sojk...@fel.cvut.cz wrote: This adds an algorithm to filter out duplicate addresses from address outputs (sender, receivers). The algorithm can

[PATCH v5 0/7] notmuch search --output=sender/recipients

2014-10-30 Thread Michal Sojka
: Add support for parsing keyword-flag arguments Michal Sojka (6): cli: search: Refactor passing of command line options cli: search: Convert --output to keyword-flag argument cli: search: Add --output={sender,recipients} cli: search: Do not output duplicate addresses cli: search: Add

[PATCH v5 1/7] cli: search: Refactor passing of command line options

2014-10-30 Thread Michal Sojka
Many functions that implement the search command need to access command line options. Instead of passing each option in a separate variable, put them in a structure and pass only this structure. This will become handy in the following commits. --- notmuch-search.c | 125

[PATCH v5 2/7] cli: Add support for parsing keyword-flag arguments

2014-10-30 Thread Michal Sojka
From: Jani Nikula j...@nikula.org This allows having multiple --foo=bar --foo=baz options on the command line, with the corresponding values OR'd together. [Test added by Michal Sojka] --- command-line-arguments.c | 6 +- command-line-arguments.h | 1 + test/T410-argument

[PATCH v5 3/7] cli: search: Convert --output to keyword-flag argument

2014-10-30 Thread Michal Sojka
This converts notmuch search to use the recently introduced keyword-flag argument parser. At this point, it only makes the code slightly less readable but following commits that add new --output keywords will profit from this. --- notmuch-search.c | 35 ++- 1 file

[PATCH v5 4/7] cli: search: Add --output={sender,recipients}

2014-10-30 Thread Michal Sojka
The new outputs allow printing senders, recipients or both of matching messages. To print both, the user can use --output=sender and --output=recipients simultaneously. Currently, the same address can appear multiple times in the output. The next commit will change this. For this reason, tests

[PATCH v5 6/7] cli: search: Add --output=count

2014-10-30 Thread Michal Sojka
This output can be used with --output=recipients or --output=sender and in addition to the addresses, it prints how many times was each address encountered during search. --- completion/notmuch-completion.bash | 2 +- completion/notmuch-completion.zsh | 2 +- doc/man1/notmuch-search.rst

[PATCH v5 7/7] cli: search: Add --filter-by option to configure address filtering

2014-10-30 Thread Michal Sojka
This option allows to configure the criterion for duplicate address filtering. Without this option, all unique combinations of name and address parts are printed. This option allows to filter the output more, for example to only contain unique address parts. --- completion/notmuch-completion.bash

<    1   2   3   4   5   6   7   8   9   >