eclipse / eavp

Eclipse advanced visualization project
Eclipse Public License 1.0
9 stars 15 forks source link

All source bundles (not tests) need Required Plug-ins cleaned up #121

Open jayjaybillings opened 8 years ago

jayjaybillings commented 8 years ago

We've gotten very lax on putting bundles in the "Required Plug-ins" part instead of the "Imported Packages" part of the Manifest. We should, to the extent possible, NEVER use Required Plug-ins.

SmithRWORNL commented 8 years ago

Fixed by commit 715f59185e7eeef2d73de1159f031a31d01d1fad.

SmithRWORNL commented 8 years ago

Scratch that, closed the wrong issue.

SmithRWORNL commented 8 years ago

In commit ca52bef197fe0b737773eb24ff688ce519fca47e I've tried to minimize the number of required bundles we have. The largest problems are the xtext packages, which I can't get working in the build without requiring the emf and xtext bundles.