This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 528004 - Add EGit to the SDK
Summary: Add EGit to the SDK
Status: RESOLVED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: IDE (show other bugs)
Version: 4.8   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Platform-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-12-01 04:38 EST by Andrey Loskutov CLA
Modified: 2017-12-06 03:21 EST (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Loskutov CLA 2017-12-01 04:38:32 EST
Now CVS is finally gone (bug 359466) and removed from SDK, what about to adding Git support to SDK?

It is kind of awkward to have IDE which does not have Git support by default. I know we have all those epp packages, but I mean that the Git support is a basic functionality one needs if one uses SDK, so it should be part of the SDK.

I honestly don't know if this is possible from the release engineering point of view, but just wanted to start a discussion.
Comment 1 Lars Vogel CLA 2017-12-01 05:08:37 EST
+1
Comment 2 Lars Vogel CLA 2017-12-01 05:09:02 EST
Definitely possible, we pull in other dependencies like EMF.
Comment 3 Dani Megert CLA 2017-12-01 05:28:06 EST
We've already discussed this at length in the PMC when we discussed the removal of CVS. After removing CVS we don't want to have a team provider in the Platform. It is up to each EPP to decide which team provider(s) to include.
Comment 4 Dani Megert CLA 2017-12-01 05:31:10 EST
NOTE: If you are using the SDK and not the EPP, then you can install EGit into a shared location. That way you don't have to install EGit into every new SDK. That's what I do for years.
Comment 5 Lars Vogel CLA 2017-12-01 05:45:33 EST
(In reply to Dani Megert from comment #3)
> We've already discussed this at length in the PMC when we discussed the
> removal of CVS. 

+1

> After removing CVS we don't want to have a team provider in
> the Platform. 

I don't remember that we discussed this part. https://wiki.eclipse.org/Eclipse/PMC Wiki only says:

----
Discussion if CVS should be removed from the SDK build, Alex +1, Lars +1, Dani +1
----

I suggest we discuss this in the next PMC call.
Comment 6 Dani Megert CLA 2017-12-01 06:07:47 EST
(In reply to Lars Vogel from comment #5)
> > After removing CVS we don't want to have a team provider in
> > the Platform. 
> 
> I don't remember that we discussed this part.
> https://wiki.eclipse.org/Eclipse/PMC Wiki only says:

You need to go back in years. When we discussed the removal of PDE Build we agreed that when we remove a technology we won't replace it with another one. EGit was mentioned and I'm sure it's in the meeting notes.


Also EGit is +1 in the train and is using other +1 projects. Investing time to bring EGit to 0 while we already have the IDE and other packages containing EGit seems a waste of time.
Comment 7 Lars Vogel CLA 2017-12-01 06:14:43 EST
(In reply to Dani Megert from comment #6)
> (In reply to Lars Vogel from comment #5)
> > > After removing CVS we don't want to have a team provider in
> > > the Platform. 
> > 
> > I don't remember that we discussed this part.
> > https://wiki.eclipse.org/Eclipse/PMC Wiki only says:
> 
> You need to go back in years. 

In this case, let's discuss again, thing may have changed in the the last years. IIRC we discussed in the not so long past, that we want to have at least one consumer of the team code in the SDK. This used to be a blocker to remove CVS.

> Also EGit is +1 in the train and is using other +1 projects. Investing time
> to bring EGit to 0 while we already have the IDE and other packages
> containing EGit seems a waste of time.

EMF is also +1 or +2 AFAIK, so moving it to 0 is not necessary.
Comment 8 Dani Megert CLA 2017-12-06 03:21:06 EST
We discussed this yesterday:
https://wiki.eclipse.org/Eclipse/PMC#Meeting_Minutes