Will Glass-Husain <[EMAIL PROTECTED]> wrote: > As a side note, I accidentally just made the patch to testcases.xml that > requires ant 1.5+. (Or more precise, has only been tested with ant 1.5+).
> I'll leave it in for the moment so we can see if this fixes gump. > > I'll pull it out if their are objections to my proposal. Dependencies required for the development of velocity should not be as strictly limited as dependencies required to run Velocity. Developers should have a great deal more flexibility than deployments. I see no problems with ant 1.5 for testing. Actually, I don't see any problems with it for Velocity users either, as using Velocity doesn't require using ant. -Mike > ----- Original Message ----- > From: "Will Glass-Husain" <[EMAIL PROTECTED]> > To: <[email protected]> > Sent: Monday, February 14, 2005 11:56 PM > Subject: require ant 1.5+ ? > > > > Hi, > > > > Any objections if we raise the minimum required ant version to 1.5.1? > > (Currently our docs read minimum ant 1.3) For the record, ant 1.5 was > > released more than 2 years ago (October 2002). > > > > The reason is texen compatibility. The latest update to Velocity [1] > > changed the way the classes are loaded. This breaks "ant test" under > > gump. > > If we remove the archaic, undocumented ant attribute "reverseloader" this > > is fixed. (See my email of 2/7 for details). Then we can rid of those > > annoying nag messages from Gump. > > > > I've tested this with ant 1.6 and ant 1.5. With both cases "ant > > test-texen-classpath" works if we remove the reverseloader attribute. I > > can't test ant 1.3 and ant 1.4 as they are incompatible with JDK 1.4.2 > > which I'm using. [2] Earlier versions of ant will probably continue to > > work with everything except Texen. > > > > Incidentally, if we raise the minimum bar to ant 1.6, we can include the > > javacc task to automatically build the parser code [3]. Of course, we can > > include this anyway and just complain if the version of ant is too early. > > > > > > WILL > > > > [1] http://issues.apache.org/bugzilla/show_bug.cgi?id=31721 > > > > [2] > > http://www.mail-archive.com/[email protected]/msg13724.html > > > > [3] http://issues.apache.org/bugzilla/show_bug.cgi?id=24341 > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
