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 8B1E4429E21 for ; Wed, 16 Nov 2011 05:04:06 -0800 (PST) X-Virus-Scanned: Debian amavisd-new at olra.theworths.org X-Spam-Flag: NO X-Spam-Score: -0.7 X-Spam-Level: X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5 tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled 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 a2BAOiAKGq5a for ; Wed, 16 Nov 2011 05:04:06 -0800 (PST) Received: from mail-wy0-f181.google.com (mail-wy0-f181.google.com [74.125.82.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by olra.theworths.org (Postfix) with ESMTPS id BEAEF431FB6 for ; Wed, 16 Nov 2011 05:04:05 -0800 (PST) Received: by wyg19 with SMTP id 19so475275wyg.26 for ; Wed, 16 Nov 2011 05:04:04 -0800 (PST) Received: by 10.180.81.73 with SMTP id y9mr35459759wix.37.1321448644212; Wed, 16 Nov 2011 05:04:04 -0800 (PST) Received: from localhost ([109.131.148.49]) by mx.google.com with ESMTPS id gf10sm6630561wib.2.2011.11.16.05.04.02 (version=TLSv1/SSLv3 cipher=OTHER); Wed, 16 Nov 2011 05:04:03 -0800 (PST) From: Pieter Praet To: David Bremner , Thomas Jost , notmuch@notmuchmail.org Subject: Re: [PATCH 00/13] Test prereqs and screen-based Emacs tests In-Reply-To: <871utb6hl5.fsf@zancas.localnet> References: <871uuuywtt.fsf@convex-new.cs.unb.ca> <1317660447-27520-1-git-send-email-schnouki@schnouki.net> <87ehxrvcs6.fsf@praet.org> <871utb6hl5.fsf@zancas.localnet> User-Agent: Notmuch/0.9+76~g2fd88e6 (http://notmuchmail.org) Emacs/23.3.1 (x86_64-unknown-linux-gnu) Date: Wed, 16 Nov 2011 14:03:04 +0100 Message-ID: <874ny4fcdz.fsf@praet.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii 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: Wed, 16 Nov 2011 13:04:06 -0000 On Sun, 13 Nov 2011 13:46:30 -0400, David Bremner wrote: > On Tue, 01 Nov 2011 20:54:49 +0100, Pieter Praet wrote: > > > > I'll be commenting on these modified commits where needed, and have > > re-submitted my original series (rebased to current master) in a new > > thread [1]. > > > > I'm having trouble sorting out which of these series should be > considered for notmuch master. Is there something in the rebased version > of Pieter's patches that is not present in Thomas's rebased version? > Nope, I simply rebased my original series, warts and all. Thomas' submission contains a number of fixes (missing prereqs) and improvements (test_set_bin_prereq) that should definitely be included before the series is applied, as well as some stuff that belongs in separate commits (e.g. screen prereq, which should now check for dtach instead) and a tiny mistake in the "Reply within emacs" subtest [1]. Also, both Jameson [2] and Ali [3] have brought up some valid points/concerns re the way we find and set prereqs. Thomas, could you please rebase your fixes on top of my rebased series, and submit them in the new thread [4] ? The whole deal could then be applied on a feature branch and merged into mainline integrally. > d Peace -- Pieter [1] id:"87fwi7obso.fsf@praet.org" [2] id:"87hb2n4k5c.fsf@servo.finestructure.net" [3] id:"20111101202025.GA8248@hayalet" [4] id:"1320176954-4897-1-git-send-email-pieter@praet.org"