priorities for 0.13

2012-04-29 Thread Mark Walters
On Sun, 29 Apr 2012, David Bremner wrote: > David Bremner writes: > >> Hi All; >> >> I'd like to have a feature freeze for 0.13 sometime in the first week of >> May. What do people feel are priorities to try to get reviewed and >> pushed for 0.13? >> > > So here are some patches that people

[Patch v2 0/3] emacs: allow show to colour based on tags and flags

2012-04-29 Thread Austin Clements
I haven't really looked at this series yet, but I do have a quick high-level question. Why use separate customization variables for the colors in search and show mode? Wouldn't it make more sense to set the colors just once and use them in both modes? BTW, I like how this clearly distinguishes

priorities for 0.13

2012-04-29 Thread David Bremner
Mark Walters writes: > > Hi > > I think Jamie already posted a patch with a nicer comment > > id:"1334436547-10260-1-git-send-email-jrollins at finestructure.net" > > (ie that replaces 2/5 in the original series) Oh, yeah, I just got lost in the thread. OK, that series is pushed. d

priorities for 0.13

2012-04-29 Thread David Bremner
David Bremner writes: > Hi All; > > I'd like to have a feature freeze for 0.13 sometime in the first week of > May. What do people feel are priorities to try to get reviewed and > pushed for 0.13? > So here are some patches that people expressed interest in for 0.13, but need more review.

[PATCH 1/3] NEWS: untabified and added file local variables block

2012-04-29 Thread David Bremner
Tomi Ollila writes: > Changed all tabs to 8 spaces (M-x untabify over region of the > whole file). All three of these look OK to me. I do wonder whether we should call the file NEWS.mdwn or something like that if we intend to enforce markdown in it. d

[PATCH v3 1/4] emacs: Let the user choose where to compose new mails

2012-04-29 Thread David Bremner
Thomas Jost writes: > AFAICT, Emacs 23 is just buggy in this case. By reading the code of > message-send-and-exit and message-bury [1], here is what happens when > you call message-send-buffer-and-exit with message-kill-buffer-on-exit > set to nil: > - message is sent > - buffer is buried with

[RFC] Use JSON in emacs interface

2012-04-29 Thread Austin Clements
Quoth Chris Gray on Apr 29 at 10:22 am: > Hi, > > My thinking about this arises from the fact that there is a person on > one of the lists that I read who puts a semicolon after his name. Of > course, this confuses the regex in notmuch-search-process-filter, which > expects that the first

[PATCH 0/6] Make notmuch_database_{open, create} return status codes

2012-04-29 Thread Justus Winter
Hi Austin :) Quoting Austin Clements (2012-04-29 00:17:47) > Since we're breaking binary and source compatibility with the next > release anyway, it's about time we fix notmuch_database_{open,create} > to return status codes. Awesome, thanks for taking care of this. The python patch looks fine.

[RFC] Use JSON in emacs interface

2012-04-29 Thread Adam Wolfe Gordon
Hi Chris, On Sun, Apr 29, 2012 at 10:22, Chris Gray wrote: > I first thought of changing the regex so that it looked for the last > semicolon in the string or something like that, but that would just move > the problem. ?(Semicolons are probably more frequent in subject lines > than in author

[PATCH] NEWS: add news item for 'config list'

2012-04-29 Thread Peter Wang
--- NEWS |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/NEWS b/NEWS index a2cd080..4f36dff 100644 --- a/NEWS +++ b/NEWS @@ -52,6 +52,11 @@ Raw show format changes encoded in the original message, including the part's headers. Leaf parts, as before, output the

[RFC] Use JSON in emacs interface

2012-04-29 Thread Chris Gray
Hi, My thinking about this arises from the fact that there is a person on one of the lists that I read who puts a semicolon after his name. Of course, this confuses the regex in notmuch-search-process-filter, which expects that the first semicolon in the string representing a thread is after all

[ANN] New awesome vim plug-in using Ruby bindings

2012-04-29 Thread Alex Ghitza
Felipe Contreras wrote: > > vim -c ':NotMuchR' > > > > all I get is an error message: > > > > Error detected while processing command line: > > E492: Not an editor command: :NotMuchR > > I don't know if you did anything special to get the normal plug-in to > work. Maybe you are doing 'source

[Patch v4 2/3] emacs: add a filter option to show

2012-04-29 Thread Mark Walters
Show the current thread with a different filter (i.e., open messages in the thread matching the new query). Bound to 'l' for "limit". Note that it is not the same as filter in search mode as it replaces the existing query rather than ANDing with it (but it does keep the thread-id part of the

[Patch v4 1/3] emacs: split notmuch-show-apply-state

2012-04-29 Thread Mark Walters
Separate out a notmuch-show-goto-msg-id sub-function from notmuch-show-apply-state. There should be no functional change but the next patch will call the new function. --- emacs/notmuch-show.el | 18 +++--- 1 files changed, 11 insertions(+), 7 deletions(-) diff --git

[Patch v4 0/3] Add filter to emacs show mode

2012-04-29 Thread Mark Walters
This is version 4 of the patch set allowing the user to "filter" which messages are open in emacs show mode. The previous version was at id:"1335467689-6513-1-git-send-email-markwalters1009 at gmail.com" The change in this version is to keep position in the thread (at least to the same message).

[PATCH v2] emacs: do not modify subject in search or show

2012-04-29 Thread David Bremner
Jameson Graef Rollins writes: > A previous patch [0] replaced blank subject lines with '[No Subject]' > in search and show mode. pushed. d

[RFC] Use JSON in emacs interface

2012-04-29 Thread Chris Gray
Hi, My thinking about this arises from the fact that there is a person on one of the lists that I read who puts a semicolon after his name. Of course, this confuses the regex in notmuch-search-process-filter, which expects that the first semicolon in the string representing a thread is after all

Re: [RFC] Use JSON in emacs interface

2012-04-29 Thread Adam Wolfe Gordon
Hi Chris, On Sun, Apr 29, 2012 at 10:22, Chris Gray chrismg...@gmail.com wrote: I first thought of changing the regex so that it looked for the last semicolon in the string or something like that, but that would just move the problem.  (Semicolons are probably more frequent in subject lines

Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails

2012-04-29 Thread David Bremner
Thomas Jost schno...@schnouki.net writes: AFAICT, Emacs 23 is just buggy in this case. By reading the code of message-send-and-exit and message-bury [1], here is what happens when you call message-send-buffer-and-exit with message-kill-buffer-on-exit set to nil: - message is sent - buffer is

Re: [PATCH 1/3] NEWS: untabified and added file local variables block

2012-04-29 Thread David Bremner
Tomi Ollila tomi.oll...@iki.fi writes: Changed all tabs to 8 spaces (M-x untabify over region of the whole file). All three of these look OK to me. I do wonder whether we should call the file NEWS.mdwn or something like that if we intend to enforce markdown in it. d

Re: priorities for 0.13

2012-04-29 Thread David Bremner
David Bremner da...@tethera.net writes: Hi All; I'd like to have a feature freeze for 0.13 sometime in the first week of May. What do people feel are priorities to try to get reviewed and pushed for 0.13? So here are some patches that people expressed interest in for 0.13, but need more

Re: priorities for 0.13

2012-04-29 Thread Mark Walters
On Sun, 29 Apr 2012, David Bremner da...@tethera.net wrote: David Bremner da...@tethera.net writes: Hi All; I'd like to have a feature freeze for 0.13 sometime in the first week of May. What do people feel are priorities to try to get reviewed and pushed for 0.13? So here are some

Re: [RFC] Use JSON in emacs interface

2012-04-29 Thread Austin Clements
Quoth Chris Gray on Apr 29 at 10:22 am: Hi, My thinking about this arises from the fact that there is a person on one of the lists that I read who puts a semicolon after his name. Of course, this confuses the regex in notmuch-search-process-filter, which expects that the first semicolon in

[Patch v2 0/3] emacs: allow show to colour based on tags and flags

2012-04-29 Thread Mark Walters
This is a rebased (but otherwise unchanged) version of id:1334431301-27303-1-git-send-email-markwalters1...@gmail.com. It's probably too late for 0.13 but in case anyone would like to look at it this version applies cleanly to master so should be easier to test. The first two patches are

[Patch v2 1/3] emacs: Move colour line from search to lib

2012-04-29 Thread Mark Walters
This patch moves the overlay/colouring from notmuch.el to notmuch-lib.el. This is in preparation for its use by notmuch-show in the next patch. This is just a rebased version of the emacs/notmuch.el and emacs/notmuch-lib.el parts of David Edmondson's patch (see

[Patch v2 2/3] emacs: Add `notmuch-show-line-faces' and apply it.

2012-04-29 Thread Mark Walters
Similar to `notmuch-search-line-faces', `notmuch-show-line-faces' allows the header line in `notmuch-show-mode' buffers to be coloured according to the tags of the message. This is just a rebased version of the emacs/notmuch-show.el of David Edmondson's patch

Re: [Patch v2 0/3] emacs: allow show to colour based on tags and flags

2012-04-29 Thread Austin Clements
I haven't really looked at this series yet, but I do have a quick high-level question. Why use separate customization variables for the colors in search and show mode? Wouldn't it make more sense to set the colors just once and use them in both modes? BTW, I like how this clearly distinguishes

Re: [Patch v2 0/3] emacs: allow show to colour based on tags and flags

2012-04-29 Thread Mark Walters
On Mon, 30 Apr 2012, Austin Clements amdra...@mit.edu wrote: I haven't really looked at this series yet, but I do have a quick high-level question. Why use separate customization variables for the colors in search and show mode? Wouldn't it make more sense to set the colors just once and

Re: [Patch v2 3/3] emacs: allow notmuch-show-line-faces to use flags for colouring

2012-04-29 Thread Tomi Ollila
On Mon, Apr 30 2012, Mark Walters markwalters1...@gmail.com wrote: [ ... ] --- emacs/notmuch-show.el | 21 - 1 files changed, 16 insertions(+), 5 deletions(-) [ ... ] - (setq notmuch-search-line-faces '((\delete\ . (:foreground \red\ + (setq notmuch-search-line-faces