Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 366553 - EMF Compare doesn't allow editing model when opened in EGit Merge Tool
Summary: EMF Compare doesn't allow editing model when opened in EGit Merge Tool
Status: CLOSED FIXED
Alias: None
Product: EMFCompare
Classification: Modeling
Component: UI (show other bugs)
Version: 1.3   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: Kepler M5   Edit
Assignee: EMF Compare CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-13 10:26 EST by Nikolay Kasyanov CLA
Modified: 2013-01-23 07:29 EST (History)
2 users (show)

See Also:


Attachments
Repository to easily reproduce (54.18 KB, application/zip)
2013-01-18 06:44 EST, Laurent Goubet CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nikolay Kasyanov CLA 2011-12-13 10:26:10 EST
I've found that when EMF Compare opens in Merge Tool during EGit merge/rebase conflict resolution, there is no options to move changes from one model to another. I makes EMF Compare only capable of showing differences, but not capable to merge.

initial forum thread: http://www.eclipse.org/forums/index.php/mv/msg/262814/759184/#msg_759184
Comment 1 Laurent Goubet CLA 2013-01-18 06:44:25 EST
Created attachment 225812 [details]
Repository to easily reproduce

This bug is due to EGit using its own CompareEditorInput, and EMF Compare not being able to infer all necessary information from there.

It will not be fixed for 1.3. For 2.1, This bug is even worse as an NPE is thrown right at the opening of the merge tool.

Attached is a zipped git repository which only includes a single project "sandbox" and two branches : master and conflictual. Simply import "sandbox" into a workspace, checkout "conflictual" and try to rebase on "master". This will trigger the detection of a conflict on "sandbox/My.ecore". Trying to open the merge tool (right-click on conflictual file > Team > Merge Tool) on that ecore will trigger an NPE from EMF Compare.
Comment 2 Mikaël Barbero CLA 2013-01-22 10:14:29 EST
fixed in 1294c59f24d8450799661d58918fc82bc724fd11