Return-Path: X-Original-To: notmuch@notmuchmail.org Delivered-To: notmuch@notmuchmail.org Received: from localhost (localhost [127.0.0.1]) by olra.theworths.org (Postfix) with ESMTP id 313394196F2 for ; Sat, 24 Apr 2010 14:45:47 -0700 (PDT) X-Virus-Scanned: Debian amavisd-new at olra.theworths.org X-Spam-Flag: NO X-Spam-Score: -2.89 X-Spam-Level: X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5 tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01] autolearn=ham Received: from olra.theworths.org ([127.0.0.1]) by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id gO1KA9hMpoPA; Sat, 24 Apr 2010 14:45:45 -0700 (PDT) Received: from yoom.home.cworth.org (localhost [127.0.0.1]) by olra.theworths.org (Postfix) with ESMTP id C8489431FC1; Sat, 24 Apr 2010 14:45:45 -0700 (PDT) Received: by yoom.home.cworth.org (Postfix, from userid 1000) id 7DF24568DEC; Sat, 24 Apr 2010 14:45:45 -0700 (PDT) From: Carl Worth To: Dirk Hohndel , notmuch@notmuchmail.org Subject: Re: Wrapping up the 0.3 release In-Reply-To: References: <87ljcchnfc.fsf@yoom.home.cworth.org> User-Agent: Notmuch/0.2-188-g9757095 (http://notmuchmail.org) Emacs/23.1.1 (i486-pc-linux-gnu) Date: Sat, 24 Apr 2010 14:45:45 -0700 Message-ID: <87aassh6d2.fsf@yoom.home.cworth.org> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha1; protocol="application/pgp-signature" X-BeenThere: notmuch@notmuchmail.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: "Use and development of the notmuch mail system." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 24 Apr 2010 21:45:47 -0000 --=-=-= Content-Transfer-Encoding: quoted-printable On Sat, 24 Apr 2010 09:53:17 -0700, Dirk Hohndel wr= ote: > On Sat, 24 Apr 2010 08:37:11 -0700, Carl Worth wrote: > > I sent a patch last night - but it's not realtive to the last thing that > I sent, instead relative to last night's master. Do you want me to > create another one? No, what you sent last night is perfect. That will be easier for me. > 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. 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. > 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. 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. Thanks, =2DCarl =2D-=20 carl.d.worth@intel.com --=-=-= Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) iD8DBQFL02aJ6JDdNq8qSWgRAohHAKCX02UzOwZ8xoxjFa7oHldGF16tdACfb+Hz tmq8YaVSlpHHY4YCP4sPdpo= =n0+3 -----END PGP SIGNATURE----- --=-=-=--