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

Bug 331929

Summary:	[sync] Wrong resolving of common ancestor for new files
Product:	[Technology] EGit	Reporter:	Dariusz Luksza <dariusz.luksza>
Component:	UI	Assignee:	Dariusz Luksza <dariusz.luksza>
Status:	CLOSED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	caniszczyk, matthias.sohn, stefan.lay
Version:	0.9.0
Target Milestone:	0.10.0-M3
Hardware:	PC
OS:	Linux
Whiteboard:

Description Dariusz Luksza

2010-12-06 12:59:25 EST

This issue was discovered by Stefan Lay while he was testing fix for bug 326674.

Here is scenario for reproducing this issue:
* create and commit new file to repository
* make change in nearly created file and stage it
* launch synchronization with included local changes
When compare view for new this file is launched it shows conflicting change
because common ancestor is empty.

In this case common ancestor should be same as remote version. Therefore we
have an issue in algorithm of calculating common ancestor.

Comment 1 Dariusz Luksza

2010-12-06 15:51:49 EST

This patch set should fix this issue:
http://egit.eclipse.org/r/2050

Because we cannot configure RevWalk used in GitModelObjectContainer.calculateAncestor(RevCommit) so that it will return base commit for files that does not exist in parent commit. Therefore I decided to overwrite getAncestor() method in GitCacheCompareInput.

Comment 2 Dariusz Luksza

2010-12-15 14:09:51 EST

Fixed with patch 536566d1cac672f70da9ed7c3b97893629dae2aa