DO NOT REPLY [Bug 29264] New: - Ant task to iterate over a list of files
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=29264. 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=29264 Ant task to iterate over a list of files Summary: Ant task to iterate over a list of files Product: Ant Version: 1.5 Platform: All OS/Version: All Status: NEW Severity: Enhancement Priority: Other Component: Optional Tasks AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] It would be nice to have an Ant task that can iterate over a list of files (or all files in a given directory) and execute a particular command/sub-task. This is different from existing javac/java tasks that inherently do this over a list of files. This reequest is to invoke any arbitrary task. For instance if I want to invoke an exec on a list of files. If this can also support parallelism by creting as many threads as possible to run thru thru the list, it would be helpful. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
RE: [GUMP@brutus]: ant/test-ant failed
I can´t see anything ... besides the last message build success ... Can anybody else find the error? Jan -Original Message- From: Gump Integration Build [mailto:[EMAIL PROTECTED] Sent: Thursday, May 27, 2004 9:42 PM To: [EMAIL PROTECTED] Subject: [EMAIL PROTECTED]: ant/test-ant failed To whom it may engage... This is an automated request, but not an unsolicited one. For more information please visit http://gump.apache.org/nagged.html, and/or contact folk at [EMAIL PROTECTED] Project test-ant has an issue affecting its community integration. Project State : 'Failed', Reason 'Build Failed' Full details are available at: http://brutus.apache.org:8080/gump/ant/test-ant/index.html That said, some snippets follow: The following annotations were provided: -INFO- Failed with reason build failed -INFO- Enable debug output, due to build failure. The following work was performed: http://brutus.apache.org:8080/gump/ant/test-ant/gump_work/buil d_ant_test-ant.html Work Name: build_ant_test-ant (Type: Build) State: Failed Elapsed: 0 hours, 6 minutes, 5 seconds Command Line: java -Djava.awt.headless=true -Xbootclasspath/p:/usr/local/gump/public/workspace/xml-xerces2 /java/build/xercesImpl.jar:/usr/local/gump/public/ workspace/xml-xerces2/java/build/xml-apis.jar:/usr/local/gump/ public/workspace/xml-commons/java/external/build/xml-apis.jar: /usr/local/gump/public/workspace/xml- xalan/java/build/xalan-unbundled.jar org.apache.tools.ant.Main -Dgump.merge=/usr/local/gump/public/gump/work/merge.xml -Dbuild.sysclasspath=only -Dtest.haltonfailure=false -Dant.home=/usr/local/gump/public/workspace/ant/dist run-tests [Working Directory: /usr/local/gump/public/workspace/ant] CLASSPATH : /usr/local/j2sdk1.4.2_04/lib/tools.jar:/usr/local/gump/public/ workspace/ant/build/testcases:/usr/local/gump/public/workspace /ant/src/testcases:/usr/local/gump/public/workspace/ant/src/et c/testcases:/usr/local/gump/public/workspace/ant/build/lib/ant -stylebook.jar:/usr/local/gump/public/workspace/ant/build/lib/ ant-swing.jar:/usr/local/gump/public/workspace/ant/build/lib/a nt-apache-resolver.jar:/usr/local/gump/public/workspace/ant/bu ild/lib/ant-nodeps.jar:/usr/local/gump/public/workspace/ant/bu ild/lib/ant-apache-regexp.jar:/usr/local/gump/public/workspace /ant/build/lib/ant-commons-net.jar:/usr/local/gump/public/work space/ant/build/lib/ant-apache-log4j.jar:/usr/local/gump/publi c/workspace/ant/build/lib/ant-commons-logging.jar:/usr/local/g ump/public/workspace/ant/build/lib/ant-jai.jar:/usr/local/gump /public/workspace/ant/build/lib/ant-jmf.jar:/usr/local/gump/pu blic/workspace/ant/build/lib/ant-launcher.jar:/usr/local/gump/ public/workspace/ant/build/lib/ant-jdepend.jar:/usr/local/gump /public/workspace/ant/build/lib/ant-apache-bsf.jar:/usr/local/ gump/public/workspace/ant/build/lib/ant-xalan2.jar:/usr/local/ gump/public/workspace/ant/build/lib/ant-apache-oro.jar:/usr/lo cal/gump/public/workspace/ant/build/lib/ant-apache-bcel.jar:/u sr/local/gump/public/workspace/ant/build/lib/ant- trax.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-an tlr.jar:/usr/local/gump/public/workspace/ant/build /lib/ant-junit.jar:/usr/local/gump/public/workspace/ant/build/ lib/ant-jsch.jar:/usr/local/gump/public/workspace/ant/build/li b/ant-javamail.jar:/usr/local/gump/public/workspace/ant/build/ lib/ant.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant- stylebook.jar:/usr/local/gump/public/workspace/ant/dist/lib/an t-jmf.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-sw ing.jar:/usr/local/gump/public/workspace/ant/dist/ lib/ant-trax.jar:/usr/local/gump/public/workspace/ant/dist/lib /ant-junit.jar:/usr/local/gump/public/workspace/ant/dist/lib/a nt-launcher.jar:/usr/local/gump/public/workspace/ant/dist/lib/ ant-xalan2.jar:/us! r/local/ gump/public/workspace/ant/dist/lib/ant-nodeps.jar:/usr/local/g ump/public/workspace/ant/dist/lib/ant.jar:/usr/local/gump/publ ic/workspace/ant/bootstrap/lib/ant-launcher.jar:/usr/local/gum p/public/workspace/ant/bootstrap/lib/ant.jar:/usr/local/gump/p ublic/workspace/xml-stylebook/bin/stylebook-1.0-b3_xalan-2.jar :/usr/local/gump/public/workspace/dist/junit/junit .jar:/usr/local/gump/packages/antlr-2.7.3/antlr.jar:/usr/local /gump/public/workspace/jakarta-commons/logging/dist/commons-lo gging.jar:/usr/local/gump/public/workspace/jakarta -commons/logging/dist/commons-logging-api.jar:/usr/local/gump/ public/workspace/jakarta-commons/net/dist/commons-net-20040527 .jar:/usr/local/gump/packages/jaf- 1.0.1/activation.jar:/usr/local/gump/public/workspace/jakarta- bcel/bin/bcel.jar:/usr/local/gump/public/workspace/jakarta-bsf /build/lib/bsf.jar:/usr/local/gump/public/workspac e/logging-log4j/log4j-20040527.jar:/usr/local/gump/public/work space/logging-log4j/log4j-chainsaw-20040527.jar:/usr/local/gum p/public/workspace/logging-log4j/log4j-lf5-20040527.jar:/usr/l
DO NOT REPLY [Bug 29264] - Ant task to iterate over a list of files
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=29264. 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=29264 Ant task to iterate over a list of files [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |RESOLVED Resolution||WONTFIX --- Additional Comments From [EMAIL PROTECTED] 2004-05-28 05:05 --- Use Ants apply for a nested exec or AntContribs for/foreach you´ll find at http://ant-contrib.sourceforge.net/. It´s multiple time rejected to enhance the buildfile syntax to a complete script language. And loops are one of the constructs which tend to be used in a non descriptive manner ... - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [GUMP@brutus]: ant/test-ant failed
On Fri, 28 May 2004, Jan Materne [EMAIL PROTECTED] wrote: I can´t see anything ... Neither can I. Stefan - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: namespaced targets via import
On Thu, 27 May 2004, Erik Hatcher [EMAIL PROTECTED] wrote: It would be nice if all imported build file targets were namespaced regardless of whether the targets were overridden or not. +1 - as an alternative to using the unadorned target name. Stefan - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: Ant 1.6.2 release?
On Thu, 27 May 2004, Stefan Bodewig [EMAIL PROTECTED] wrote: Dear committers, do you think we can get a list of must-get-fixed reports by some fixed date - say end of next week? Looks as if some of us can 8-) I volunteer to compile the list and kick of next steps after that. Stefan - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: namespaced targets via import
Stefan Bodewig wrote: On Thu, 27 May 2004, Erik Hatcher [EMAIL PROTECTED] wrote: It would be nice if all imported build file targets were namespaced regardless of whether the targets were overridden or not. +1 - as an alternative to using the unadorned target name. Yes, it would be nice to have this too. Also any idea about how difficult it would be to have all properties defined in imported builds also be namespaced, to avoid collisions? -- Nicola Ken Barozzi [EMAIL PROTECTED] - verba volant, scripta manent - (discussions get forgotten, just code remains) - - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Definer.java
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) typedef: do not throw an NPE if attempting to load a file that does not exist Revision ChangesPath 1.53 +26 -1 ant/src/main/org/apache/tools/ant/taskdefs/Definer.java Index: Definer.java === RCS file: /home/cvs/ant/src/main/org/apache/tools/ant/taskdefs/Definer.java,v retrieving revision 1.52 retrieving revision 1.53 diff -u -r1.52 -r1.53 --- Definer.java 15 Mar 2004 18:02:35 - 1.52 +++ Definer.java 28 May 2004 08:20:22 - 1.53 @@ -21,7 +21,9 @@ import java.io.IOException; import java.io.InputStream; import java.net.URL; +import java.util.ArrayList; import java.util.Enumeration; +import java.util.List; import java.util.Locale; import java.util.NoSuchElementException; import java.util.Properties; @@ -41,6 +43,15 @@ * @since Ant 1.4 */ public abstract class Definer extends DefBase { +private static class ResourceStack extends ThreadLocal { +public Object initialValue() { +return new ArrayList(); +} +List getStack() { +return (List) get(); +} +} +private static ResourceStack resourceStack = new ResourceStack(); private String name; private String classname; private File file; @@ -179,6 +190,9 @@ Enumeration/*URL*/ urls = null; if (file != null) { final URL url = fileToURL(); +if (url == null) { +return; +} urls = new Enumeration() { private boolean more = true; public boolean hasMoreElements() { @@ -209,7 +223,18 @@ loadProperties(al, url); break; } else { -loadAntlib(al, url); +if (resourceStack.getStack().contains(url)) { +log(Warning: Attempting to recursively load + url ++ at + getLocation(), +Project.MSG_WARN); +} else { +try { +resourceStack.getStack().add(url); +loadAntlib(al, url); +} finally { +resourceStack.getStack().remove(url); +} +} } } } - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Definer.java
peterreilly2004/05/28 01:21:04 Modified:src/main/org/apache/tools/ant/taskdefs Tag: ANT_16_BRANCH Definer.java Log: sync Revision ChangesPath No revision No revision 1.44.2.8 +26 -1 ant/src/main/org/apache/tools/ant/taskdefs/Definer.java Index: Definer.java === RCS file: /home/cvs/ant/src/main/org/apache/tools/ant/taskdefs/Definer.java,v retrieving revision 1.44.2.7 retrieving revision 1.44.2.8 diff -u -r1.44.2.7 -r1.44.2.8 --- Definer.java 15 Mar 2004 18:03:56 - 1.44.2.7 +++ Definer.java 28 May 2004 08:21:04 - 1.44.2.8 @@ -21,7 +21,9 @@ import java.io.IOException; import java.io.InputStream; import java.net.URL; +import java.util.ArrayList; import java.util.Enumeration; +import java.util.List; import java.util.Locale; import java.util.NoSuchElementException; import java.util.Properties; @@ -41,6 +43,15 @@ * @since Ant 1.4 */ public abstract class Definer extends DefBase { +private static class ResourceStack extends ThreadLocal { +public Object initialValue() { +return new ArrayList(); +} +List getStack() { +return (List) get(); +} +} +private static ResourceStack resourceStack = new ResourceStack(); private String name; private String classname; private File file; @@ -179,6 +190,9 @@ Enumeration/*URL*/ urls = null; if (file != null) { final URL url = fileToURL(); +if (url == null) { +return; +} urls = new Enumeration() { private boolean more = true; public boolean hasMoreElements() { @@ -209,7 +223,18 @@ loadProperties(al, url); break; } else { -loadAntlib(al, url); +if (resourceStack.getStack().contains(url)) { +log(Warning: Attempting to recursively load + url ++ at + getLocation(), +Project.MSG_WARN); +} else { +try { +resourceStack.getStack().add(url); +loadAntlib(al, url); +} finally { +resourceStack.getStack().remove(url); +} +} } } } - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 15886] - get task
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=15886. 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=15886 get task --- Additional Comments From [EMAIL PROTECTED] 2004-05-28 13:03 --- I'm using get to update a wiki-page with the Versions of my just build program. This looks like get src=http://server/wiki/patch_wiki.pl? id=VersionListbefore=1marker=1text=. / The Problem: text is || 2004/05/28 14:57 || [http://.. and so on for 3 long lines It's really annoying to do URLEncoding by hand! Something like get src=http://server/wiki/patch_wiki.pl; querysitring=somthing thats get URLEncoded and / or some param name=gets URLEncoded value=gets URLEncoded / /get would be fine! BTW: Thanks for ANT Peter - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Bad syntax or unknown instruction
* DO NOT REPLY [Bug 15886] - * ERROR: Bad syntax or unknown instruction. SOLUTION: This is the Minimalist Mailing List Manager. Commands may be either in subject of message (one command per message) or in body (one or more commands, one per line). Batched processing starts when subject either empty or contains command 'body' (without quotes) and stops when either arrives command 'stop' or 'exit' (without quotes) or gets 10 incorrect commands. Supported commands are: subscribe list [email] : Subscribe user to list. If list contains suffix '-writers', user will be able to write to this list, but will not receive messages from it. unsubscribe list [email] : Unsubscribe user from list. Can be used with suffix '-writers' (see above description for subscribe) auth code : Confirm command, used in response to subscription requests in some cases. This command isn't standalone, it must be used only in response to a request by Minimalist. mode list email mode : Set mode for specified user on specified list. Allowed only for administrator. Mode can be (without quotes): * 'reader' - read-only access to the list for the user; * 'writer' - user can post messages to the list regardless of list's status * 'usual' - clear any two above mentioned modes * 'suspend' - suspend user subscription * 'resume' - resume previously suspended permission * 'maxsize size' - set maximum size (in bytes) of messages, which user wants to receive * 'reset' - clear all modes for specified user suspend list : Stop receiving of messages from specified mailing list resume list : Restore receiving of messages from specified mailing list maxsize list size : Set maximum size (in bytes) of messages, which user wants to receive which [email] : Return list of lists to which user is subscribed info [list] : Request information about all existing lists or about list who list : Return the list of users subscribed to list help : This message Note, that commands with email, 'who' and 'mode' can only be used by administrators (users identified in the 'mailfrom' authentication scheme or who used a correct password - either global or local). Otherwise command will be ignored. Password must be supplied in any header of message as fragment of the header in the following format: {pwd: list_password} For example: To: MML Discussion {pwd: password1235} [EMAIL PROTECTED] This fragment, of course, will be removed from the header before sending message to subscribers. -- Sincerely, the Minimalist - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
RE: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java
-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) [...] + log(Warning: Attempting to recursively load + url + + at + getLocation(), Project.MSG_WARN); Perhaps it's because I'm not a native English speaker, but the 'Attempting ...' message (like the 'Overriding reference ...' one) always makes me wonder whether the load (or override) actually take place. Could the message somehow make it more explicit it's ignored? Like: Warning: Recursively load of + url + at + getLocation() + IGNORED; Also, to troubleshoot these problems, knowing where the first import was done would help I would guess. Just suggestion guys, nothing more. I probably just need to learn English or something ;-) --DD PS: Is it a recursive import or a circular import? PPS: Shouldn't this message go to verbose mode only (MSG_VERBOSE)? Circular includes of C/C++ headers is silently ignored. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Bad syntax or unknown instruction
* cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java * ERROR: Bad syntax or unknown instruction. SOLUTION: This is the Minimalist Mailing List Manager. Commands may be either in subject of message (one command per message) or in body (one or more commands, one per line). Batched processing starts when subject either empty or contains command 'body' (without quotes) and stops when either arrives command 'stop' or 'exit' (without quotes) or gets 10 incorrect commands. Supported commands are: subscribe list [email] : Subscribe user to list. If list contains suffix '-writers', user will be able to write to this list, but will not receive messages from it. unsubscribe list [email] : Unsubscribe user from list. Can be used with suffix '-writers' (see above description for subscribe) auth code : Confirm command, used in response to subscription requests in some cases. This command isn't standalone, it must be used only in response to a request by Minimalist. mode list email mode : Set mode for specified user on specified list. Allowed only for administrator. Mode can be (without quotes): * 'reader' - read-only access to the list for the user; * 'writer' - user can post messages to the list regardless of list's status * 'usual' - clear any two above mentioned modes * 'suspend' - suspend user subscription * 'resume' - resume previously suspended permission * 'maxsize size' - set maximum size (in bytes) of messages, which user wants to receive * 'reset' - clear all modes for specified user suspend list : Stop receiving of messages from specified mailing list resume list : Restore receiving of messages from specified mailing list maxsize list size : Set maximum size (in bytes) of messages, which user wants to receive which [email] : Return list of lists to which user is subscribed info [list] : Request information about all existing lists or about list who list : Return the list of users subscribed to list help : This message Note, that commands with email, 'who' and 'mode' can only be used by administrators (users identified in the 'mailfrom' authentication scheme or who used a correct password - either global or local). Otherwise command will be ignored. Password must be supplied in any header of message as fragment of the header in the following format: {pwd: list_password} For example: To: MML Discussion {pwd: password1235} [EMAIL PROTECTED] This fragment, of course, will be removed from the header before sending message to subscribers. -- Sincerely, the Minimalist - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Bad syntax or unknown instruction
* cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java * ERROR: Bad syntax or unknown instruction. SOLUTION: This is the Minimalist Mailing List Manager. Commands may be either in subject of message (one command per message) or in body (one or more commands, one per line). Batched processing starts when subject either empty or contains command 'body' (without quotes) and stops when either arrives command 'stop' or 'exit' (without quotes) or gets 10 incorrect commands. Supported commands are: subscribe list [email] : Subscribe user to list. If list contains suffix '-writers', user will be able to write to this list, but will not receive messages from it. unsubscribe list [email] : Unsubscribe user from list. Can be used with suffix '-writers' (see above description for subscribe) auth code : Confirm command, used in response to subscription requests in some cases. This command isn't standalone, it must be used only in response to a request by Minimalist. mode list email mode : Set mode for specified user on specified list. Allowed only for administrator. Mode can be (without quotes): * 'reader' - read-only access to the list for the user; * 'writer' - user can post messages to the list regardless of list's status * 'usual' - clear any two above mentioned modes * 'suspend' - suspend user subscription * 'resume' - resume previously suspended permission * 'maxsize size' - set maximum size (in bytes) of messages, which user wants to receive * 'reset' - clear all modes for specified user suspend list : Stop receiving of messages from specified mailing list resume list : Restore receiving of messages from specified mailing list maxsize list size : Set maximum size (in bytes) of messages, which user wants to receive which [email] : Return list of lists to which user is subscribed info [list] : Request information about all existing lists or about list who list : Return the list of users subscribed to list help : This message Note, that commands with email, 'who' and 'mode' can only be used by administrators (users identified in the 'mailfrom' authentication scheme or who used a correct password - either global or local). Otherwise command will be ignored. Password must be supplied in any header of message as fragment of the header in the following format: {pwd: list_password} For example: To: MML Discussion {pwd: password1235} [EMAIL PROTECTED] This fragment, of course, will be removed from the header before sending message to subscribers. -- Sincerely, the Minimalist - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java
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) [...] + log(Warning: Attempting to recursively load + url + + at + getLocation(), Project.MSG_WARN); Perhaps it's because I'm not a native English speaker, but the 'Attempting ...' message (like the 'Overriding reference ...' one) always makes me wonder whether the load (or override) actually take place. Could the message somehow make it more explicit it's ignored? Like: Warning: Recursively load of + url + at + getLocation() + IGNORED; Ok, log(Warning: Recursive loading of + url + ignored + at + getLocation(), Also, to troubleshoot these problems, knowing where the first import was done would help I would guess. True, but not done at the moment. Just suggestion guys, nothing more. I probably just need to learn English or something ;-) --DD PS: Is it a recursive import or a circular import? I suppose it could be seen as circular loading of an antlib resource. PPS: Shouldn't this message go to verbose mode only (MSG_VERBOSE)? Circular includes of C/C++ headers is silently ignored. This could be the case for import, but this code is for typedef file=x.xml/ and typedef resource=x/y/z.xml/, which is possibly more likely to be an error to have circular loading. nitpickingcircular includes of c/c++ headers are not silently ignored - one gets for gcc the following #include nested too deeply, one avoids this by doing the #ifdef trick/nitpicking Cheers, Peter - 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]
RE: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java
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 good. PPS: Shouldn't this message go to verbose mode only (MSG_VERBOSE)? This could be the case for import, but this code is for typedef file=x.xml/ and typedef resource=x/y/z.xml/, which is possibly more likely to be an error to have circular loading. Ah, it make sense now. So it's to deal with AntLibs depending on other AntLibs then. I didn't get that the first time. Then it does indeed sound like the message should always be visible. I'm actually thinking this should be an error (raising an exception), but maybe that's too harsh. We do barf on targets with circular dependencies, so why wouldn't we barf for circular AntLib dependencies? Thanks for the explanations Peter. --DD - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Definer.java
peterreilly2004/05/28 08:23:26 Modified:src/main/org/apache/tools/ant/taskdefs Definer.java Log: change message of recursive loading of antlibs to include the original location from where it was loaded. Revision ChangesPath 1.54 +12 -9 ant/src/main/org/apache/tools/ant/taskdefs/Definer.java Index: Definer.java === RCS file: /home/cvs/ant/src/main/org/apache/tools/ant/taskdefs/Definer.java,v retrieving revision 1.53 retrieving revision 1.54 diff -u -r1.53 -r1.54 --- Definer.java 28 May 2004 08:20:22 - 1.53 +++ Definer.java 28 May 2004 15:23:26 - 1.54 @@ -21,9 +21,9 @@ import java.io.IOException; import java.io.InputStream; import java.net.URL; -import java.util.ArrayList; +import java.util.Map; +import java.util.HashMap; import java.util.Enumeration; -import java.util.List; import java.util.Locale; import java.util.NoSuchElementException; import java.util.Properties; @@ -45,10 +45,10 @@ public abstract class Definer extends DefBase { private static class ResourceStack extends ThreadLocal { public Object initialValue() { -return new ArrayList(); +return new HashMap(); } -List getStack() { -return (List) get(); +Map getStack() { +return (Map) get(); } } private static ResourceStack resourceStack = new ResourceStack(); @@ -223,13 +223,16 @@ loadProperties(al, url); break; } else { -if (resourceStack.getStack().contains(url)) { -log(Warning: Attempting to recursively load + url -+ at + getLocation(), +if (resourceStack.getStack().get(url) != null) { +log(Warning: Recursive loading of + url ++ ignored ++ at + getLocation() ++ originally loaded at ++ resourceStack.getStack().get(url), Project.MSG_WARN); } else { try { -resourceStack.getStack().add(url); +resourceStack.getStack().put(url, getLocation()); loadAntlib(al, url); } finally { resourceStack.getStack().remove(url); - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Definer.java
peterreilly2004/05/28 08:24:32 Modified:src/main/org/apache/tools/ant/taskdefs Tag: ANT_16_BRANCH Definer.java Log: sync Revision ChangesPath No revision No revision 1.44.2.9 +12 -9 ant/src/main/org/apache/tools/ant/taskdefs/Definer.java Index: Definer.java === RCS file: /home/cvs/ant/src/main/org/apache/tools/ant/taskdefs/Definer.java,v retrieving revision 1.44.2.8 retrieving revision 1.44.2.9 diff -u -r1.44.2.8 -r1.44.2.9 --- Definer.java 28 May 2004 08:21:04 - 1.44.2.8 +++ Definer.java 28 May 2004 15:24:32 - 1.44.2.9 @@ -21,9 +21,9 @@ import java.io.IOException; import java.io.InputStream; import java.net.URL; -import java.util.ArrayList; +import java.util.Map; +import java.util.HashMap; import java.util.Enumeration; -import java.util.List; import java.util.Locale; import java.util.NoSuchElementException; import java.util.Properties; @@ -45,10 +45,10 @@ public abstract class Definer extends DefBase { private static class ResourceStack extends ThreadLocal { public Object initialValue() { -return new ArrayList(); +return new HashMap(); } -List getStack() { -return (List) get(); +Map getStack() { +return (Map) get(); } } private static ResourceStack resourceStack = new ResourceStack(); @@ -223,13 +223,16 @@ loadProperties(al, url); break; } else { -if (resourceStack.getStack().contains(url)) { -log(Warning: Attempting to recursively load + url -+ at + getLocation(), +if (resourceStack.getStack().get(url) != null) { +log(Warning: Recursive loading of + url ++ ignored ++ at + getLocation() ++ originally loaded at ++ resourceStack.getStack().get(url), Project.MSG_WARN); } else { try { -resourceStack.getStack().add(url); +resourceStack.getStack().put(url, getLocation()); loadAntlib(al, url); } finally { resourceStack.getStack().remove(url); - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
RE: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java
From: Dominique Devienne [mailto:[EMAIL PROTECTED] From: Peter Reilly [mailto:[EMAIL PROTECTED] This could be the case for import, but this code is for typedef file=x.xml/ and typedef resource=x/y/z.xml/, which is possibly more likely to be an error to have circular loading. Ah, it make sense now. So it's to deal with AntLibs depending on other AntLibs then. I didn't get that the first time. Then it does indeed sound like the message should always be visible. I'm actually thinking this should be an error (raising an exception), but maybe that's too harsh. 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 binary libraries). So, it could be the case that you reach such a situation just because two 3rd party antlibs have common dependencies or something. You need to get the right behavior without needing to alter the libs. Jose Alberto - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
RE: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java
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 binary libraries). So, it could be the case that you reach such a situation just because two 3rd party antlibs have common dependencies or something. You need to get the right behavior without needing to alter the libs. That's just the point! Two AntLibs having common dependencies is not cicurlar, so won't be an error. Any scenario where AntLibs are mutually dependent, directly or not, is an error, so the AntLibs *should* be altered, i.e. fixed. And for BC, it's better to lock down the behavior initially, and possibly relax it in the future based on real world feedback, rather than being lax initially, and breaking BC is we finally discovered we should not have been that lax... I vote for raising on exception (assuming I understood the issue correctly). Peter can correct me. --DD - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 29280] New: - javac is requireing srcdir even with nested src elements
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=29280. 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=29280 javac is requireing srcdir even with nested src elements Summary: javac is requireing srcdir even with nested src elements Product: Ant Version: 1.6.1 Platform: Other OS/Version: Other Status: NEW Severity: Normal Priority: Other Component: Core tasks AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] I've tried to use nested src elements with javac - I have to compile sources from multiple directories (don't ask). However, javac is requireing me to include a srcdir attribute. The documentation states that you supply one or the other. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 29281] New: - documentation on javac for src children
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=29281. 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=29281 documentation on javac for src children Summary: documentation on javac for src children Product: Ant Version: 1.6.1 Platform: Other OS/Version: Other Status: NEW Severity: Normal Priority: Other Component: Documentation AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] I've got javac to work with a nested src element now. After much trile-and-error, I got it to behave by listing each source directory using a pathelement child. Could you document this clearly in the javac docs? So documenting that javac srcdir=foo is equivalent to javac ... srcpathelement location=foo//src /javac would have helped me out. Here is my working javac command - feel free to pilpher what you need for the documentation. javac srcdir=${name} !-- I had to add this - is there a bug? -- destdir=${build.classes.dir} depend=${javac.depend} deprecation=${javac.deprecation} source=${javac.source} debug=${javac.debug} src pathelement location=${src.dir}/ pathelement location=${build.apt.src}/ pathelement location=${build.jse.src}/ /src classpath path refid=classpath/ /classpath /javac - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 29280] - javac is requireing srcdir even with nested src elements
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=29280. 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=29280 javac is requireing srcdir even with nested src elements [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |RESOLVED Resolution||INVALID --- Additional Comments From [EMAIL PROTECTED] 2004-05-28 17:00 --- The following works fine for me: javac destdir=classes src path=src/ /javac - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 29281] - documentation on javac for src children
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=29281. 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=29281 documentation on javac for src children [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |RESOLVED Resolution||INVALID --- Additional Comments From [EMAIL PROTECTED] 2004-05-28 17:04 --- There is already an example of using multiple source paths in the manual: is the same as the previous example, with the addition of a second source path, defined by the property src2. This can also be represented using nested src elements as follows: javac destdir=${build} classpath=xyz.jar debug=on src path=${src}/ src path=${src2}/ include name=mypackage/p1/**/ include name=mypackage/p2/**/ exclude name=mypackage/p1/testpackage/**/ /javac - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
RE: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs Define r.java
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 instead of a hierarchy. I cannot give you an example right now, but I would not be surprise if one 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/src/main/org/apache/tools/ant/taskdefs Define r.java 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 binary libraries). So, it could be the case that you reach such a situation just because two 3rd party antlibs have common dependencies or something. You need to get the right behavior without needing to alter the libs. That's just the point! Two AntLibs having common dependencies is not cicurlar, so won't be an error. Any scenario where AntLibs are mutually dependent, directly or not, is an error, so the AntLibs *should* be altered, i.e. fixed. And for BC, it's better to lock down the behavior initially, and possibly relax it in the future based on real world feedback, rather than being lax initially, and breaking BC is we finally discovered we should not have been that lax... I vote for raising on exception (assuming I understood the issue correctly). Peter can correct me. --DD - 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]
[Ant Wiki] Updated: WikiSandBox
Date: 2004-05-28T14:24:29 Editor: 212.242.136.13 Wiki: Ant Wiki Page: WikiSandBox URL: http://wiki.apache.org/ant/WikiSandBox no comment Change Log: -- @@ -22,6 +22,7 @@ [EMAIL PROTECTED] +[http://lavoix.dev-6.com/ .] === Image Link === http://c2.com/sig/wiki.gif - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]