Return-Path: Delivered-To: apmail-jakarta-ant-dev-archive@apache.org Received: (qmail 88946 invoked from network); 24 May 2002 14:39:21 -0000 Received: from unknown (HELO nagoya.betaversion.org) (192.18.49.131) by daedalus.apache.org with SMTP; 24 May 2002 14:39:21 -0000 Received: (qmail 4700 invoked by uid 97); 24 May 2002 14:39:19 -0000 Delivered-To: qmlist-jakarta-archive-ant-dev@jakarta.apache.org Received: (qmail 4655 invoked by uid 97); 24 May 2002 14:39:18 -0000 Mailing-List: contact ant-dev-help@jakarta.apache.org; run by ezmlm Precedence: bulk List-Unsubscribe: List-Subscribe: List-Help: List-Post: List-Id: "Ant Developers List" Reply-To: "Ant Developers List" Delivered-To: mailing list ant-dev@jakarta.apache.org Received: (qmail 4636 invoked by uid 98); 24 May 2002 14:39:18 -0000 X-Antivirus: nagoya (v4198 created Apr 24 2002) X-Authentication-Warning: costinm.sfo.covalent.net: costin owned process doing -bs Date: Fri, 24 May 2002 07:36:43 -0700 (PDT) From: costinm@covalent.net X-X-Sender: costin@costinm.sfo.covalent.net To: Ant Developers List Subject: Re: [PATCH] Definer to allow types/tasks from the same jar to work In-Reply-To: <009601c2032b$c80d7f40$8cc8a8c0@manu.com> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N Thanks! Based on the feedback I got so far from Peter and Conor, I'll leave the default behavior unchanged ( i.e. different class loaders ). I'll also change the name of the attribute to 'loaderId' - and make sure this would work with a type. Conor - I'm not sure we should add the type. Let me know if you think it's required, it's quite easy to implement. Also for Conor - I really want to keep the magic property for backward compatibility ( so a build.xml for 1.5 can be used with 1.4 ). After I commit the patch I'll update the docs for taskdef/typedef - if you want I can only include the new attribute and leave the magic property undocumented. Costin On Fri, 24 May 2002, Magesh Umasankar wrote: > ok, you have convinced me it is a bug :-) > +1 to patch a fix. wrt the actual patch that > you have provided, I will leave it to others > to make the call whether it can go in, in its > current avatar. I am +0 on it as I have not > taken a close look at it yet and I realize > it must get in in short order, if it is gonna > make it to Beta 2. > > Cheers, > Magesh > > ************************************************ > * Criminal: A guy no different from the rest * > * ...except that he got caught. * > ************************************************ > ----- Original Message ----- > From: > To: "Ant Developers List" > Sent: Friday, May 24, 2002 9:49 AM > Subject: Re: [PATCH] Definer to allow types/tasks from the same jar to work > > > > On Fri, 24 May 2002, Magesh Umasankar wrote: > > > > > It is a nice-to-have, yes, but I don't see it as a bug > > > in 1.5. At least I would need some convincing to call > > > it a bug ;-). Depending upon that, it may or may not > > > go into 1.5 branch. > > > > Acording to ant 1.5 docs, you can define custom tasks and types > > with and . It is also a documented feature that > > both taskdef and typedef can use a user-defined classpath to > > load the implementation classes. > > > > If a user-defined task uses a user-defined type you get a > > ClassCastException. > > > > To me that means that / with user-defined > > classpath is broken. > > > > Please take a look at jakarta-tomcat-connectors/jk/native2/build.xml > > and apache2-cpptasks target as an example. It won't work with ant1.5 > > without the patch. > > > > Costin > > > > > > > > -- > > To unsubscribe, e-mail: > > For additional commands, e-mail: > > > > > -- > To unsubscribe, e-mail: > For additional commands, e-mail: > > -- To unsubscribe, e-mail: For additional commands, e-mail: