Bug 44494 - Remove direct dependancies on the Xerces plugin
Summary: Remove direct dependancies on the Xerces plugin
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Ant (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.0 M5   Edit
Assignee: Darin Wright CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 44935
  Show dependency tree
 
Reported: 2003-10-08 16:05 EDT by Darin Swanson CLA
Modified: 2003-10-24 09:58 EDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Swanson CLA 2003-10-08 16:05:14 EDT
All of the Ant code should be moved to use the JAXP interfaces etc.
Should only impact the Ant UI.
Comment 1 Darin Swanson CLA 2003-10-09 16:26:16 EDT
I have made the adjustments to the Ant UI plugins (not yet released)
The problem is with the test suites and ensuring that the tests do not rely on 
parser specific behavior (which I believe they currently do).

Ant's plan:
Have no implementation specific dependancies on Xerces.
A user running Eclipse on a 1.4 vm, Ant in the same VM as Eclipse ---> Xerces  
VM ships with
A user running Eclipse on a 1.4 vm, Ant in a 1.4 separate VM ---> Xerces VM 
ships with
A user running Eclipse on a 1.4 vm, Ant in a 1.3 separate VM ---> User will 
have to manually add the required Xerces JARs to the Ant runtime classpath
Comment 2 Darin Swanson CLA 2003-10-15 11:57:39 EDT
Released:
org.apache.ant:  removed <requires> statement for Xerces

org.eclipse.ant.core: removed Xerces from the classpath and as a required 
project for the build path.

org.eclipse.ant.ui: reworked all parsing to use the JAXP interfaces (changes 
to Parser, AntEditorCompletionProcessor, AntModel and XMLElement). Removed 
Xerces from the classpath and as a required project for the build path. 
Removed <requires> statement for Xerces.

org.eclipse.ant.tests.ui: reworked all parsing to use the JAXP interfaces 
(changes to CodeCompletionTest, EnclosingTargetSearchingHandlerTest, 
AbstractAntUITest). Removed Xerces from the classpath and as a required 
project for the build path. Removed <requires> statement for Xerces.

Three tests still fail...need to see if bad tests or changes resulting from 
the parser change.
Comment 3 Darin Swanson CLA 2003-10-20 18:17:50 EDT
The Ant plugins no longer directly reference the Xerces plugin.
The Ant Editor code assist is not as good as it was before but I do not believe 
that the average user will notice.  We will log these as separate issues that 
can be dealt with or not as we have the time. See bug 45244.
Comment 4 Darin Swanson CLA 2003-10-20 18:18:21 EDT
Please verify DarinW.
Comment 5 DJ Houghton CLA 2003-10-21 17:14:43 EDT
Darin, just curious how you worked through the Locator/column number problems
that we discussed. (no doubt others (like PDE) will have the same problems)
Comment 6 Darin Swanson CLA 2003-10-21 17:20:14 EDT
We haven't fully (bug 45244) but basically we added support to pass around the 
underlying AntEditor IDocument and then use the searching capabilities 
(FindReplaceDocumentAdapter) and the ability to find line numbers and therefore 
column numbers using offsets.
Comment 7 DJ Houghton CLA 2003-10-22 14:04:29 EDT
We were talking about it around here and one suggestion that McQ had was to wrap
the input stream in your own stream and keep track of column position by
counting #read calls (and watching for CR/LF). Not sure if that is any easier or
not...it would probably be easy to get confused along the way.
Comment 8 Darin Swanson CLA 2003-10-22 14:07:40 EDT
And all the fun of multiple platforms and different line delimiter lenghts...
But it is another idea we could investigate..thanks
Comment 9 Darin Wright CLA 2003-10-24 09:58:39 EDT
Verified.