[WIP Patch 0/7] emacs: show tag changes in buffer

2014-02-14 Thread Mark Walters
This verson is not intended for general use/review but so Austin can see my attempt at integrating his regexp tag matching code into the added/deleted code. However, of course, comments from anyone else are gratefully received. Patch 1/7 is basically Austin's patch

[WIP Patch 2/7] emacs: tag split customise option for format-tags into a widget

2014-02-14 Thread Mark Walters
We will re-use the customize option for format-tags for formattting deleted tags to added tags in the next patch so split it into a widget. There should be no functional change. --- emacs/notmuch-tag.el | 55 ++--- 1 files changed, 29 insertions(+),

[WIP Patch 5/7] emacs: show: use orig-tags for tag display

2014-02-14 Thread Mark Walters
This uses the previous patch to show the tag changes that have occured in the show buffer since it was last loaded/refreshed. --- emacs/notmuch-show.el |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index 1ce56f9..97243dc

[WIP Patch 3/7] emacs: tag: add customize for deleted/added tag formats

2014-02-14 Thread Mark Walters
Add customize options for deleted/added tag formats. These are not used yet but will be later in the series. --- emacs/notmuch-tag.el | 33 + 1 files changed, 33 insertions(+), 0 deletions(-) diff --git a/emacs/notmuch-tag.el b/emacs/notmuch-tag.el index

[WIP Patch 6/7] emacs: search: use orig-tags in search

2014-02-14 Thread Mark Walters
This uses the recent functionality to show the tag changes in the search buffer. Currently this is only used to show changes the search buffer makes itself: i.e., it does not make display any changes reflecting tagging done by other notmuch-buffers. --- emacs/notmuch.el | 40

[WIP Patch 4/7] emacs: show: mark tags changed since buffer loaded

2014-02-14 Thread Mark Walters
This allows (and requires) the original-tags to be passed along with the current-tags to be passed to notmuch-tag-format-tags. This allows the tag formatting to show added and deleted tags.By default a removed tag is displayed with strike-through in red (if strike-through is not available, eg on a

[WIP Patch 7/7] emacs: tree: use orig-tags in search

2014-02-14 Thread Mark Walters
This uses the recent functionality to show the tag changes in the tree buffer. Currently this is only used to show changes the tree buffer makes itself: i.e., it does not make display any changes reflecting tagging done by other notmuch-buffers. --- emacs/notmuch-tree.el |9 ++--- 1 files

Re: [PATCH v3 6/8] nmbug-status: Add inter-message padding

2014-02-14 Thread David Bremner
W. Trevor King wk...@tremily.us writes: Sure. Sorry I botched v3 :/. Do you want to nibble off patches one through three (and maybe five, which applies cleanly on top of three) first, or should I include them in v4? OK, those 4 patches are pushed. d

Re: [PATCH v2] new: Detect dirent.d_type support at configure time

2014-02-14 Thread David Bremner
Austin Clements amdra...@mit.edu writes: This is a trivial rebase of id:1385262952-29240-1-git-send-email-amdra...@mit.edu, which I apparently never got around to sending. Tomi and Jani checked off v1 of this patch, so I'm marking it ready. pushed, d

[PATCH v4 4/4] nmbug-status: Hardcode UTF-8 instead of using the user's locale

2014-02-14 Thread W. Trevor King
David [1] and Tomi [2] both feel that the user's choice of LANG is not explicit enough to have such a strong effect on nmbug-status. For example, cron jobs usually default to LANG=C, and that is going to give you ASCII output: $ LANG=C python -c 'import locale;

[PATCH v4 0/4] nmbug-status: Python-3-compatibility and general refactoring

2014-02-14 Thread W. Trevor King
This fixes two rebase-induced typos from v3 [1]: * Doubled meta-http-equiv entries in the color-threads patch [2]. * An extra close-paren in the HTML header format arguments [3]. The remainder of v3 has already landed in master [4]. Cheers, Trevor [1]: id:cover.1392309570.git.wk...@tremily.us

[PATCH v4 1/4] nmbug-status: Color threads in HTML output

2014-02-14 Thread W. Trevor King
Add tbody sections so we don't have to color every row. Multiple tbody sections are allowed [1]. Use CSS 3's nth-child to handle even/odd coloring (skipping the spacer rows) [2], which is supported on the major browsers [3]. border-spacing is from CCS 2.1 [4,5]. I'm using it to avoid

[PATCH v4 3/4] nmbug-status: Encode output using the user's locale

2014-02-14 Thread W. Trevor King
Instead of always writing UTF-8, allow the user to configure the output encoding using their locale. This is useful for previewing output in the terminal, for poor souls that don't use UTF-8 locales ;). --- devel/nmbug/nmbug-status | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff

[PATCH v4 2/4] nmbug-status: Add inter-message padding

2014-02-14 Thread W. Trevor King
We already had the tbody with a blank row separating threads (which is not colored); this commit adds a bit of spacing to separate messages within a thread. It will also add a bit of colored padding above the first message and below the final message, but the main goal is to add padding *between*

Re: [PATCH 1/1] emacs: always write emacs/.eldeps when the target is remade

2014-02-14 Thread Tomi Ollila
On Thu, Feb 13 2014, Austin Clements amdra...@mit.edu wrote: Quoth Tomi Ollila on Feb 13 at 9:26 am: On Thu, Feb 13 2014, Austin Clements amdra...@mit.edu wrote: Quoth Tomi Ollila on Jan 25 at 12:21 pm: So that the target is newer than its prerequisites. --- emacs/Makefile.local | 3

Re: [PATCH 1/1] emacs: always write emacs/.eldeps when the target is remade

2014-02-14 Thread Austin Clements
Quoth Tomi Ollila on Feb 14 at 9:24 pm: On Thu, Feb 13 2014, Austin Clements amdra...@mit.edu wrote: Quoth Tomi Ollila on Feb 13 at 9:26 am: On Thu, Feb 13 2014, Austin Clements amdra...@mit.edu wrote: Quoth Tomi Ollila on Jan 25 at 12:21 pm: So that the target is newer than its

[PATCH] man: escape backslash in notmuch-tag example

2014-02-14 Thread David Bremner
The example was originally intended to have a literal backslash in it, but '\ ' is interpreted by nroff as a non-breaking space. It doesn't make much difference to the example, but the non-breaking space triggers a bug in doclifter. --- man/man1/notmuch-tag.1 | 2 +- 1 file changed, 1

[WIP Patch 0/7] emacs: show tag changes in buffer

2014-02-14 Thread Mark Walters
This verson is not intended for general use/review but so Austin can see my attempt at integrating his regexp tag matching code into the added/deleted code. However, of course, comments from anyone else are gratefully received. Patch 1/7 is basically Austin's patch

[WIP Patch 1/7] Make keys of notmuch-tag-formats regexps and use caching

2014-02-14 Thread Mark Walters
From: Austin Clements This was a little hack to test the feasibility of switching notmuch-tag-formats to use regexps with caching for performance. In the end it works fine and isn't particularly complex, though there were a few gotchas: 1) We have to clear the cache somehow

[WIP Patch 2/7] emacs: tag split customise option for format-tags into a widget

2014-02-14 Thread Mark Walters
We will re-use the customize option for format-tags for formattting deleted tags to added tags in the next patch so split it into a widget. There should be no functional change. --- emacs/notmuch-tag.el | 55 ++--- 1 files changed, 29 insertions(+),

[WIP Patch 3/7] emacs: tag: add customize for deleted/added tag formats

2014-02-14 Thread Mark Walters
Add customize options for deleted/added tag formats. These are not used yet but will be later in the series. --- emacs/notmuch-tag.el | 33 + 1 files changed, 33 insertions(+), 0 deletions(-) diff --git a/emacs/notmuch-tag.el b/emacs/notmuch-tag.el index

[WIP Patch 4/7] emacs: show: mark tags changed since buffer loaded

2014-02-14 Thread Mark Walters
This allows (and requires) the original-tags to be passed along with the current-tags to be passed to notmuch-tag-format-tags. This allows the tag formatting to show added and deleted tags.By default a removed tag is displayed with strike-through in red (if strike-through is not available, eg on a

[WIP Patch 5/7] emacs: show: use orig-tags for tag display

2014-02-14 Thread Mark Walters
This uses the previous patch to show the tag changes that have occured in the show buffer since it was last loaded/refreshed. --- emacs/notmuch-show.el |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index 1ce56f9..97243dc

[WIP Patch 6/7] emacs: search: use orig-tags in search

2014-02-14 Thread Mark Walters
This uses the recent functionality to show the tag changes in the search buffer. Currently this is only used to show changes the search buffer makes itself: i.e., it does not make display any changes reflecting tagging done by other notmuch-buffers. --- emacs/notmuch.el | 40

[WIP Patch 7/7] emacs: tree: use orig-tags in search

2014-02-14 Thread Mark Walters
This uses the recent functionality to show the tag changes in the tree buffer. Currently this is only used to show changes the tree buffer makes itself: i.e., it does not make display any changes reflecting tagging done by other notmuch-buffers. --- emacs/notmuch-tree.el |9 ++--- 1 files

[PATCH v3 6/8] nmbug-status: Add inter-message padding

2014-02-14 Thread David Bremner
"W. Trevor King" writes: > > Sure. Sorry I botched v3 :/. Do you want to nibble off patches one > through three (and maybe five, which applies cleanly on top of three) > first, or should I include them in v4? > OK, those 4 patches are pushed. d

[PATCH v2] new: Detect dirent.d_type support at configure time

2014-02-14 Thread David Bremner
Austin Clements writes: > This is a trivial rebase of > id:1385262952-29240-1-git-send-email-amdragon at mit.edu, which I > apparently never got around to sending. Tomi and Jani checked off v1 > of this patch, so I'm marking it ready. pushed, d

[PATCH v4 4/4] nmbug-status: Hardcode UTF-8 instead of using the user's locale

2014-02-14 Thread W. Trevor King
David [1] and Tomi [2] both feel that the user's choice of LANG is not explicit enough to have such a strong effect on nmbug-status. For example, cron jobs usually default to LANG=C, and that is going to give you ASCII output: $ LANG=C python -c 'import locale;

[PATCH v4 0/4] nmbug-status: Python-3-compatibility and general refactoring

2014-02-14 Thread W. Trevor King
This fixes two rebase-induced typos from v3 [1]: * Doubled meta-http-equiv entries in the color-threads patch [2]. * An extra close-paren in the HTML header format arguments [3]. The remainder of v3 has already landed in master [4]. Cheers, Trevor [1]: id:"cover.1392309570.git.wking at

[PATCH v4 2/4] nmbug-status: Add inter-message padding

2014-02-14 Thread W. Trevor King
We already had the tbody with a blank row separating threads (which is not colored); this commit adds a bit of spacing to separate messages within a thread. It will also add a bit of colored padding above the first message and below the final message, but the main goal is to add padding *between*

[PATCH v4 3/4] nmbug-status: Encode output using the user's locale

2014-02-14 Thread W. Trevor King
Instead of always writing UTF-8, allow the user to configure the output encoding using their locale. This is useful for previewing output in the terminal, for poor souls that don't use UTF-8 locales ;). --- devel/nmbug/nmbug-status | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff

[PATCH 1/1] emacs: always write emacs/.eldeps when the target is remade

2014-02-14 Thread Tomi Ollila
On Thu, Feb 13 2014, Austin Clements wrote: > Quoth Tomi Ollila on Feb 13 at 9:26 am: >> On Thu, Feb 13 2014, Austin Clements wrote: >> >> > Quoth Tomi Ollila on Jan 25 at 12:21 pm: >> >> So that the target is newer than its prerequisites. >> >> --- >> >> emacs/Makefile.local | 3 +-- >> >>

[PATCH 1/1] emacs: always write emacs/.eldeps when the target is remade

2014-02-14 Thread Austin Clements
Quoth Tomi Ollila on Feb 14 at 9:24 pm: > On Thu, Feb 13 2014, Austin Clements wrote: > > > Quoth Tomi Ollila on Feb 13 at 9:26 am: > >> On Thu, Feb 13 2014, Austin Clements wrote: > >> > >> > Quoth Tomi Ollila on Jan 25 at 12:21 pm: > >> >> So that the target is newer than its

[PATCH] man: escape backslash in notmuch-tag example

2014-02-14 Thread David Bremner
The example was originally intended to have a literal backslash in it, but '\ ' is interpreted by nroff as a non-breaking space. It doesn't make much difference to the example, but the non-breaking space triggers a bug in doclifter. --- man/man1/notmuch-tag.1 | 2 +- 1 file changed, 1