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 579044196F2 for ; Wed, 5 May 2010 00:24:02 -0700 (PDT) X-Virus-Scanned: Debian amavisd-new at olra.theworths.org X-Spam-Flag: NO X-Spam-Score: 0.8 X-Spam-Level: X-Spam-Status: No, score=0.8 tagged_above=-999 required=5 tests=[BAYES_50=0.8] 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 D1GDSRf4o5wG for ; Wed, 5 May 2010 00:24:01 -0700 (PDT) Received: from max.feld.cvut.cz (max.feld.cvut.cz [147.32.192.36]) by olra.theworths.org (Postfix) with ESMTP id 4AF584196F0 for ; Wed, 5 May 2010 00:24:01 -0700 (PDT) Received: from localhost (unknown [192.168.200.4]) by max.feld.cvut.cz (Postfix) with ESMTP id 65D7E19F340B; Wed, 5 May 2010 09:24:00 +0200 (CEST) X-Virus-Scanned: IMAP AMAVIS Received: from max.feld.cvut.cz ([192.168.200.1]) by localhost (styx.feld.cvut.cz [192.168.200.4]) (amavisd-new, port 10044) with ESMTP id OKJzucNZPh0k; Wed, 5 May 2010 09:23:59 +0200 (CEST) Received: from imap.feld.cvut.cz (imap.feld.cvut.cz [147.32.192.34]) by max.feld.cvut.cz (Postfix) with ESMTP id D9C2B19F3378; Wed, 5 May 2010 09:23:58 +0200 (CEST) Received: from steelpick.2x.cz (k335-30.felk.cvut.cz [147.32.86.30]) (Authenticated sender: sojkam1) by imap.feld.cvut.cz (Postfix) with ESMTPSA id 4366BFA003; Wed, 5 May 2010 09:23:58 +0200 (CEST) Received: from wsh by steelpick.2x.cz with local (Exim 4.71) (envelope-from ) id 1O9YxZ-0004NY-W8; Wed, 05 May 2010 09:23:58 +0200 From: Michal Sojka To: Jesse Rosenthal , Notmuch developer list Subject: Re: [PATCH] Modularize test suite In-Reply-To: <87sk6cdbxz.fsf@jhu.edu> References: <87sk6cdbxz.fsf@jhu.edu> User-Agent: Notmuch/0.3.1-31-g60b2517 (http://notmuchmail.org) Emacs/23.1.1 (x86_64-pc-linux-gnu) Date: Wed, 05 May 2010 09:23:57 +0200 Message-ID: <87bpcustf6.fsf@steelpick.2x.cz> 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, 05 May 2010 07:24:02 -0000 On Fri, 30 Apr 2010, Jesse Rosenthal wrote: > Responding to this email is a patch to modularize the test suite. It > follows a relatively simple model, but it does seem like it will offer > some degree of flexibility. Plus, no relicensing is necessary. Hi Jesse, Junio has just applied relicensing patch for the Git test suite (http://git.kernel.org/?p=git/git.git;a=commit;h=64b90323f65d2a3ac8d1ff4ed73be881f76aac8a) so we can use it for notmuch. I have quickly looked at your modularization and I do not like that each test resided in a separate file. I've done similar test suite for one of my projects and it is a pain to modify the tests. If one test depends on the previous ones, you always have to switch between multiple files in your editor. If you agree, I can refresh my git based modularization and send the patches. What do you (and others) think? Michal