AW: svn commit: r557907 - /ant/core/trunk/docs/manual/install.html

2007-07-31 Thread Jan.Materne
Thanks - I'll try that in future commits ;-) Jan >-Ursprüngliche Nachricht- >Von: Stefan Bodewig [mailto:[EMAIL PROTECTED] >Gesendet: Dienstag, 31. Juli 2007 21:36 >An: dev@ant.apache.org >Betreff: Re: svn commit: r557907 - >/ant/core/trunk/docs/manual/install.html > >On Fri, 20 Jul 20

Last chance: vmbuild migration & ant nightlies

2007-07-31 Thread Brett Porter
Hi Antoine, all, A reminder from my earlier mail... Currently, it appears that Ant is running nightly builds on vmbuild, which is being migrated to a new (better) machine. Is this process still needed, or does anything else from ~antoine need to be recovered before the old instance is turn

Re: is broken with Jython in the recent trunk builds

2007-07-31 Thread Matt Benson
--- Alexey Solofnenko <[EMAIL PROTECTED]> wrote: > Sorry about that. Personally I would prefer > "expression" attribute to > contain an expression itself - it is supposed to be > simple. > That still doesn't address the situation I mentioned, whereby some arbitrary scripty stuff is done to calc

Re: is broken with Jython in the recent trunk builds

2007-07-31 Thread Alexey Solofnenko
Sorry about that. Personally I would prefer "expression" attribute to contain an expression itself - it is supposed to be simple. - Alexey. On 7/31/07, Matt Benson <[EMAIL PROTECTED]> wrote: > > > --- Alexey Solofnenko <[EMAIL PROTECTED]> wrote: > > > It is just a thought - should ScriptFilter, >

Re: is broken with Jython in the recent trunk builds

2007-07-31 Thread Matt Benson
--- Alexey Solofnenko <[EMAIL PROTECTED]> wrote: > It is just a thought - should ScriptFilter, > ScriptMapper, ... also return a > value instead of setting it? > I don't know! :) In hindsight maybe it would have been better to leave this can of worms unmolested rather than deal with

Re: is broken with Jython in the recent trunk builds

2007-07-31 Thread Alexey Solofnenko
It is just a thought - should ScriptFilter, ScriptMapper, ... also return a value instead of setting it? - Alexey. On 7/31/07, Matt Benson <[EMAIL PROTECTED]> wrote: > > > --- Alexey Solofnenko <[EMAIL PROTECTED]> wrote: > > > Old code was executing self.setValue() and the > > current behaviour b

Re: is broken with Jython in the recent trunk builds

2007-07-31 Thread Matt Benson
--- Alexey Solofnenko <[EMAIL PROTECTED]> wrote: > Old code was executing self.setValue() and the > current behaviour breaks > backward compatibility. I have tried old > ScriptCondition.eval() and it fixed > the problem. I think we should add "expression" > attribute to > AbstractScriptComponent

Re: Functional testing with AntUnit

2007-07-31 Thread James Fuller
probably won't help you directly, but if you want to see some more antunit examples in real situations, I recently generated an antunit test suite for the eXist XMLDB project (exist.sourceforge.net) https://sourceforge.net/tracker/?func=detail&atid=317691&aid=1743796&group_id=17691 eXist XMLDB ha

Re: is broken with Jython in the recent trunk builds

2007-07-31 Thread Alexey Solofnenko
Old code was executing self.setValue() and the current behaviour breaks backward compatibility. I have tried old ScriptCondition.eval() and it fixed the problem. I think we should add "expression" attribute to AbstractScriptComponent and change it to use it with evaluateScript(), otherwise nested t

Re: Functional testing with AntUnit

2007-07-31 Thread Stefan Bodewig
On Tue, 17 Jul 2007, Chris Connelly <[EMAIL PROTECTED]> wrote: > I apologize in advance if this is the wrong forum for this question. This is the best forum available. > My questions: the AntUnit home page says: "AntUnit could also be > used for functional and integration tests outside of the sc

Re: svn commit: r557907 - /ant/core/trunk/docs/manual/install.html

2007-07-31 Thread Alexey Solofnenko
Yes it is and it keeps track of branch merges, so the merges are trivial. - Alexey. On 7/31/07, Stefan Bodewig <[EMAIL PROTECTED]> wrote: > > On Fri, 20 Jul 2007, Peter Reilly <[EMAIL PROTECTED]> wrote: > > > I know that there is an easy way (using svn merge), but > > I do not know how to make it

Re: svn commit: r554390 - /ant/core/trunk/src/main/org/apache/tools/ant/types/Resource.java

2007-07-31 Thread Matt Benson
--- Stefan Bodewig <[EMAIL PROTECTED]> wrote: > On Tue, 31 Jul 2007, Matt Benson > <[EMAIL PROTECTED]> wrote: > > --- Stefan Bodewig <[EMAIL PROTECTED]> wrote: > > >> It's not too important to me, but in a way I'd > prefer to stick with > >> 80 chars. > >> > > > > Yeah, checkstyle is set for 1

Re: svn commit: r554390 - /ant/core/trunk/src/main/org/apache/tools/ant/types/Resource.java

2007-07-31 Thread Stefan Bodewig
On Tue, 31 Jul 2007, Matt Benson <[EMAIL PROTECTED]> wrote: > --- Stefan Bodewig <[EMAIL PROTECTED]> wrote: >> It's not too important to me, but in a way I'd prefer to stick with >> 80 chars. >> > > Yeah, checkstyle is set for 100... Peter and I > discussed this; I saw that several mails later,

Re: svn commit: r557907 - /ant/core/trunk/docs/manual/install.html

2007-07-31 Thread Stefan Bodewig
On Fri, 20 Jul 2007, Peter Reilly <[EMAIL PROTECTED]> wrote: > I know that there is an easy way (using svn merge), but > I do not know how to make it work. svn merge -r REVISION_BEFORE_COMMIT:REVISION_AFTER_COMMIT \ http:///trunk performed inside the branch directory after the commit in

Re: svn commit: r554390 - /ant/core/trunk/src/main/org/apache/tools/ant/types/Resource.java

2007-07-31 Thread Matt Benson
--- Stefan Bodewig <[EMAIL PROTECTED]> wrote: > On Sun, 08 Jul 2007, <[EMAIL PROTECTED]> wrote: > > > URL: > http://svn.apache.org/viewvc?view=rev&rev=554390 > > Log: > > fmt > > These changes make lines longer than 80 chars that > used to be our > standard. > > It's not too important to me, b

Re: svn commit: r554390 - /ant/core/trunk/src/main/org/apache/tools/ant/types/Resource.java

2007-07-31 Thread Stefan Bodewig
On Sun, 08 Jul 2007, <[EMAIL PROTECTED]> wrote: > URL: http://svn.apache.org/viewvc?view=rev&rev=554390 > Log: > fmt These changes make lines longer than 80 chars that used to be our standard. It's not too important to me, but in a way I'd prefer to stick with 80 chars. Stefan

DO NOT REPLY [Bug 43004] - OutOfMemory caused by System.out when running simple JUnit test in .

2007-07-31 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT . ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE. http://issues.apache.org/bugzilla/show_bu

DO NOT REPLY [Bug 43004] - OutOfMemory caused by System.out when running simple JUnit test in .

2007-07-31 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT . ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE. http://issues.apache.org/bugzilla/show_bu

DO NOT REPLY [Bug 43004] New: - OutOfMemory caused by System.out when running simple JUnit test in .

2007-07-31 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT . ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE. http://issues.apache.org/bugzilla/show_bu

Re: is broken with Jython in the recent trunk builds

2007-07-31 Thread Peter Reilly
For BSF there are two methods to run a script: eval and exec, In ant 1.6.* the only method supported was exec. Hence all the types called methods on self to set the return value. For ant 1.7.0, I modified the scripting code to allow access to eval and exec, but did not modify any calling types

Re: is broken with Jython in the recent trunk builds

2007-07-31 Thread Dominique Devienne
On 7/31/07, Matt Benson <[EMAIL PROTECTED]> wrote: > > PS: The error message could be nicer though ;-) > > Which error message? The one Alexey is getting from > the jython problem? Yeah, but I now see that you probably can't do much about it though. --DD -

Re: is broken with Jython in the recent trunk builds

2007-07-31 Thread Matt Benson
--- Dominique Devienne <[EMAIL PROTECTED]> wrote: > On 7/31/07, Matt Benson <[EMAIL PROTECTED]> > wrote: > > originally behaved such that a > > default value can be declared on the task as an > > attribute, and the embedded script can set the > > condition value. I preserved this behavior, but

Re: is broken with Jython in the recent trunk builds

2007-07-31 Thread Matt Benson
--- Dominique Devienne <[EMAIL PROTECTED]> wrote: > > > Alexey N. Solofnenko wrote: > > > Recent trunk ANT fails in my builds. > > > > > > > > > > > > > > > > Matt Benson wrote: > > Fairly recently I added return handling to > > scriptcondition from the perspective that it was > more > >

Re: is broken with Jython in the recent trunk builds

2007-07-31 Thread Dominique Devienne
On 7/31/07, Matt Benson <[EMAIL PROTECTED]> wrote: > originally behaved such that a > default value can be declared on the task as an > attribute, and the embedded script can set the > condition value. I preserved this behavior, but added > a preference for a return value, if any, from the > scri

Re: is broken with Jython in the recent trunk builds

2007-07-31 Thread Dominique Devienne
> > Alexey N. Solofnenko wrote: > > Recent trunk ANT fails in my builds. > > > > > > > > > > Matt Benson wrote: > Fairly recently I added return handling to > scriptcondition from the perspective that it was more > natural to treat the whole construct like a function... Well, what's the po

Re: is broken with Jython in the recent trunk builds

2007-07-31 Thread Matt Benson
--- Steve Loughran <[EMAIL PROTECTED]> wrote: > Alexey N. Solofnenko wrote: > > Recent trunk ANT fails in my builds because > does > > something with the script that Jython cannot > digest (new lines?). > > with the same code works fine. This is a > test script: > > > > > >

DO NOT REPLY [Bug 41024] - Problem with Input task

2007-07-31 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT . ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE. http://issues.apache.org/bugzilla/show_bu

Re: is broken with Jython in the recent trunk builds

2007-07-31 Thread Steve Loughran
Alexey N. Solofnenko wrote: Recent trunk ANT fails in my builds because does something with the script that Jython cannot digest (new lines?). with the same code works fine. This is a test script: