[PATCH] emacs: target notmuch-one.el[c] -- emacs client in one .el[c] file

2013-09-02 Thread Tomi Ollila
Added Makefile recipes to create notmuch emacs client in just one .el[c] file. This is an experimental feature and not built by default. This is useful for example when one wants to build the latest emacs client and then distribute the elisp file to many machines. This is also useful for (initia

Re: [PATCH] test: update insert tests for new maildir synchronization rules

2013-09-02 Thread Jani Nikula
On Tue, 03 Sep 2013, da...@tethera.net wrote: > From: David Bremner > > As of id:1355952747-27350-4-git-send-email-sojk...@fel.cvut.cz > we are more conservative about moving messages from ./new to ./cur. > This updates the insert tests to match > --- > > Hopefully this makes sense, and is not jus

[PATCH 1/2] test: add more maildir flag syncing related tests to insert

2013-09-02 Thread Jani Nikula
Specifically test maildir flag syncing with insert. --- These are additional tests on top of David's fix. --- test/insert | 36 1 file changed, 36 insertions(+) diff --git a/test/insert b/test/insert index 1718120..9b448e5 100755 --- a/test/insert +++ b/test

[PATCH 2/2] test: improve insert test reliability by checking message-id instead of count

2013-09-02 Thread Jani Nikula
There isn't a reported issue this would fix. Spotted by reading the test. --- test/insert | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/test/insert b/test/insert index 9b448e5..550b413 100755 --- a/test/insert +++ b/test/insert @@ -68,14 +68,14 @@ test_expect_equ

[PATCH] test: update insert tests for new maildir synchronization rules

2013-09-02 Thread da...@tethera.net
From: David Bremner As of id:1355952747-27350-4-git-send-email-sojkam1 at fel.cvut.cz we are more conservative about moving messages from ./new to ./cur. This updates the insert tests to match --- Hopefully this makes sense, and is not just papering over a real problem that should be fixed. te

[Patch v3 3/3] contrib: pick: test: refresh view

2013-09-02 Thread Mark Walters
A recent proposed patch was buggy when refreshing the view. Add a test for refresh so that this does not reoccur. --- contrib/notmuch-pick/test/emacs-pick | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/contrib/notmuch-pick/test/emacs-pick b/contrib/notmuch-pic

[Patch v3 2/3] contrib: pick: add docstring for notmuch-pick-worker

2013-09-02 Thread Mark Walters
--- contrib/notmuch-pick/notmuch-pick.el |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/contrib/notmuch-pick/notmuch-pick.el b/contrib/notmuch-pick/notmuch-pick.el index 32bd6b5..5dcc942 100644 --- a/contrib/notmuch-pick/notmuch-pick.el +++ b/contrib/notmuch-pick/notm

[Patch v3 1/3] contrib: pick: remove unneeded variable notmuch-pick-buffer-name

2013-09-02 Thread Mark Walters
This variable is essentially unused: it was only used for making sure it itself got reset after a refresh of the buffer. It did this by passing an unnecessary argument to notmuch-pick-worker so remove that too. --- contrib/notmuch-pick/notmuch-pick.el | 25 + 1 files cha

[Patch v3 0/3] contrib: pick: remove an unused variable

2013-09-02 Thread Mark Walters
v2 of this is at id:1377551345-27147-1-git-send-email-markwalters1009 at gmail.com This is unchanged except it is rebased on top of the series id:1377460534-6022-1-git-send-email-markwalters1009 at gmail.com so that should be applied first. The only difference is in the context of patch 1 of thi

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

2013-09-02 Thread Jani Nikula
On Wed, 19 Dec 2012, Michal Sojka wrote: > patches in this series should supersede the patches in > id:1316039001-32602-5-git-send-email-l.rilling at av7.net. They address > the comments from Jani Nikula and add tests for the behavior that was > unintentionally changed by the previous version of t

[Patch v4 0/3] Add some thread based actions to pick

2013-09-02 Thread Tomi Ollila
On Sun, Aug 25 2013, Mark Walters wrote: > This is a rebased version of > id:1371195472-441-1-git-send-email-markwalters1009 at gmail.com > > The only other change is a new patch 3 adding previous/next thread > commands: they are both very simple functions based on the first two > patches. > > B

[Patch v2 0/3] contrib: pick: remove an unused variable

2013-09-02 Thread Tomi Ollila
On Tue, Aug 27 2013, Mark Walters wrote: > v2 of this series is at > id:1377460214-4795-1-git-send-email-markwalters1009 at gmail.com > > v2 had a bug on refresh view (which I should have tested more). The > main pick view only worked by fluke as the initial call to pick-worker > was inside a let

[PATCH] test: update insert tests for new maildir synchronization rules

2013-09-02 Thread david
From: David Bremner As of id:1355952747-27350-4-git-send-email-sojk...@fel.cvut.cz we are more conservative about moving messages from ./new to ./cur. This updates the insert tests to match --- Hopefully this makes sense, and is not just papering over a real problem that should be fixed. test/

[Patch v3 3/3] contrib: pick: test: refresh view

2013-09-02 Thread Mark Walters
A recent proposed patch was buggy when refreshing the view. Add a test for refresh so that this does not reoccur. --- contrib/notmuch-pick/test/emacs-pick | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/contrib/notmuch-pick/test/emacs-pick b/contrib/notmuch-pic

[Patch v3 0/3] contrib: pick: remove an unused variable

2013-09-02 Thread Mark Walters
v2 of this is at id:1377551345-27147-1-git-send-email-markwalters1...@gmail.com This is unchanged except it is rebased on top of the series id:1377460534-6022-1-git-send-email-markwalters1...@gmail.com so that should be applied first. The only difference is in the context of patch 1 of this serie

[Patch v3 1/3] contrib: pick: remove unneeded variable notmuch-pick-buffer-name

2013-09-02 Thread Mark Walters
This variable is essentially unused: it was only used for making sure it itself got reset after a refresh of the buffer. It did this by passing an unnecessary argument to notmuch-pick-worker so remove that too. --- contrib/notmuch-pick/notmuch-pick.el | 25 + 1 files cha

[Patch v3 2/3] contrib: pick: add docstring for notmuch-pick-worker

2013-09-02 Thread Mark Walters
--- contrib/notmuch-pick/notmuch-pick.el |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/contrib/notmuch-pick/notmuch-pick.el b/contrib/notmuch-pick/notmuch-pick.el index 32bd6b5..5dcc942 100644 --- a/contrib/notmuch-pick/notmuch-pick.el +++ b/contrib/notmuch-pick/notm

[PATCH] emacs: target notmuch-one.el[c] -- emacs client in one .el[c] file

2013-09-02 Thread Tomi Ollila
Added Makefile recipes to create notmuch emacs client in just one .el[c] file. This is an experimental feature and not built by default. This is useful for example when one wants to build the latest emacs client and then distribute the elisp file to many machines. This is also useful for (initia

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

2013-09-02 Thread Jani Nikula
On Wed, 19 Dec 2012, Michal Sojka wrote: > patches in this series should supersede the patches in > id:1316039001-32602-5-git-send-email-l.rill...@av7.net. They address > the comments from Jani Nikula and add tests for the behavior that was > unintentionally changed by the previous version of the

Re: [Patch v4 0/3] Add some thread based actions to pick

2013-09-02 Thread Tomi Ollila
On Sun, Aug 25 2013, Mark Walters wrote: > This is a rebased version of > id:1371195472-441-1-git-send-email-markwalters1...@gmail.com > > The only other change is a new patch 3 adding previous/next thread > commands: they are both very simple functions based on the first two > patches. > > Best

Re: [Patch v2 0/3] contrib: pick: remove an unused variable

2013-09-02 Thread Tomi Ollila
On Tue, Aug 27 2013, Mark Walters wrote: > v2 of this series is at > id:1377460214-4795-1-git-send-email-markwalters1...@gmail.com > > v2 had a bug on refresh view (which I should have tested more). The > main pick view only worked by fluke as the initial call to pick-worker > was inside a let bi

[PATCH 0/4] contrib: pick: allow more general format lines

2013-09-02 Thread Mark Walters
On Mon, 02 Sep 2013, Mark Walters wrote: > Currently pick makes the tree box graphics part of the "subject". This > is rather unsatisfactory: the tree graphics should be a field in their > own right. > > However, there is no mechanism in the current setup for allowing 2 > fields to have fixed com

[PATCH 4/4] contrib: pick: make the tree graphics a proper part of the format

2013-09-02 Thread Mark Walters
Previously the box graphics in the pick view were always attached to the subject. Make them a field in their own right. We use the recursive insert to change the default notmuch-pick-result-format so that the user view does not change. (The subject touches the tree box graphics but the next column

[PATCH 3/4] contrib: pick: allow recursive message field formats

2013-09-02 Thread Mark Walters
Previously, the message format was fixed: each part had to be a certain width and either left or right justified. This allows the user to specify that two parts can be variable width but that combined they should be some fixed width. We do this by allowing the user to set as a "field" a list of the

[PATCH 2/4] contrib: pick: move the insertion of fields up a level

2013-09-02 Thread Mark Walters
This moves the actual insertion of message fields up from the field formatting function into the message insertion function. This will be useful in the next patch as we can apply further formatting to the insertion string before inserting. --- contrib/notmuch-pick/notmuch-pick.el | 41 ++

[PATCH 1/4] contrib: pick: print () for a message with no tags

2013-09-02 Thread Mark Walters
Dating back to the earliest notmuch-pick we have not printed anything for the tag field for a message with no tags. This is inconsistent with search and show both of which print "()". Change pick to be consistent. --- contrib/notmuch-pick/notmuch-pick.el |7 +++ 1 files changed, 3 insertio

[PATCH 0/4] contrib: pick: allow more general format lines

2013-09-02 Thread Mark Walters
Currently pick makes the tree box graphics part of the "subject". This is rather unsatisfactory: the tree graphics should be a field in their own right. However, there is no mechanism in the current setup for allowing 2 fields to have fixed combined width but variable individual width. Thus making

Re: [Patch v2 0/3] contrib: pick: remove an unused variable

2013-09-02 Thread Mark Walters
I should have included a diff from v2: see below (obviously the test in patch 3 is also new). Best wishes Mark diff --git a/contrib/notmuch-pick/notmuch-pick.el b/contrib/notmuch-pick/notmuch-pick.el index f6710e9..5d46d42 100644 --- a/contrib/notmuch-pick/notmuch-pick.el +++ b/contrib/notm