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

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

Date: Thu, 26 Jan 2012 14:40:26 +0000

To: Pieter Praet

Cc: Notmuch Mail

From: 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
  really care.

- Don't base the patch on the thing that I posted, just on master from
  the repository - no need to make David's life harder.

• 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]