[PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-02-19 Thread Pieter Praet
v6: - Rebased to current master (be851ad3). - Broke test out into a separate commit. v5: - id:"1327907574-12760-1-git-send-email-pieter at praet.org" - id:"1327907574-12760-2-git-send-email-pieter at praet.org" v4: - id:"1327653866-30084-1-git-send-email-pieter at praet.org" v3: -

Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-02-19 Thread Pieter Praet
v6: - Rebased to current master (be851ad3). - Broke test out into a separate commit. v5: - id:1327907574-12760-1-git-send-email-pie...@praet.org - id:1327907574-12760-2-git-send-email-pie...@praet.org v4: - id:1327653866-30084-1-git-send-email-pie...@praet.org v3: -

[RFC] Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-30 Thread Pieter Praet
On Fri, 27 Jan 2012 20:36:40 +0400, Dmitry Kurochkin wrote: > On Fri, 27 Jan 2012 09:42:23 +0100, Pieter Praet wrote: > > On Thu, 26 Jan 2012 14:40:26 +, David Edmondson wrote: > > > In general, I like this. > > > > > > - I think that the stash function(s) should take an optional argument

Re: [RFC] Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-29 Thread Pieter Praet
On Fri, 27 Jan 2012 20:36:40 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: On Fri, 27 Jan 2012 09:42:23 +0100, Pieter Praet pie...@praet.org wrote: On Thu, 26 Jan 2012 14:40:26 +, David Edmondson d...@dme.org wrote: In general, I like this. - I think that the stash

[RFC] Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-27 Thread Dmitry Kurochkin
On Fri, 27 Jan 2012 09:42:23 +0100, Pieter Praet wrote: > On Thu, 26 Jan 2012 14:40:26 +, David Edmondson wrote: > > In general, I like this. > > > > - I think that the stash function(s) should take an optional argument > > specifying the archive to use. That will make testing simpler and

[RFC] Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-27 Thread Pieter Praet
On Thu, 26 Jan 2012 14:40:26 +, David Edmondson wrote: > In general, I like this. > > - I think that the stash function(s) should take an optional argument > specifying the archive to use. That will make testing simpler and also > allow people to produce preferred bindings more easily. >

Re: [RFC] Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-27 Thread Pieter Praet
On Thu, 26 Jan 2012 14:40:26 +, David Edmondson d...@dme.org wrote: In general, I like this. - I think that the stash function(s) should take an optional argument specifying the archive to use. That will make testing simpler and also allow people to produce preferred bindings more

[RFC] Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-26 Thread David Edmondson
In general, I like this. - I think that the stash function(s) should take an optional argument specifying the archive to use. That will make testing simpler and also allow people to produce preferred bindings more easily. - "Message archive: " feels better than "ML Archive: ", but I don't

[RFC] Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-26 Thread Pieter Praet
--- Depends on id:"1327397873-20596-1-git-send-email-dme at dme.org" emacs/notmuch-show.el | 55 ++-- test/emacs|2 +- 2 files changed, 40 insertions(+), 17 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el

[PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-26 Thread Pieter Praet
On Wed, 25 Jan 2012 06:31:43 +, David Edmondson wrote: > On Wed, 25 Jan 2012 01:50:08 +0100, Pieter Praet wrote: > > In a perfect world, everyone would be using Notmuch and have a local > > copy of every message ever sent to any ML. While we're waiting for > > Atlantis to resurface, we'll

[PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-26 Thread Pieter Praet
On Wed, 25 Jan 2012 15:00:22 +0200, Tomi Ollila wrote: > [...] > So, 'completing-read' -- or something that does the same thing better (ymmv). > Thanks for the suggestion, but `completing-read' will do just fine... ;D > Tomi Peace -- Pieter

Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-26 Thread Pieter Praet
On Wed, 25 Jan 2012 15:00:22 +0200, Tomi Ollila tomi.oll...@iki.fi wrote: [...] So, 'completing-read' -- or something that does the same thing better (ymmv). Thanks for the suggestion, but `completing-read' will do just fine... ;D Tomi Peace -- Pieter

Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-26 Thread Pieter Praet
On Wed, 25 Jan 2012 06:31:43 +, David Edmondson d...@dme.org wrote: On Wed, 25 Jan 2012 01:50:08 +0100, Pieter Praet pie...@praet.org wrote: In a perfect world, everyone would be using Notmuch and have a local copy of every message ever sent to any ML. While we're waiting for Atlantis

[RFC] Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-26 Thread Pieter Praet
--- Depends on id:1327397873-20596-1-git-send-email-...@dme.org emacs/notmuch-show.el | 55 ++-- test/emacs|2 +- 2 files changed, 40 insertions(+), 17 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index

Re: [RFC] Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-26 Thread David Edmondson
In general, I like this. - I think that the stash function(s) should take an optional argument specifying the archive to use. That will make testing simpler and also allow people to produce preferred bindings more easily. - Message archive: feels better than ML Archive: , but I don't

[PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-25 Thread Tomi Ollila
On Wed, 25 Jan 2012 10:41:26 +, David Edmondson wrote: > On Wed, 25 Jan 2012 12:18:22 +0200, Tomi Ollila wrote: > > On Wed, 25 Jan 2012 06:31:43 +, David Edmondson wrote: > > > > > > Your point about Gmane not having everything applies equally to any > > > service, suggesting that

[PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-25 Thread Tomi Ollila
On Wed, 25 Jan 2012 06:31:43 +, David Edmondson wrote: > > Your point about Gmane not having everything applies equally to any > service, suggesting that perhaps the user should also have an option to > choose which service to use at stash time. Thoughts? Completing-read which has all

[PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-25 Thread David Edmondson
On Wed, 25 Jan 2012 12:18:22 +0200, Tomi Ollila wrote: > On Wed, 25 Jan 2012 06:31:43 +, David Edmondson wrote: > > > > Your point about Gmane not having everything applies equally to any > > service, suggesting that perhaps the user should also have an option to > > choose which service to

[PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-25 Thread David Edmondson
On Wed, 25 Jan 2012 01:50:08 +0100, Pieter Praet wrote: > In a perfect world, everyone would be using Notmuch and have a local > copy of every message ever sent to any ML. While we're waiting for > Atlantis to resurface, we'll need an interim solution. > > +1 for the idea, but Gmane doesn't

[PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-25 Thread Pieter Praet
In a perfect world, everyone would be using Notmuch and have a local copy of every message ever sent to any ML. While we're waiting for Atlantis to resurface, we'll need an interim solution. +1 for the idea, but Gmane doesn't archive *all* MLs, so we should leave the user some legroom. Since

Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-25 Thread Tomi Ollila
On Wed, 25 Jan 2012 06:31:43 +, David Edmondson d...@dme.org wrote: Your point about Gmane not having everything applies equally to any service, suggesting that perhaps the user should also have an option to choose which service to use at stash time. Thoughts? Completing-read which has

Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-25 Thread David Edmondson
On Wed, 25 Jan 2012 12:18:22 +0200, Tomi Ollila tomi.oll...@iki.fi wrote: On Wed, 25 Jan 2012 06:31:43 +, David Edmondson d...@dme.org wrote: Your point about Gmane not having everything applies equally to any service, suggesting that perhaps the user should also have an option to

Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-25 Thread Tomi Ollila
On Wed, 25 Jan 2012 10:41:26 +, David Edmondson d...@dme.org wrote: On Wed, 25 Jan 2012 12:18:22 +0200, Tomi Ollila tomi.oll...@iki.fi wrote: On Wed, 25 Jan 2012 06:31:43 +, David Edmondson d...@dme.org wrote: Your point about Gmane not having everything applies equally to any

[PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-24 Thread David Edmondson
`notmuch-show-stash-gmane' stashes a URI for the current message at Gmane, presuming that Gmane has a copy of the message with the same Message-Id. `notmuch-show-stash-gmane-and-go' both stashes the message and opens the relevant page in the configured browser. Add the relevant test for

[PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-24 Thread David Edmondson
`notmuch-show-stash-gmane' stashes a URI for the current message at Gmane, presuming that Gmane has a copy of the message with the same Message-Id. `notmuch-show-stash-gmane-and-go' both stashes the message and opens the relevant page in the configured browser. Add the relevant test for

Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'.

2012-01-24 Thread David Edmondson
On Wed, 25 Jan 2012 01:50:08 +0100, Pieter Praet pie...@praet.org wrote: In a perfect world, everyone would be using Notmuch and have a local copy of every message ever sent to any ML. While we're waiting for Atlantis to resurface, we'll need an interim solution. +1 for the idea, but Gmane