You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by bu...@apache.org on 2008/09/19 17:10:58 UTC
DO NOT REPLY [Bug 45845] New: Possibly remove reverse() in
Classloader task
https://issues.apache.org/bugzilla/show_bug.cgi?id=45845
Summary: Possibly remove reverse() in Classloader task
Product: Ant
Version: 1.7.1
Platform: PC
OS/Version: Windows XP
Status: NEW
Severity: normal
Priority: P2
Component: Core tasks
AssignedTo: notifications@ant.apache.org
ReportedBy: robert.flaherty@oracle.com
Can we drop one of these, I was trying both not realizing that they were
changing the same property, but in reverse.
/**
* Set reverse attribute.
* @param b if true reverse the normal classloader lookup.
*/
public void setReverse(boolean b) {
this.parentFirst = !b;
}
/**
* Set reverse attribute.
* @param b if true reverse the normal classloader lookup.
*/
public void setParentFirst(boolean b) {
this.parentFirst = b;
}
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
DO NOT REPLY [Bug 45845] Possibly remove reverse() in Classloader
task
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=45845
--- Comment #2 from Robert Flaherty <ro...@oracle.com> 2008-09-25 07:22:52 PST ---
The only reason I thought drop was b/c it was listed as "EXPERIMENTAL" in the
comment block.
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
DO NOT REPLY [Bug 45845] Possibly remove reverse() in Classloader
task
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=45845
Stefan Bodewig <bo...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
--- Comment #5 from Stefan Bodewig <bo...@apache.org> 2008-09-26 02:53:42 PST ---
OK, reverse is deprecated now.
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
DO NOT REPLY [Bug 45845] Possibly remove reverse() in Classloader
task
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=45845
Stefan Bodewig <bo...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|normal |enhancement
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
DO NOT REPLY [Bug 45845] Possibly remove reverse() in Classloader
task
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=45845
--- Comment #1 from Stefan Bodewig <bo...@apache.org> 2008-09-25 04:57:20 PST ---
we can deprecate one but drop neither since they are part of the published
public API and we try to not break backwards compatibility.
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
DO NOT REPLY [Bug 45845] Possibly remove reverse() in Classloader
task
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=45845
--- Comment #4 from Robert Flaherty <ro...@oracle.com> 2008-09-25 08:05:08 PST ---
I know I know, actually I was desperate and searched the code b/c I couldn't
find what I needed in the published manual tasks. And actually, someone else
put in a bug report for it a few days after I started working up a workaround
with the classloader task. I'll submit that in a few days, as it will be quite
lengthy I think (getting FTP to work without ANT_HOME/lib or
${user.home}/.ant/lib, but rather <classloader>, <path>, <classpath>, etc).
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
DO NOT REPLY [Bug 45845] Possibly remove reverse() in Classloader
task
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=45845
--- Comment #3 from Stefan Bodewig <bo...@apache.org> 2008-09-25 07:44:10 PST ---
sure, we thought that would scare people away from the class.
But still some seem to have found and even started to report bugs against it
;-)
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.