Return-Path: Delivered-To: apmail-jakarta-ant-dev-archive@jakarta.apache.org Received: (qmail 32982 invoked by uid 500); 5 Jul 2001 06:56:08 -0000 Mailing-List: contact ant-dev-help@jakarta.apache.org; run by ezmlm Precedence: bulk Reply-To: ant-dev@jakarta.apache.org list-help: list-unsubscribe: list-post: Delivered-To: mailing list ant-dev@jakarta.apache.org Received: (qmail 32965 invoked from network); 5 Jul 2001 06:56:07 -0000 Content-Type: text/plain; charset="iso-8859-1" From: Peter Donald To: ant-dev@jakarta.apache.org Subject: Re: [Patch] Target attribute property expansion Date: Thu, 5 Jul 2001 16:48:32 +1000 X-Mailer: KMail [version 1.2] References: <000901c10472$8bef6770$d900a8c0@viquity.com> In-Reply-To: MIME-Version: 1.0 Message-Id: <01070516483201.02692@helm.realityforge.org> Content-Transfer-Encoding: 8bit X-Spam-Rating: h31.sny.collab.net 1.6.2 0/1000/N On Wed, 4 Jul 2001 23:48, Stefan Bodewig wrote: > > For what I saw of the patch, it assumes that properties will be > > evaluated as they are parsed. Since as you say it preserves DAG > > being created at parsing time. That would mean that we will be stuck > > on having to execute directives while we parse, instead > > of in an initialization phase. Am I wrong in my understanding? > > Either that or construct the DAG after that initialization phase > (where initialization phase would execute all tasks that have not been > wrapped into targets, if they exist). I actually think that we should do lazy evaluation of the DAG. ie only look up next element in DAG as we "get" to it. This is for two reasons mainly; 1. Support crazy self-modifying projects like Jose wanted 2. Not force the loading of all project files at startup 3. Cutdown on redundent scanning at start that is not needed and costly in large projects Cheers, Pete *-----------------------------------------------------* | "Faced with the choice between changing one's mind, | | and proving that there is no need to do so - almost | | everyone gets busy on the proof." | | - John Kenneth Galbraith | *-----------------------------------------------------*