Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 349439 - Unconvenient usage of Graphiti in RCP development
Summary: Unconvenient usage of Graphiti in RCP development
Status: CLOSED FIXED
Alias: None
Product: Graphiti
Classification: Modeling
Component: Core (show other bugs)
Version: 0.8.0   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 0.9.0   Edit
Assignee: Tim Kaiser CLA
QA Contact:
URL:
Whiteboard: Juno M1 theme_round_offs
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-15 09:58 EDT by Daniel Santos CLA
Modified: 2012-06-28 10:37 EDT (History)
2 users (show)

See Also:
michael.wenz: juno+

Attachments
mylyn/context/zip (1.96 KB, application/octet-stream)
2011-07-05 09:53 EDT, Tim Kaiser CLA 		no flags Details
Fix for 0.7.0 version (28.79 KB, patch)
2011-07-27 05:33 EDT, Daniel Santos CLA 		no flags Details | Diff
Show Obsolete (1) 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 Daniel Santos CLA 2011-06-15 09:58:08 EDT 
Build Identifier: 

As developer team we would like to create a graphical editor based on the Graphiti framework which enabled us to focus on our business case. By using the framework in our RCP application we are not able to get rid of the dependency of the plugin "org.eclipse.ide". Today our application it is very lightweight and we wanna keep it how it is. 

From our point of view, this requirement is a blocker.

Reproducible: Always
Comment 1 Michael Wenz CLA 2011-06-16 10:59:37 EDT 
IDE dependencies should be removed or seperated.
Comment 2 Tim Kaiser CLA 2011-07-05 04:21:15 EDT 
Hi,

is a dependency to org.eclipse.ui.workbench tolerable for your use case?

Best, Tim
Comment 3 Daniel Santos CLA 2011-07-05 04:43:41 EDT 
That is fine with me.
Comment 4 Tim Kaiser CLA 2011-07-05 09:53:04 EDT 
We only found three classes which need to be loaded from org.eclipse.ui.ide.
One of them (WorkspaceModifyOperation) could be eliminated.
The other two (ResourceTransfer, IFileEditorInput) can be loaded dynamically if there.
The dependency to org.eclipse.ui.ide is marked optional now.
Comment 5 Tim Kaiser CLA 2011-07-05 09:53:06 EDT 
Created attachment 199127 [details]
mylyn/context/zip
Comment 6 Michael Wenz CLA 2011-07-14 08:19:15 EDT 
Marked as part of Juno
Comment 7 Daniel Santos CLA 2011-07-27 05:33:19 EDT 
Created attachment 200434 [details]
Fix for 0.7.0 version
Comment 8 Tim Kaiser CLA 2011-08-10 05:52:22 EDT 
Hi Daniel,

thanks for the fix.
When reviewing your patch i see a lot of differences
introduced by new formatting of comments (line breaks).
Could you add a patch without formatting
comments differently? This would highly improve readability.

Best, Tim
Comment 9 Michael Wenz CLA 2012-04-11 10:28:26 EDT 
Bookkeeping: Set target release
Comment 10 Michael Wenz CLA 2012-06-28 10:37:39 EDT 
Part of Graphiti 0.9.0 (Eclipse Juno)