Return-Path: Delivered-To: apmail-incubator-geronimo-dev-archive@www.apache.org Received: (qmail 73083 invoked from network); 1 Sep 2003 18:52:25 -0000 Received: from daedalus.apache.org (HELO apache.org) (208.185.179.12) by minotaur-2.apache.org with SMTP; 1 Sep 2003 18:52:25 -0000 Received: (qmail 87070 invoked by uid 500); 1 Sep 2003 18:52:03 -0000 Delivered-To: apmail-incubator-geronimo-dev-archive@incubator.apache.org Received: (qmail 86917 invoked by uid 500); 1 Sep 2003 18:52:01 -0000 Mailing-List: contact geronimo-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: list-post: Reply-To: geronimo-dev@incubator.apache.org Delivered-To: mailing list geronimo-dev@incubator.apache.org Received: (qmail 86868 invoked from network); 1 Sep 2003 18:52:00 -0000 Received: from unknown (HELO reason.planet57.com) (202.183.214.7) by daedalus.apache.org with SMTP; 1 Sep 2003 18:52:00 -0000 Received: from coredevelopers.net (localhost [127.0.0.1]) by reason.planet57.com (Postfix) with ESMTP id 71BB846F8E7 for ; Tue, 2 Sep 2003 01:52:01 +0700 (ICT) Date: Tue, 2 Sep 2003 01:52:00 +0700 Mime-Version: 1.0 (Apple Message framework v552) Content-Type: text/plain; charset=US-ASCII; format=flowed Subject: [buildsystem] Added conditional test execution From: Jason Dillon To: geronimo-dev@incubator.apache.org Content-Transfer-Encoding: 7bit Message-Id: <5EAD55F8-DCAD-11D7-9831-000A9566A360@coredevelopers.net> X-Mailer: Apple Mail (2.552) X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N X-Spam-Rating: minotaur-2.apache.org 1.6.2 0/1000/N This is related to some comments made by Dain a while ago. I was spending entirely too much time waiting for tests to pass which had already passed before, so I added a some fluff to only run tests if bits under src/* have changed. I personally commented on this before and thought it was a bad idea. I objected because a depend could have changed and cause the tests to fail. I changed my thinking on this, in that the tests for a module are for that modules code and not the depend. If a depend module changes its tests should catch problems and such. Anyways, you can set -Dmaven.test.force=true to bypass the check and get the default maven behavior. Tests are a good thing and we should add more of them, but that should not be a hindrance to the build in general, so I added this hack to bypass if the tests ran successfully and none of the module's sources changed. --jason