Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 362997 - Multiple eSupertypes fail to compare
Summary: Multiple eSupertypes fail to compare
Status: CLOSED FIXED
Alias: None
Product: EMFCompare
Classification: Modeling
Component: Core (show other bugs)
Version: 2.7.0   Edit
Hardware: PC Windows Vista
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: EMF Compare CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 363224 365285
  Show dependency tree
 
Reported: 2011-11-06 11:27 EST by Ed Willink CLA
Modified: 2013-01-17 08:32 EST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ed Willink CLA 2011-11-06 11:27:59 EST
M2: When comparing a minor change to

org.eclipse.ocl\examples\org.eclipse.ocl.examples.xtext.base\model\BaseCST.ecore

wherever there is more than one supertype, the final one is reported as added and removed.
Comment 1 Laurent Goubet CLA 2011-11-07 02:49:51 EST
Haven't looked at it in details, but the description reminds me of bug 362351 .
Comment 2 Ed Willink CLA 2011-11-07 03:02:17 EST
(In reply to comment #1)
> Haven't looked at it in details, but the description reminds me of bug 362351 .

Yes. (A bug on an ecore file may be easier to debug than on a UML profile.)
Comment 3 Laurent Goubet CLA 2011-11-07 03:33:46 EST
> Yes. (A bug on an ecore file may be easier to debug than on a UML profile.)

Clearly ;). I stated that other bug here in order to remember that I should look at both when debugging this :).
Comment 4 Ed Willink CLA 2011-11-08 15:44:04 EST
This change now breaks the MDT/OCL Junit tests.

Build I201108090415 which was all that was available from the download page at M2 is ok.

1.3.0M3 is bad. Problem occurs on OCLEcore.ecore too,
Comment 5 Cedric Brun CLA 2011-11-23 12:07:34 EST
Thanks for your feedback. I added a testcase to detect this case. (org.eclipse.emf.compare.uml2.diff.test.TestUMLEcore) 

We now just have to make it green again ;)
Comment 6 Laurent Goubet CLA 2011-12-12 08:45:53 EST
Cedric's test case is green, the fix is commited on master. Now to try with the different sample models provided on the X different bugzillas :p.
Comment 7 Laurent Goubet CLA 2013-01-17 08:32:03 EST
batch-closing a bunch of "RESOLVED" bugs.