Bug 238624 - Remove unused UI dependencies from org.eclipse.jst.ws.consumption
Summary: Remove unused UI dependencies from org.eclipse.jst.ws.consumption
Status: CLOSED FIXED
Alias: None
Product: WTP Webservices
Classification: WebTools
Component: jst.ws (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.0.1   Edit
Assignee: Kathy Chan CLA
QA Contact: Kathy Chan CLA
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2008-06-26 11:39 EDT by Troy Bishop CLA
Modified: 2008-10-27 14:35 EDT (History)
0 users

See Also:

Attachments
Possible patch to remove the direct JFace dependency (898 bytes, patch)
2008-06-26 11:39 EDT, Troy Bishop CLA 		bjorn.freeman-benson: iplog+
Details | Diff
View All 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 Troy Bishop CLA 2008-06-26 11:39:06 EDT 
Build ID: WTP 3.0.0

Steps To Reproduce:
The plug-in org.eclipse.jst.ws.consumption contains a dependency for org.eclipse.jface when the plug-in does not contain any JFace functionality.
This causes a problem as we provide a headless Eclipse + WTP application and as a result of this dependency we automatically pull in the JFace plug-in when it's not needed.

More information:
It would be nice to have this fix for WTP 3.0.1.  If there is JFace functionality within this plug-in (that I didn't see) then it will be pulled in automatically via the optional plug-in dependency org.eclipse.ui.  For our headless tool we ignore optional dependencies so removing the direct jface dependency will be sufficient for us.
Comment 1 Troy Bishop CLA 2008-06-26 11:39:41 EDT 
Created attachment 105924 [details]
Possible patch to remove the direct JFace dependency
Comment 2 Kathy Chan CLA 2008-07-16 18:48:07 EDT 
Patch committed and released to WTP 3.0.1 as v200807162242.
Comment 3 Troy Bishop CLA 2008-07-30 12:37:35 EDT 
Thanks - the fix looks great!
Comment 4 Kathy Chan CLA 2008-10-27 14:35:54 EDT 
Closing.