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

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

Date: Wed, 25 Jan 2012 10:41:26 +0000

To: Tomi Ollila, Pieter Praet

Cc: Notmuch Mail

From: David Edmondson

On Wed, 25 Jan 2012 12:18:22 +0200, Tomi Ollila <tomi.ollila@iki.fi> wrote:
> On Wed, 25 Jan 2012 06:31:43 +0000, David Edmondson <dme@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 all choice url's expanded so user may even edit
> those

`completing-read' on the _names_ of the services, sure. I'm not sure
that showing the URLs as a set of choices would be nice.

> (like fix -2- to -1- if the don't have cover letter ;)

Should a single patch need a cover letter?

• Previous message (by thread): Re: [PATCH v6 1/3] emacs: `notmuch-show-get-message-id': optionally return Message-Id sans prefix

Thread:

• David Edmondson—[PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'. [inbox, notmuch::emacs, notmuch::feature, notmuch::obsolete, notmuch::patch, unread]
  • Pieter Praet—Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'. [inbox, notmuch::emacs, notmuch::feature, notmuch::obsolete, notmuch::patch, unread]
    • David Edmondson—Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'. [inbox, signed, unread]
      • Tomi Ollila—Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'. [inbox, unread]
        • David Edmondson—Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'. [inbox, signed, unread]
          • Tomi Ollila—Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'. [inbox, unread]
            • Pieter Praet—Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'. [inbox, unread]
      • Pieter Praet—Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'. [inbox, unread]
        • Pieter Praet—[RFC] Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'. [inbox, notmuch::emacs, notmuch::feature, notmuch::obsolete, notmuch::patch, unread]
          • David Edmondson—Re: [RFC] Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'. [inbox, signed, unread]
            • Pieter Praet—Re: [RFC] Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'. [inbox, unread]
              • Pieter Praet—[PATCH v4?] emacs: add `notmuch-show-stash-mlarchive-link{,-and-go}' [inbox, notmuch::emacs, notmuch::feature, notmuch::obsolete, notmuch::patch, unread]
                • David Edmondson—Re: [PATCH v4?] emacs: add `notmuch-show-stash-mlarchive-link{, -and-go}' [inbox, signed, unread]
                  • Pieter Praet—Re: [PATCH v4?] emacs: add `notmuch-show-stash-mlarchive-link{, -and-go}' [inbox, unread]
              • Dmitry Kurochkin—Re: [RFC] Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'. [inbox, unread]
                • Pieter Praet—Re: [RFC] Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'. [inbox, unread]
  • Pieter Praet—[PATCH v2 1/2] emacs: `notmuch-show-get-message-id': optionally return Message-Id sans prefix [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    • David Edmondson—Re: [PATCH v2 1/2] emacs: `notmuch-show-get-message-id': optionally return Message-Id sans prefix [inbox, notmuch::review, signed, unread]
  • Pieter Praet—[PATCH v2 2/2] emacs: add `notmuch-show-stash-mlarchive-link{, -and-go}' [inbox, notmuch::emacs, notmuch::feature, notmuch::obsolete, notmuch::patch, unread]
    • David Edmondson—Re: [PATCH v2 2/2] emacs: add `notmuch-show-stash-mlarchive-link{, -and-go}' [inbox, notmuch::review, signed, unread]
  • Pieter Praet—Re: [PATCH] emacs: Add `notmuch-show-stash-gmane' and `notmuch-show-stash-gmane-and-go'. [inbox, unread]
    • Pieter Praet—[PATCH v6 1/3] emacs: `notmuch-show-get-message-id': optionally return Message-Id sans prefix [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
      • David Bremner—Re: [PATCH v6 1/3] emacs: `notmuch-show-get-message-id': optionally return Message-Id sans prefix [inbox, unread]
        • Pieter Praet—Re: [PATCH v6 1/3] emacs: `notmuch-show-get-message-id': optionally return Message-Id sans prefix [inbox, unread]
          • David Bremner—Re: [PATCH v6 1/3] emacs: `notmuch-show-get-message-id': optionally return Message-Id sans prefix [inbox, unread]
            • Pieter Praet—Re: [PATCH v6 1/3] emacs: `notmuch-show-get-message-id': optionally return Message-Id sans prefix [inbox, unread]
    • Pieter Praet—[PATCH v6 2/3] test: emacs: expand subtest "Stashing in notmuch-show" wrt stashing Mailing List Archive URIs [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
    • Pieter Praet—[PATCH v6 3/3] emacs: add `notmuch-show-stash-mlarchive-link{, -and-go}' [inbox, notmuch::emacs, notmuch::feature, notmuch::patch, notmuch::pushed, unread]