RE: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java

2004-05-28 Thread Dominique Devienne
> -Original Message- > From: Jose Alberto Fernandez [mailto:[EMAIL PROTECTED] > > Not sure, in particular in Java you may et into funny class > dependencies. > > In principle, you can have circular dependencies between jars, that is > solved > by loading both jars in the same classloader

RE: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java

2004-05-28 Thread Jose Alberto Fernandez
could get to the same situation with antlibs which are just jars with descriptors. Jose Alberto > -Original Message- > From: Dominique Devienne [mailto:[EMAIL PROTECTED] > Sent: 28 May 2004 16:38 > To: 'Ant Developers List' > Subject: RE: cvs commit: > ant/s

RE: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java

2004-05-28 Thread Dominique Devienne
> From: Jose Alberto Fernandez [mailto:[EMAIL PROTECTED] > > We do barf on targets with circular dependencies, so why wouldn't > > we barf for circular AntLib dependencies? > > Remember that in general you have no control of the dependencies > of Antlibs (since they may be shrink-wrap 3rd party bi

RE: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java

2004-05-28 Thread Jose Alberto Fernandez
> From: Dominique Devienne [mailto:[EMAIL PROTECTED] > > > From: Peter Reilly [mailto:[EMAIL PROTECTED] > > This could be the case for , but this code is for > file="x.xml"/> and , which > is possibly > > more likely to be an error to have circular loading. > > Ah, it make sense now. So it's

RE: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java

2004-05-28 Thread Dominique Devienne
> From: Peter Reilly [mailto:[EMAIL PROTECTED] > >Could the message somehow make it more explicit it's ignored? Like: > Ok, > log("Warning: Recursive loading of " + url > + " ignored" > + " at " + getLocation(), Sounds

Re: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java

2004-05-28 Thread Peter Reilly
Dominique Devienne wrote: -Original Message- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] peterreilly2004/05/28 01:20:22 Modified:src/main/org/apache/tools/ant/taskdefs Definer.java Log: typedef: check for recursive loading of xml files/resources (aka antlibs) [...

RE: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java

2004-05-28 Thread Dominique Devienne
> -Original Message- > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] > > peterreilly2004/05/28 01:20:22 > > Modified:src/main/org/apache/tools/ant/taskdefs Definer.java > Log: > typedef: check for recursive loading of xml >files/resources (aka antlibs) > [...