Bug 481467

Summary: SWT Tools: Split Sleak and Spy into separate plugins
Product: [Eclipse Project] Platform Reporter: Marc-André Laperle <malaperle>
Component: SWTAssignee: Marc-André Laperle <malaperle>
Status: RESOLVED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: akurtakov, Lars.Vogel
Version: 4.6Keywords: noteworthy
Target Milestone: 4.7 M3   
Hardware: All   
OS: All   
See Also: https://git.eclipse.org/r/59626
https://git.eclipse.org/c/platform/eclipse.platform.swt.git/commit/?id=13aff36cf5e487afc79574bb15ceacadeb15daaf
https://git.eclipse.org/r/82129
https://git.eclipse.org/c/www.eclipse.org/eclipse/news.git/commit/?id=e0de1615827bdd4ed91c5537aff6da28c949a50b
Whiteboard:

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