Bug 481467 - SWT Tools: Split Sleak and Spy into separate plugins
Summary: SWT Tools: Split Sleak and Spy into separate plugins
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 4.6   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 4.7 M3   Edit
Assignee: Marc-André Laperle CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
Depends on:
Blocks:
 
Reported: 2015-11-04 23:27 EST by Marc-André Laperle CLA
Modified: 2016-10-04 08:13 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marc-André Laperle CLA 2015-11-04 23:27:27 EST
It would be great if it was possible to use the Sleak and Spy tools independently from the rest of the tools plugin. For example, we could add those tools in our target definitions and use them in the runtime Eclipse. Right now, depending on swt.tools means depending on JDT which our project does not use.
So my suggesting is splitting the swt.tools plugin this way:
-org.eclipse.swt.tools.sleak : Contains Sleak view and related code
-org.eclipse.swt.tools.spy : Contains Spy view and related code
-org.eclipse.swt.tools.base : Adds the SWT Tools category. All swt.tools.* depends on this to be able to add their respective views to the category
-org.eclipse.swt.tools : Same plugin as today except the Sleak view and Spy view are removed. JNIGen, Ant build, etc are left untouched.

tools.base, tools.spy, tools.sleak are reexported in swt.tools to ensure maximum backwards compatilbity.
org.eclipse.swt.tools.feature includes all the swt.tools.* plugins.

More things could be split off from swt.tools later as necessary.

I have a patch underway for this. Let me know if you have any other suggestions or concern.
Comment 1 Marc-André Laperle CLA 2015-11-04 23:52:02 EST
https://git.eclipse.org/r/#/c/59626/
Comment 2 Alexander Kurtakov CLA 2015-11-20 04:05:44 EST
The use case described is sound enough to make it worth the effort.
Comment 4 Alexander Kurtakov CLA 2016-09-21 05:30:26 EDT
Pushed to master.
Comment 5 Marc-André Laperle CLA 2016-09-21 11:03:42 EDT
Thanks a lot!
Comment 6 Lars Vogel CLA 2016-09-21 11:22:11 EDT
Cool, stuff.

Marc-Andre, can you upload a Gerrit for the N&N M3? Repo: https://git.eclipse.org/r/#/admin/projects/www.eclipse.org/eclipse/news
Comment 7 Eclipse Genie CLA 2016-09-28 23:41:41 EDT
New Gerrit change created: https://git.eclipse.org/r/82129