DO NOT REPLY [Bug 15729] - StarTeam rootLocalFolder should be java.io.File
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=15729. 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=15729 [EMAIL PROTECTED] changed: What|Removed |Added Version|1.7.0RC1|1.7.0 -- 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 15729] - StarTeam rootLocalFolder should be java.io.File
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=15729. 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=15729 [EMAIL PROTECTED] changed: What|Removed |Added Version|1.7.0Beta3 |1.7.0RC1 -- 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 15729] - StarTeam rootLocalFolder should be java.io.File
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=15729. 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=15729 [EMAIL PROTECTED] changed: What|Removed |Added Version|1.7.0Beta2 |1.7.0Beta3 -- 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 15729] - StarTeam rootLocalFolder should be java.io.File
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=15729. 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=15729 [EMAIL PROTECTED] changed: What|Removed |Added Version|1.7.0Beta1 |1.7.0Beta2 -- 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 15729] - StarTeam rootLocalFolder should be java.io.File
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=15729. 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=15729 [EMAIL PROTECTED] changed: What|Removed |Added Version|1.7Alpha (nightly) |1.7.0Beta1 -- 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 15729] - StarTeam rootLocalFolder should be java.io.File
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=15729. 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=15729 [EMAIL PROTECTED] changed: What|Removed |Added Component|Optional Tasks |Optional SCM tasks -- 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 15729] - StarTeam rootLocalFolder should be java.io.File
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=15729. 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=15729 StarTeam rootLocalFolder should be java.io.File --- Additional Comments From [EMAIL PROTECTED] 2003-09-02 15:57 --- Steve, I'll trust ypur judgement here. If you say a relative path has never made too much sense anyway (and there is no way to detect what the user wanted at runtime), go ahead and change it. Printing a warning is good. We should also add a not to Ant's WHATSNEW file in the this can break your build section. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 15729] - StarTeam rootLocalFolder should be java.io.File
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=15729. 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=15729 StarTeam rootLocalFolder should be java.io.File --- Additional Comments From [EMAIL PROTECTED] 2003-09-01 18:46 --- Now, that at last, I have some free time, I am taking a fresh look at this bug. If I may summarize the discussion to this point: Lee Marlow (supported by Jose Alberto Fernandez) wants the rootlocalfolder attribute to be set relative to basedir when not specified as an absolute path. They point out that this gives more conceptual unity with the rest of ant. Stefan Bodewig worries that this will break backward compatibility with users who expect it to be relative to current working directory. I, Steve Cohen, point out that while backward compatibility will break, Stefan incorrectly understands part of what current functionality is. While it is true that if rootlocalfolder is specified as a relative path, that will currently be interpreted relative to CWD, there is a further twist that if NO rootlocalfolder is specified, rlf will be the default local directory stored in starteam. This is primarily useful in the StarTeam GUI itself, and losing this functionality in ant would be no great loss, IMHO. (It could be remedied by adding yet another attribute to the task, possibly). But Stefan is correct, there would still be a significant backward compatibility issue when relative paths are specified. I wasn't thinking about THAT issue at the time of my earlier comments. However, I am inclined to press forward despite this. Current functionality is a mess: 1) if an absolute path is specified, use that. 2) if a relative path is specified, it is relative to CWD 3) if no rootlocalfolder is specified, the starteam default is used. The problem is that 2 is undocumented (no documentation is provided on what happens when a relative rootlocalfolder is specified). Additionally, its lack of documentation makes it inconsistent with 3. User is arguably entitled to assume that not specifying the attribute would be the same as specifying a relative .. In my use cases, it would make no difference. I always specify rootlocalfolder as an absolute path (perhaps unconsciously aware of the relative path ambiguity here). So what I propose to do is that whenever a relative path is specified, print a WARNING message indicating that the meaning of this attribute has changed from what it meant in 1.5. Something along the lines of WARNING - a relative rootlocalfolder is now interpreted relative to basedir and not to the current working directory as it was in Ant versions prior to 1.6. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
DO NOT REPLY [Bug 15729] - StarTeam rootLocalFolder should be java.io.File
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=15729. 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=15729 StarTeam rootLocalFolder should be java.io.File [EMAIL PROTECTED] changed: What|Removed |Added Status|REOPENED|ASSIGNED
DO NOT REPLY [Bug 15729] - StarTeam rootLocalFolder should be java.io.File
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=15729. 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=15729 StarTeam rootLocalFolder should be java.io.File --- Additional Comments From [EMAIL PROTECTED] 2003-04-04 06:24 --- Jose Alberto, not everything is resolved to basedir, unfortunately. Sometimes the code predates IntrospectionHelper, sometimes people just didn't know about the magic of File attributes. Whenever we change an attribute to resolve files relative to basedir and not the current working directory, we run the danger of breaking backwards compatibility. As an example, several Jakarta builds got broken when style's stylesheet attribute was changed. I had to revert to the tactics we use now - which pretty much is option (2) from my last comment. Consistency is good. Breaking backwards compatibility for consistency's sake is not.
DO NOT REPLY [Bug 15729] - StarTeam rootLocalFolder should be java.io.File
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=15729. 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=15729 StarTeam rootLocalFolder should be java.io.File --- Additional Comments From [EMAIL PROTECTED] 2003-04-04 17:49 --- Well by that no bug should be fixed because it is a backward compatibility issue ;-P No seriously, I think the StarTeam people needs to decide whether allowing whether to do ant to a file in a different directory should break a build (because CWD is different) or it should work because files are relative to basedir. I think the later is much better. Maybe some backward comp option should be OK or warning message. We should audit the code to see how many cases of this issues we stil have.
DO NOT REPLY [Bug 15729] - StarTeam rootLocalFolder should be java.io.File
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=15729.
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=15729
StarTeam rootLocalFolder should be java.io.File
--- Additional Comments From [EMAIL PROTECTED] 2003-04-03 14:26 ---
My two cents:
Anything in ANT that is suppose to point to a file or directory is by
convention assumed to be relative to ${basedir} (unless you specify a full
path). By having the parameter be of type File, ANT takes care of this
transparently.
Using CWD for relative files means that buildfiles cannot be used correctly
when called ant from one another.
It seem not only worth, but a bug not to work that way.
DO NOT REPLY [Bug 15729] - StarTeam rootLocalFolder should be java.io.File
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=15729. 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=15729 StarTeam rootLocalFolder should be java.io.File --- Additional Comments From [EMAIL PROTECTED] 2003-03-27 07:50 --- While this fixes the behavior, it also introduces a backwards incompatibility. There may be people relying on the interpretation relative to CWD. I see a couple of options: (1) Add a new attribute (2) Try to see whether rootLocalFolder as defined by the attribute exists. If it doesn't, use getproject().resolveFile() to see whether this one exists. If so, log a warning and use it. I don't understand Starteams concepts well enough to know whether option (2) is feasible at all.
DO NOT REPLY [Bug 15729] - StarTeam rootLocalFolder should be java.io.File
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=15729. 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=15729 StarTeam rootLocalFolder should be java.io.File [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |RESOLVED Resolution||WONTFIX --- Additional Comments From [EMAIL PROTECTED] 2003-03-27 14:35 --- I don't think this patch solves anything. And may break uses that were depending on the relative path. Nothing stops the user from specifying an absolute reference as the root local folder if that is what he wants. Possibly storing of this property as a File is a good idea but I cannot endorse this patch without a lot of investigation and further testing.
