eclipse-sisu / sisu-project

Sisu Inject
https://eclipse.dev/sisu/
Eclipse Public License 2.0
18 stars 15 forks source link

Remove tycho #71

Closed cstamas closed 1 year ago

cstamas commented 1 year ago

Removes tycho completely, project is now two modules:

Builds upon https://github.com/eclipse/sisu.inject/pull/61

cstamas commented 1 year ago

@mcculls @kwin @laeubi ping

As I said, this PR "builds upon" https://github.com/eclipse/sisu.inject/pull/61 (adds +1 commit), so we can figure out what we want, but this change makes sisu "plain java" project now.

cstamas commented 1 year ago

I left sisu manifest, but am unsure is it needed (for consumers in eclipse)?

kwin commented 1 year ago

We should keep the OSGi bundle headers but let us use bnd-maven-plugin to generate them automatically from pom.xml and sources.

cstamas commented 1 year ago

@kwin I'd leave that to you, am not at home in osgi stuff...

mcculls commented 1 year ago

I can look over the OSGi part - previously we had 2 artifacts, one main jar and one extender whose manifest listed the activator that kick-started dynamic wiring of bundles. This was so people could choose to deploy it or not. We could always make that configurable via a system property if we want to flatten the project completely to a single module (although keeping at least one level in place does make it easier to add extra assemblies or BOM modules later on)

cstamas commented 1 year ago

So, IMHO let's move :smile: Lets merge this one and continue step-by-step?

kwin commented 1 year ago

@cstamas Based on your work I created #72 which removes the manual MANIFEST.MF

cstamas commented 1 year ago

Superseded by https://github.com/eclipse/sisu.inject/pull/72