Wrapping up the 0.3 release
Dirk Hohndel
hohndel at infradead.org
Sat Apr 24 15:05:55 PDT 2010
On Sat, 24 Apr 2010 14:45:45 -0700, Carl Worth <cworth at cworth.org> wrote:
> > It doesn't for me with origin/master. Or let me double check... what do
> > you think would be the correct order (as this is a matter of taste for
> > some people)...
>
> The order in the reply buffer is fine. But with "m" I get the User-Agent
> first which looks a bit strange.
Yep, same here.
> Dirk also mentioned in IRC that there's a regression with the signature
> being mispositioned before the quoted text with a reply buffer. Now that
> I've added a signature, I'm noticing this as well.
Well - we don't seem to be adding the signature ourselves anymore... I
still don't quite understand where and how we hand over to the existing
message-mode functions - I why those decide to insert a signature at
point.
Here's a trivial patch that ALSO adds a signature at the end of the
message buffer (where it belongs). But I haven't figured out how to get
rid of the one above the reply...
diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el
index acb7dbf..493cd0e 100644
--- a/emacs/notmuch-mua.el
+++ b/emacs/notmuch-mua.el
@@ -63,6 +63,7 @@
;; line and then the body.
(with-temp-buffer
(call-process notmuch-command nil t nil "reply" query-string)
+ (message-insert-signature)
(goto-char (point-min))
(if (re-search-forward "^$" nil t)
(save-excursion
> > I think we should make this a "requirement" for patches to include a
> > little NEWS blurb and either a test case or an explanation why there
> > isn't a test case...
>
> I've asked for these, but I haven't been pushing hard on this.
I will start playing the nagger
> Review for some of these simple things would be much appreciated from
> anybody on the list, (and would help ensure that patches are more likely
> to be ready-to-go once I get them). So let's see more of things like
> this from anyone on the list:
>
> Looks like a great feature---now it just needs a test case.
>
> I've tested this and it does just what I want. Here's a
> follow-on patch that adds an item to the NEWS file for this.
>
> I can't common on the specific logic of the patch, but I did
> notice some trailing whitespace. You'll want to clean that up
> and resubmit so the patch won't be rejected.
I can do all of those.
/D
--
Dirk Hohndel
Intel Open Source Technology Center
More information about the notmuch
mailing list