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

Bug 500253

Summary:	WorkspaceClassLoading should handle parameters whose type comes from dependencies declared by import-package
Product:	[Modeling] Sirius	Reporter:	Maxime Porhel <maxime.porhel>
Component:	Core	Assignee:	Maxime Porhel <maxime.porhel>
Status:	CLOSED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	laurent.redor, pierre-charles.david, pierre.guilet, steve.monnier
Version:	3.1.0	Keywords:	triaged
Target Milestone:	4.1.0
Hardware:	All
OS:	All
See Also:	https://git.eclipse.org/r/79707 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=729f6525c2b0519d3afe20b1fc0cf2df28654d2d https://git.eclipse.org/r/83420 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=46535e02c56964b313bd27ebe097a0c0893eeb77 https://git.eclipse.org/r/83913 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=621d888bfed8768a94baf5b39c1c3ce67e3529be https://git.eclipse.org/r/88100 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=7601456a9f51f9cbd3f3f92cf864d4c967b0efbf https://git.eclipse.org/r/88143 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=a0daaed694185518090f64ea93308e5ed662fab5 https://git.eclipse.org/r/124337
Whiteboard:

Description Maxime Porhel

2016-08-25 04:56:14 EDT

I have a NoClassDefError in the following situation : 
. I have a sample model and a VSP in my runtime
. the metamodels are in the dev workspace
. the VSP has a require-bundle to one metamodel plugin and an import-package to another one. 
. I have a service class which declares a service which use a type of the imported package. 

> The workspace class loading is not able to load the find the class from the imported package. 

If I put back the VSP in the dev worskpace, it seems I have no issue, the service is found.

Comment 1 Eclipse Genie

2016-08-25 05:04:27 EDT

New Gerrit change created: https://git.eclipse.org/r/79707

Comment 2 Pierre-Charles David

2016-08-29 08:12:06 EDT

Maxime, is there some word missing in the title?

Comment 3 Eclipse Genie

2016-08-30 03:59:59 EDT

Gerrit change https://git.eclipse.org/r/79707 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=729f6525c2b0519d3afe20b1fc0cf2df28654d2d

Comment 4 Pierre-Charles David

2016-09-08 10:30:18 EDT

For clarification, this is fixed, the only remaining work is to add the corresponding automated test.

Comment 5 Laurent Redor

2016-10-06 03:15:19 EDT

The tests auto will be made later (the keyword needtest is here to identify this remaining work).

Comment 6 Pierre Guilet

2016-10-18 04:32:47 EDT

Before fix, the problem occurred only when no direct dependency to org.eclipse.emf.ecore was declared in the VSP.

Comment 7 Eclipse Genie

2016-10-18 04:33:31 EDT

New Gerrit change created: https://git.eclipse.org/r/83420

Comment 8 Pierre-Charles David

2016-10-18 11:07:42 EDT

Available in Sirius 4.1.0, see https://wiki.eclipse.org/Sirius/4.1.0 for details.

Comment 9 Eclipse Genie

2016-10-19 11:57:24 EDT

Gerrit change https://git.eclipse.org/r/83420 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=46535e02c56964b313bd27ebe097a0c0893eeb77

Comment 10 Eclipse Genie

2016-10-26 04:24:33 EDT

New Gerrit change created: https://git.eclipse.org/r/83913

Comment 11 Eclipse Genie

2016-11-18 04:16:50 EST

Gerrit change https://git.eclipse.org/r/83913 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=621d888bfed8768a94baf5b39c1c3ce67e3529be

Comment 12 Eclipse Genie

2017-01-05 11:06:04 EST

New Gerrit change created: https://git.eclipse.org/r/88100

Comment 13 Eclipse Genie

2017-01-05 11:59:10 EST

Gerrit change https://git.eclipse.org/r/88100 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=7601456a9f51f9cbd3f3f92cf864d4c967b0efbf

Comment 14 Eclipse Genie

2017-01-06 03:31:50 EST

New Gerrit change created: https://git.eclipse.org/r/88143

Comment 15 Eclipse Genie

2017-01-06 03:40:12 EST

Gerrit change https://git.eclipse.org/r/88143 was merged to [v4.1.x].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=a0daaed694185518090f64ea93308e5ed662fab5

Comment 16 Eclipse Genie

2018-06-11 07:27:17 EDT

New Gerrit change created: https://git.eclipse.org/r/124337