[egit-dev] Moving the EGit Mylyn connector (Was: Re: [mylyn-dev] Not an

[Thomas Wolf <twolf@xxxxxxxxxx>]

On 06.01.23 22:19 , Matthias Sohn wrote:
> We didn't yet discuss in the EGit project if we want to take up 
> maintenance again
> since Mylyn is revived or if we hand it over to the Mylyn project.
>
> @EGit committers: what do you think?

Sure, if Mylyn wants to maintain and improve it, that's perfectly fine
with me. We never did much with it. I think the only new development in
the past couple of years in the Mylyn connector was my changing the
repository set-up dialog to support GitHub PATs better.

However, see below:
* Keep org.eclipse.egit.github.core where it currently is; don't move
  that one to Mylyn.
* Also move org.eclipse.egit.mylyn.ui to Mylyn.

> On Fri, Jan 6, 2023 at 3:46 PM Wim Jongman <wim.jongman@xxxxxxxxx 
> <mailto:wim.jongman@xxxxxxxxx>> wrote:
>
>     Matthias, we are moving the EGit GitHub connector into Mylyn. WDYT?
>
>     PR is here:
>     https://github.com/eclipse-mylyn/org.eclipse.mylyn/pull/38
>     <https://github.com/eclipse-mylyn/org.eclipse.mylyn/pull/38>

This PR also copies org.eclipse.egit.github.core, which is **not** part
of the Mylyn connector. It is a completely separate thing and does not
depend on Mylyn or EGit at all.

I would prefer that one to stay where it is, in the
org.eclipse.egit.github repository. Reasons:

1. It is independent.
2. It uses a BREE of JAVA 8 for Android compatibility.
3. It should be versioned, built, tested, and published independently.

In particular it should be published to maven central.

Some technical things:

The connector makes use of EGit internal classes. EGit has removed it
from x-friends in MANIFEST.MF. Expect a lot of PDE warnings about
discouraged accesses.

I would expect that these accesses to internal classes would be analyzed
and be removed eventually. We'd have to figure out what stable APIs EGit
core and UI would have to provide instead.

EGit does not guarantee anything about internal classes.

Bundle org.eclipse.egit.mylyn.ui, which was removed in the EGit
repository in commit 12a21a6c, should probably also be moved to Mylyn.

There, too, we have internal accesses.

Cheers,

  Thomas