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

Bug 364967

Summary:	tycho-equinox/tycho-equinox-api names are confusing
Product:	z_Archived	Reporter:	Igor Fedorenko <igor>
Component:	Tycho	Assignee:	Tobias Oberlies <t-oberlies>
Status:	RESOLVED FIXED	QA Contact:
Severity:	minor
Priority:	P3
Version:	unspecified
Target Milestone:	---
Hardware:	All
OS:	All
Whiteboard:

Description Igor Fedorenko

2011-11-28 10:43:04 EST

These two modules provide generic Equinox/Plexus bridge and are not specific to Tycho. It would be helpful to at least to remove "tycho" from module names, but ideally move them out of Tycho main source tree.

Comment 1 Tobias Oberlies

2011-11-28 10:56:42 EST

+1 :-)

How about 
 tycho-equinox-api -> equinox-api
 tycho-equinox -> equinox-embedder

We could also add another prefix, e.g. to make it "plexus-equinox-api" or "maven-equinox-api", but I'm not sure about this.

Comment 2 Tobias Oberlies

2011-11-28 12:14:52 EST

On today's tycho dev phone call, we came up with the following proposals:
tycho-equinox-api -> sisu-equinox-api
tycho-equinox -> sisu-equinox-embedder
tycho-equinox-launching -> sisu-equinox-launching

Sisu is the re-implementation of plexus based on Google guice. The modules may move to Sisu eventually, but not right now. I'll ask for more feedback on tycho-dev, and probably do this change in the next couple of days.

Comment 3 Tobias Oberlies

2011-11-29 12:29:32 EST

Done with commit 704e4e8 [1] and 8f517bb [2].

[1] http://git.eclipse.org/c/tycho/org.eclipse.tycho.git/commit/?id=704e4e84e86f92451ba49f411e0734dba20ce289
[2] http://git.eclipse.org/c/tycho/org.eclipse.tycho.git/commit/?id=8f517bb0a707abb7095b089b320bc42efeadbbc8

Comment 4 Tobias Oberlies

2011-11-30 03:48:53 EST

Just for reference: This bug was created as a follow-up to a discussion on bug 364837. See comments 7 to 11 on that bug.