Return-Path: Delivered-To: apmail-ant-dev-archive@www.apache.org Received: (qmail 55384 invoked from network); 21 Sep 2007 08:34:25 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 21 Sep 2007 08:34:25 -0000 Received: (qmail 21516 invoked by uid 500); 21 Sep 2007 08:33:58 -0000 Delivered-To: apmail-ant-dev-archive@ant.apache.org Received: (qmail 21477 invoked by uid 500); 21 Sep 2007 08:33:58 -0000 Mailing-List: contact dev-help@ant.apache.org; run by ezmlm Precedence: bulk List-Unsubscribe: List-Help: List-Post: List-Id: "Ant Developers List" Reply-To: "Ant Developers List" Delivered-To: mailing list dev@ant.apache.org Received: (qmail 21434 invoked by uid 99); 21 Sep 2007 08:33:58 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 21 Sep 2007 01:33:58 -0700 X-ASF-Spam-Status: No, hits=-0.0 required=10.0 tests=SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of peter.kitt.reilly@gmail.com designates 209.85.198.190 as permitted sender) Received: from [209.85.198.190] (HELO rv-out-0910.google.com) (209.85.198.190) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 21 Sep 2007 08:33:55 +0000 Received: by rv-out-0910.google.com with SMTP id k15so698211rvb for ; Fri, 21 Sep 2007 01:33:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=beta; h=domainkey-signature:received:received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; bh=X+gvUvDOr6ykuxt2s7A/WZbhqJ5FMErWqLdMirU/yho=; b=pEnMvGbeTfJZc/REQpKDVh+iMJ/si6nZiviR31IxJ4JNlgzcS+98wVS67K865KoXRkq2QL3bcjbplzGZeDkDlXXoQjIqcXB+S7Ae7oGh4s63PAbD/pBt2FUNf+lI4ZR6mZ9AOteb07Vu55FzFHVLbF5yAVGSVe0BUFBxpKQ+RQk= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=ABXqh8g4Rsi5jw9uCZjwvhohFCHG00rroXc0+3Veer032FgXpM6xSo+cD2k2vuY2XyWtJFlH8wZYmxFWudkl18jc2WIry2jbRrX7ioMiI36lcrLAwdNSJ9oxs7aVjR+MUOKrJi0jx26A5oIeTa/RvRAEJ8K3yEFoawXObk0TKPs= Received: by 10.114.197.1 with SMTP id u1mr3222800waf.1190363611459; Fri, 21 Sep 2007 01:33:31 -0700 (PDT) Received: by 10.114.198.12 with HTTP; Fri, 21 Sep 2007 01:33:31 -0700 (PDT) Message-ID: Date: Fri, 21 Sep 2007 09:33:31 +0100 From: "Peter Reilly" To: "Ant Developers List" Subject: Re: AW: ant really slow, 100% CPU? In-Reply-To: <46F29168.6060404@apache.org> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <46F12A94.6070107@apache.org> <46F27B02.3080200@apache.org> <255d8d690709200819i5e58f92bh7e2ea08a289187ab@mail.gmail.com> <46F29168.6060404@apache.org> X-Virus-Checked: Checked by ClamAV on apache.org 1) Some of the traces seem to imply a bug. org.apache.tools.ant.taskdefs.Property$PropertyResolver.evaluate(Property.java:110) at org.apache.tools.ant.PropertyHelper.getProperty(PropertyHelper.java:787) - locked <0x00002aaab4013548> (a org.apache.tools.ant.PropertyHelper) at org.apache.tools.ant.PropertyHelper.parseNextProperty(PropertyHelper.java:541) at org.apache.tools.ant.PropertyHelper.parseProperties(PropertyHelper.java:502) at org.apache.tools.ant.RuntimeConfigurable.maybeConfigure(RuntimeConfigurable.java:390) The intent of Property.PropertyResolver is I think to remain in the Property task - the trace seems to imply that it has escaped from that task. 2) some of the new code seems to be very inefficent: private void resolveAllProperties(Map props) throws BuildException { PropertyHelper propertyHelper = (PropertyHelper) PropertyHelper.getPropertyHelper( getProject()).clone(); propertyHelper.add(new PropertyResolver(props)); This means that when one does , for each env variable, the propertyhelper is cloned. 3) The reason for this is to allow different property demarcations - using ProperyExpander's. The use-case for this is to allow "${${a}-${b}}" type expansion (this is a much requested enhancement). and in something like a properties file: action=cut cut=the cut action do=the action is ${${action}} paste=the paste action can get difficult to parse. The new code does this is a nice way (except for the ThreadLocal). Peter On 9/20/07, Steve Loughran wrote: > Dominique Devienne wrote: > > On 9/20/07, Steve Loughran wrote: > >> it appears to hang at 100% CPU during gzip > >> > >> release.gzip-rpm: > >> [tar] Building tar: ... > >> [gzip] Building:.../smartfrog-rpm-bundle-3.12.003dev.gz > >> > >> through the miracle that is kill -QUIT, here's the thread dump > > > > Did I miss the gzip task in the stack trace? In both dumps you > > provided, I see only property stuff, so how does that relate to > > gzip??? Thanks, --DD > > > > nothing, its just after that task that everything seems to lock up. > > But sometimes it jams in other places; other times it just keeps going. > That is all the sign of race conditions to me: inconsistent behaviour It appears that the new code for > -- > Steve Loughran http://www.1060.org/blogxter/publish/5 > Author: Ant in Action http://antbook.org/ > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org > For additional commands, e-mail: dev-help@ant.apache.org > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org For additional commands, e-mail: dev-help@ant.apache.org