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

Subject: Re: [PATCH v6 1/3] emacs: `notmuch-show-get-message-id': optionally return Message-Id sans prefix

Date: Wed, 29 Feb 2012 08:19:34 -0400

To: Pieter Praet

Cc: Notmuch Mail

From: David Bremner

On Tue, 28 Feb 2012 19:40:04 +0100, Pieter Praet <pieter@praet.org> wrote:
> 
> Though I couldn't help but notice (it's a curse...) that patch #3
> seems to have gained an extra space in its commit message subject:
> 
>   Original (id:"1329684627-10708-4-git-send-email-pieter@praet.org"):
>     emacs: add `notmuch-show-stash-mlarchive-link{,-and-go}'
>                                                   ^

I don't have any applypatch-msg hook. I am using 
  "notmuch show --format=raw $messageid | git am -" to
apply the patch.  

I looked at the file on disk, and something is wrapping the subject
there.  Perhaps somebody's unwrapping code is buggy? I'm not sure what's
at fault here; maybe you could experiment with applying the the patches
from the list and let me know if e.g. going from files is better than
using "notmuch show --format=raw".

d

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