Bug 27627 - API review
Summary: API review
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Ant (show other bugs)
Version: 2.1   Edit
Hardware: PC Windows 2000
: P2 normal (vote)
Target Milestone: 2.1 M5   Edit
Assignee: Darin Wright CLA
QA Contact:
URL:
Whiteboard:
Keywords: core, ui
Depends on:
Blocks:
 
Reported: 2002-12-03 23:11 EST by Darin Wright CLA
Modified: 2003-01-27 16:21 EST (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Wright CLA 2002-12-03 23:11:45 EST 
We need to review the Ant/Ext Tool API.
* we need an Ant UI plug-in
* all extension points need doc
Comment 1 Darin Swanson CLA 2002-12-04 08:34:11 EST 
Note that an Ant UI project already exists in the Eclipse repository.
Comment 2 Darin Wright CLA 2003-01-14 09:43:07 EST 
Plan for 2.1 is to have no API and no separate plug-in. We will make all 
packages internal.
Comment 3 Darin Wright CLA 2003-01-24 09:14:59 EST 
We should do this ASAP, to see if we break any of our users. Dejan - which class
(es) do you use in External Tools (to run ant scripts)?
Comment 4 Darin Swanson CLA 2003-01-24 09:24:26 EST 
Darin, there are lots of users that have talked about using 
org.eclipse.ant.core.AntRunner (I believe this is most of Dejan's uses as 
well). 

Although it has never explicitly been published as API, I think it would be a 
mistake to suddenly change the package structure.  

Can we not just add the caveat in the class javadoc like debug had for 1.0? 
This leaves no solid API but doesn't break anyone who has been building 
plugins on top of ant core.
Comment 5 Dejan Glozic CLA 2003-01-24 09:27:44 EST 
We use AntRunner but not external tools.
Comment 6 Darin Wright CLA 2003-01-24 09:49:35 EST 
We are only changing external tools to internal - not ant core.
Comment 7 Darin Swanson CLA 2003-01-24 17:13:20 EST 
All of the external tools packages have been changed to be internal.
plugin.xml updated
Nls message accessor classes updated.

None of the ant packages have been changed

Please verify (DarinW).
Comment 8 Darin Wright CLA 2003-01-27 16:21:22 EST 
Verified