| Summary: | Reduce API tools footprint in releng scenarios | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] PDE | Reporter: | Jeff McAffer <jeffmcaffer> | ||||
| Component: | API Tools | Assignee: | PDE API Tools Inbox <pde-apitools-inbox> | ||||
| Status: | CLOSED WONTFIX | QA Contact: | |||||
| Severity: | normal | ||||||
| Priority: | P3 | CC: | pascal | ||||
| Version: | 3.7 | ||||||
| Target Milestone: | --- | ||||||
| Hardware: | PC | ||||||
| OS: | Mac OS X - Carbon (unsup.) | ||||||
| Whiteboard: | stalebug | ||||||
| Attachments: |
|
||||||
|
Description
Jeff McAffer
Created attachment 181572 [details]
patch to decouple API tools
I took a stab at this and it actually turns out to be relatively easy. Well, at least I think it is. The problem is that its hard to test because a) you can't use optional requirements in a feature based launch config (the optional things are added for free) and b) the API tools test suite is setup to use the workspace even for the Ant task tests. So you can't test the use of API tools when core.resources is absent because core.resources is needed by the test infrastructure.
In any event, attached is a patch (against HEAD) that decouples things a bit more. Basically the pde.core etc stuff is now optional. It would be interesting to consider separate bundles for this but that is way beyond my scope of understanding on how the code works.
Summary of key changes
- split Util into two, WorkspaceUtil and Util moving anything that depends on the IDE
- Copy a very modest amount of jdt.core Flags (two tiny methods)
I did not look deeply at the Ant tasks.
Let me know if this is an interesting direction and we can do further work.
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. Please remove the stalebug flag, if this issue is still relevant and can be reproduced on the latest release. |