Community
Participate
Working Groups
We've got a request to remove CVS from (most) EPP packages (see bug 438297). During my investigation about the feasibility of this request I found out that the Mylyn features org.eclipse.mylyn.cvs and org.eclipse.mylyn.ide_feature define direct and/or indirect dependencies to CVS. Unfortunately (or: fortunately for you...) these features are included in many of the packages and prevent us from removing CVS. Is it possible to remove the CVS Mylyn integration from those features and move it into a separate feature? I reckon that CVS Mylyn support is not the most-wanted feature any more, and that it would be beneficial for all parties to remove it.
The org.eclipse.mylyn.cvs feature can easily be removed from EPP packages. The ide_feature is a bit more tricky: I'm in favor of removing the CVS dependency from that feature in the next Mylyn release. If we do that we should either EOL the CVS integration in Mylyn or create a separate feature for the org.eclipse.mylyn.team.cvs bundle to enable users to still install it. Sam, what do you think?
I'm in favour of creating a separate feature. I realize that usage of CVS is low, but we haven't had to do any maintenance on that integration for years so I think it's justifiable to keep it as long as there is some demand. Also, I hesitate to EOL the CVS integration since I think it is much more robust than any other SCM integration we have (perhaps because the Eclipse CVS integration is more robust than Subclipse and Subversive)...
Any progress here?
Well, the bug is assigned to you. Were you planning on making a contribution? If not, please reassign to default and we'll see if we can do this in time for Mars.
I don't know why it was assigned to me... maybe I was dreaming... or clicked the wrong button in the middle of the night... In any case I don't think I will have the time to work a contribution, but I would be very thankful if you could solve it for us in the Mars timeframe. Many thanks in advance!
I can't make any promises, but I'm tentatively putting this on the next milestone.
Thanks, that would be great.
Sam, let me know if you want me to take a pass at this.
Proposed feature (org.eclipse.mylyn.team.cvs): https://git.eclipse.org/r/44182
Sam, did you get a chance to look at the review?
No, I won't have a chance before next week.
Once the change is merged, we should do a snapshot build and verify that installation works as expected.
This review adds the feature to the main site: https://git.eclipse.org/r/#/c/47221/.
Both changes have been merged.
Great, thanks Steffen.
Steffen, since the CVS client is still available from http://download.eclipse.org/releases/mars, do you think we should add the new feature to mylyn.b3aggrcon?
That makes sense to me (although there have been complaints in the past that Mylyn contributes too many visible features).