Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 315886

Summary: Unnecessary dependency on emf transaction.
Product: z_Archived Reporter: Miles Parker <milesparker>
Component: Gef3dAssignee: Jens Von Pilgrim <developer>
Status: RESOLVED FIXED QA Contact: Jens Von Pilgrim <developer>
Severity: normal    
Priority: P3 CC: developer
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Patch for fix.
none
mylyn/context/zip none

Description Miles Parker CLA 2010-06-05 15:13:37 EDT
o.e.gef.ext has an (apprarantly) unneeded dependency on emf.transaction. I've replaced it with o.e.ecore and o.e.ecore.edit in this patch.
Comment 1 Miles Parker CLA 2010-06-05 15:13:58 EDT
Created attachment 171203 [details]
Patch for fix.
Comment 2 Jens Von Pilgrim CLA 2010-06-06 06:02:25 EDT
Fixed by extracting all EMF related classes to a new plugin called org.eclipse.gef3d.ext.emf. The EMF related classes were moved to sub-packages *.emf. The ext plugin should be independent from EMF, since important features such as assimilators should be available w/o requiring EMF.

Plugin org.eclipse.gef3d.gmf re-exports the newly created plugin for convenience. 

The ecore and uml examples were adjusted accordingly, in order to adust your own code (in case of compile errors), you will have to:
- import org.eclipse.gef3d.ext.multieditor.emf for INestableEditorWithEditingDomain/ResourceSet
- in case your plugin does not require  org.eclipse.gef3d.gmf, you will have to add  org.eclipse.gef3d.ext.emf
Comment 3 Jens Von Pilgrim CLA 2010-06-06 06:02:29 EDT
Created attachment 171214 [details]
mylyn/context/zip