DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn|29248 | -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug, or are watching the assignee. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |RESOLVED Resolution||FIXED --- Additional Comments From [EMAIL PROTECTED] 2004-11-19 18:17 --- -Dant.executor.class=org.apache.tools.ant.helper.SingleCheckExecutor works and there are ways to alias your own switch as well. Closing. -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug, or are watching the assignee. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421.
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=21421
[Patch] to allow to run several targets without rerunning their dependencies
--- Additional Comments From [EMAIL PROTECTED] 2004-09-03 19:57 ---
I think the logic should be different:
class Main {
String executorClassName;
void processArgs(...) {
...
else if (arg.equals(-keep-going)...) {
this.executorClassName=KeepGoingExecutor.class.getName(); // with optional
check that the field is not set already
}
else if (arg.equals(-single-check)...) {
executorClassName=SingleCheckExecutor.class.getName(); // with optional
check that the field is not set already
}
...
if (definedProps.hasKey(ant.executor.class)) { // system properties too?
executorClassName=definedProps.getProperty(ant.executor.class);
}
}
void runBuild() {
...
project.setExecutor((Executor)Class.forName(executorClassName).newInstance());
...
}
}
class Project {
// remove keepGoingMode
// add
public final Executor executor;
void initSubProject(...) {
...
subProject.setExecutor(this.executor);
}
void executeTargets(list) { // executeTarget,
// maybe check if executor==null
this.executor.executeTargets(list);
}
}
class DefaultExecutor implements Executor {
private boolean keepGoingMode;
public void executeTargets() {
...
}
}
class KeepGoingExecutor extends DefaultExecutor { // for code reuse
KeepGoingExecutor() {
super(true); // keep-going mode
}
}
class ParallelExecutor {
// thread pool, ...
}
class NoDependenciesExecutor {
}
.
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-09-02 13:55 --- Oh. Ant is used by CallTarget (antcall) as well, and I already modified it to use a SingleCheckExecutor. ;) - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-09-01 19:46 --- Thank you, works great. Not sure about closing the bug - I think we still need a command line option that one would be able to easily type. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-09-01 20:08 --- Regarding the ParallelExecutor--I may. This is not trivial however. Assuming the Executor does not get shot out of Ant, anyone could write it. As for the command-line option, my concern is when would they end? Someone else can come along next week and ask for an option for Ant to run everything except for the specified targets ... and so on. If you really needed a switch to do this, it's not impossible to do it yourself... for example, if you added this to your .antrc (assuming you use the ant script): _ant_exec_args= for arg in $@ ; do if [ $arg = --single ] ; then arg=-Dant.executor.class=org.apache.tools.ant.helper.SingleCheckExecutor fi _ant_exec_args=$_ant_exec_args \$arg\ done ant_exec_args=$_ant_exec_args unset _ant_exec_args You have aliased -Dant.executor.class=org.apache.tools.ant.helper.SingleCheckExecutor to --single. PS: guess who wrote I have my own ANT launcher script that preconfigures my build environment, I can just add the option into it. ;) - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-09-01 20:15 --- It was me... But some other people said they wanted this feature too. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-09-01 20:45 --- I think the execution logic should be moved out of Project.executeTarget() into Executor.executeTargets(). In this case It would be easier to create ParallelExecutor and also KeepGoingExecutor becomes meaningful. Also the executor needs to be propagated into sub-ants – we will need to share thread pooling in ParallelExecutor. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-09-01 22:56 --- I did not mean just subant, but ant and antcall too. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-08-31 22:38 --- Alexey, you're so good about testing... do you mind trying out -Dant.executor.class=org.apache.tools.ant.helper.SingleCheckExecutor from CVS HEAD to see if it satisfies the request, and mark this as FIXED if so? Thanks, Matt - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-08-06 09:54 --- I like -singlecheck as you sugested. I will also suggest -s (is that taken?) as its short version (like -projecthelp -p). I will also suggest using the common UNIX practice for boolean switches of using + to negate an option. So +s (or +singlecheck) will mean NO -singlecheck. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-08-06 15:15 --- -s is the alternative form of -find. As for the + concept, what a step! Firstly, won't you agree that if we make this available for one option, it should be available for all switch-style options? Parsing the command line suddenly becomes much more involved a process. At this point I would seriously consider switching to commons CLI and incur the 30K jar. Now I will duck... - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-08-06 20:16 --- Matt, I will join you. I thought about suggesting +singlecheck, but I think that using plus sign for negating is strange. -singlecheck- would be better. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-08-06 20:20 --- To clarify my position-- :) --I am not opposed to using + (although it does seem a little strange for this particular option). I am still not entirely convinced that we need any form of negation for any option switch when not specifying that switch is equivalent. If we do need negation of one switch, I am for adding it for all switches via CLI. But I highly doubt many will join me due to the notorious aversion of the Ant developer community to external dependencies. :) Also check my latest comments on Bug 29248 regarding making target execution mode a property instead of a switch. -Matt - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-08-05 14:01 --- Since commas can be part of target names (for good or ill) and targets so named can be called from the command line--despite the fact that such targets are broken in that they cannot be specified in a depends attribute--the BC issue is real. Under the circumstances Jose Alberto's suggestion of a new Ant option sounds like a reasonable compromise. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-08-05 14:28 --- It is up to you to decide. It is just convenient to use comma – I do not have to type more than usual to have noredo functionality, but maybe there can be (or will be) a way to configure ANT to make the option default. I also have not seen any build scripts that have commas in the target names. One would be able to run those targets in a very limited number of situations (command line, for example). I would say it is reasonable to assume there are no such targets in the wild. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-08-05 14:37 --- Yes, but we're not dealing with reality here. We are unfortunately constrained to dealing with the worst-case scenario. And Jose mentioned environment customization; you should be able to add the option to your ANT_ARGS environment variable to make that behavior the default. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-08-05 14:42 --- Actually it is even easier - I have my own ANT launcher script that preconfigures my build environment, I can just add the option into it. Some of my other changes use this feature, but now they can just directly call new topoSort(). Lets have an additional command line option. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-08-05 15:45 --- Created an attachment (id=12344) proposed patch to Main.java - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-08-05 15:52 --- Alexey--how does that last patch strike you? - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-08-05 17:28 --- Thank you. The patch should work. May I ask you to add a short alias for the option and a way to turn it off (maybe -nosinglecheck)? - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
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=21421. 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=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-08-05 18:31 --- The first request I have no ideological problem with, but what option to use? Most short options are a single letter, and for example, -singlecheck, while longer than -noredo, is no longer than -diagnostics, and indeed is shorter than -propertyfile or -inputhandler. As for the -nosinglecheck idea, I think I can see how you would want to use it, but I can't say that I really think it's necessary to basic Ant. Not specifying the option would be equivalent to specifying a negative version of the same option. In fact, I changed -noredo to the hardly more descriptive - singlecheck due to a reluctance on my part to use such a negative option. I would have preferred a boolean argument to two different forms of the same option, but did not because of the implementation of Ant's command-line options. This is very straightforward; for example to use the -s option without specifying a buildfile name, -s has to be the last option given. This makes me reluctant to solve the issue by adding an optional argument to this new option. In summary, if these issues came to a committer vote, my responses would be: [+1] adding -singlecheck (or equivalent) option to work in conjunction with the new Target sorting options on Project [-0] adding a short form of -singlecheck [-1] complementing -singlecheck with -nosinglecheck br, Matt - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-03-07 17:04 --- I hate the current behavior just as anyone else, but there is a BC issue here and really I see few ways out. For an example just think what the following does today: ant clean test1 clean test2 clean test3 Assuming the target clean just wipes clean the build, then the above list of targets will have very different behavior if the way targets are executed from the command line changes. So, BC is a real issue. This is why the proposal uses , to indicate execution without dependency reexecution: clean,test1,clean,test2,clean,test3 which is equivalent to: clean,test1,test2,test3. Alternatively, we could provide an option -noredo (or something like that) that will treat the list of targets as comma separated. So if one wants that behavior all the time just stick the option in ANT_OPTS in your environment. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2004-03-05 01:21 --- *** Bug 27459 has been marked as a duplicate of this bug. *** - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-03-05 01:23 --- Fixed text: By default ant A B will re-execute all common A's and B's dependencies. It is at least not efficient. This patch allows ant 'A,B' arguments. In this case the patch creates a new pseudo target named A,B with dependencies A,B and executes the target. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-03-05 01:46 --- What is needed to get this patch into HEAD? If it needs more testing, I volunteer. Also, can someone please explain why running the dependency twice is by design, according to Dominique Devienne's comment to bug 23783? I can't imagine how this would be expected behavior. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-03-05 01:48 --- I just want to note that we are running our builds with this patch all the time. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-03-05 10:29 --- How do people feel about this new feature? Shall we put it in the 1.7 target list? If we were to do this, I think we need to do the complete job and allow similar functionality to [sub]ant[call] which means to be able to say 'target=A, B' with the same effect. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-03-05 15:04 --- When I wrote by-design, I meant it is coded up like that. Hence, it can't be changed for BC reasons. I never said it was the good design decision ;-) I often execute more than one target from the CLI, and run afool of this too. For critial target, I guard against re-execution with a property, since properties are still shared (unlike the target dependency analysis). I agree with Jose Alberto. [sub]ant[call] should be made to work the same, except I wouldn't override the meaning of the target attribute, and add a new targets attributes. For BC reasons, again, . You can currently call a target with a comma in its name with [sub]ant[call], so supporting multi-target calling using the 'target' attribute would not be BC. --DD So +1 to the feature of that patch. (haven't look at the code itself). - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-03-05 15:07 --- I think all ant, subant, ... tasks are automatically targeted, since the patch changes how Project class executes targets. If it sees a non existing target with comma, it just adds it (do we need synchronized here?). - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-03-05 19:37 --- When I wrote by-design, I meant it is coded up like that. Well, just because a feature is coded up doesn't mean that it was by design. If the author made the wrong assumptions or missed something, then it's not design; it's a bug. Hence, it can't be changed for BC reasons. Is BC really an issue here? I don't see how anyone would rely on dependencies being run multiple times. Once a dependency has been resolved, any further exececutions of that dependency would be redundant. Remember, too, that this is the command line we're talking about, so no scripts would be affected by changing the semantics of the argument list. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-03-05 19:50 --- ... then it's not design; it's a bug. Maybe. Is BC really an issue here? I think it always is. I'm sure many people have .bat or .sh script, or crontab entries or whatever that call several targets. I don't think it would be nice to break those people. Believe it or not, for purely selfish reasons, I'd vote to change to existing behavior. But if I have to consider BC, I'd vote -0. BTW, my vote doesn't count ;-) --DD - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2004-03-05 20:19 --- I'm sure many people have .bat or .sh script, or crontab entries or whatever that call several targets. I don't think it would be nice to break those people. I understand, but what I'm saying is, how could these batch files and crontabs possibly get broken? Dependencies are idempotent: Running a dependency twice produces results no different from when the dependency was first resolved. Unless you can give me an example where a dependency in Ant is not idempotent, I can't agree that there is a BC issue here. I'd vote to change to existing behavior. You mean, change nothing? - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2003-10-13 16:37 --- Can some body incorporate this into the main ant. it seems odd that the project share the same properties but not the depedency graph. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2003-07-08 22:09 --- Created an attachment (id=7176) Project.java patch - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2003-07-08 22:10 --- Created an attachment (id=7177) test file - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2003-07-08 22:27 --- Created an attachment (id=7178) updated test build.xml file - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 21421] - [Patch] to allow to run several targets without rerunning their dependencies
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE. http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21421 [Patch] to allow to run several targets without rerunning their dependencies --- Additional Comments From [EMAIL PROTECTED] 2003-07-08 22:32 --- Created an attachment (id=7179) updated patch without debug output - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
