Mailing-List: contact dev-help@ant.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Ant Developers List" <dev@ant.apache.org>
From: bugzilla@apache.org
To: dev@ant.apache.org
Subject: DO NOT REPLY [Bug 37328]  New:  -
    Definer blows away current thread task
Message-Id: <20051101230807.51FF259E@ajax.apache.org>
Date: Wed,  2 Nov 2005 00:08:07 +0100 (CET)

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=37328>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=37328

           Summary: Definer blows away current thread task
           Product: Ant
           Version: 1.6.5
          Platform: All
        OS/Version: All
            Status: NEW
          Keywords: PatchAvailable
          Severity: normal
          Priority: P3
         Component: Core
        AssignedTo: dev@ant.apache.org
        ReportedBy: carej@us.ibm.com


Copied from the developer mailing list:
---------------------------------------
All:

I've recently encountered some difficulties with the multithreaded build 
extensions that I've mentioned on the list before. 

As part of the Mantis project that we use to build WAS we have a custom 
logger. This logger uses a different banner format ([project/target/task]) 
from the default logger. The current problem is that the banners on 
messages from the sub-builds have the wrong context information; 
specifically they either no context (i.e. the banner is empty) or they 
have the parent project's context.

I've done a lot of debugging in the past few days and found that the 
parent context bit is due to a bug in our code, so mea culpa there. But 
the no context bit I traced to some code in oata.taskdefs.Definer. Definer 
creates an Antlib task instance to load our antlib. Now, creating a 
"delegate task" is a fairly common pattern; I do it in Mantis a lot. The 
problem (as I see it anyway) is that Definer calls "perform" instead of 
"execute" to make the Antlib instance do its work.  Because "perform" is 
used that blows away the currently registered task for the current thread. 
I've overridden Definer locally to call "execute" instead of "perform" and 
my problems seem to have cleared up with no side-effects. (Also 
interesting that this isn't really threading related - it happens even 
when everything is running on the main thread.)

So, getting to the point, is there a general policy regarding 
"execute"/"perform" for delegate task instances? Based on my experience I 
would think that "execute" would be preferred, so I wasn't sure if 
"perform" was required in this instance. I've prepared a patch for 
Definer, but I didn't want to open a bugzilla report until we had some 
discussion here on the list.

Cheers,
JEC
-- 
Jeffrey E. Care (carej@us.ibm.com)
WebSphere v7 Release Engineer
WebSphere Build Tooling Lead (Project Mantis)

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org