Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 363924 - Mylyn Reviews restructuring to create "Review for Eclipse (R4E)" sub-sub-project
Summary: Mylyn Reviews restructuring to create "Review for Eclipse (R4E)" sub-sub-project
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows Vista
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Alvaro Sanchez-Leon CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 364860 368836
  Show dependency tree
 
Reported: 2011-11-16 10:26 EST by Alvaro Sanchez-Leon CLA
Modified: 2012-01-24 17:55 EST (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alvaro Sanchez-Leon CLA 2011-11-16 10:26:19 EST
This task is created to discuss and refine the proposal to request the creation of the 
"Review for Eclipse Project (R4E)" as a sub-project of Mylyn Reviews.

The proposal html document is being sent to emo, The actual link will be posted in this task once it's available.
Comment 1 Steffen Pingel CLA 2011-11-16 14:44:58 EST
+1 this will provide more visibility to R4E as a full-blown standalone review tool implementation. R4E is not as tightly coupled with Mylyn's APIs and frameworks as other components in Mylyn Reviews and has been evolving more independently. A separate sub-sub project will provide R4E with more flexibility in terms of leadership, graduation, process and release scheduling and the restructuring should also facilitate building community around the project.
Comment 2 Alvaro Sanchez-Leon CLA 2011-11-28 13:09:52 EST
Some additional information as requested by EMO:

>What bits of functionality will stay in Reviews vs. what will be moved to the new subproject?
The current repository is divided into sub-folders. Moving r4e will pretty much require moving the bundles underneath the "o.e.mylyn.reviews/r4e" folder to a new repository.  
 Steffen suggests to use git filter-branch to split the single repository into two.

>I've noticed that the committers listed on the new project are all the active committers on Reviews. Will these committers all remain on Reviews as well, or are they moving? What is the implication for the parent project?

The proposal is that R4E committers remain committers under Mylyn Reviews as Ericsson and Tasktop have interest to evolve the reviews freamework and Gerrit connector.
  Mylyn Reviews will then continue to have committers from Tasktop, SAP, INSO and Ericsson. 

>When do you want to run the review?
I just submitted the IP log for approval, and mentioned and estimated time for the re-structuring review to December 5th.  Please let me know what you think.

>We'll need an IP Log approval prior to the review.
I will post an update as soon as I hear from the IP team.
Comment 3 Steffen Pingel CLA 2011-11-29 07:27:42 EST
(In reply to comment #2)
> >What bits of functionality will stay in Reviews vs. what will be moved to the
> new subproject?
> The current repository is divided into sub-folders. Moving r4e will pretty much
> require moving the bundles underneath the "o.e.mylyn.reviews/r4e" folder to a
> new repository.

Agreed. In addition I am proposing to move one more bundle to the new repository, i.e. these directories would move:

 framework/org.eclipse.mylyn.reviews.frame.core
 r4e/

All other directories would remain in the existing repository.
Comment 4 Wayne Beaton CLA 2011-11-29 10:20:38 EST
Sounds good. Can you make sure that this information is included in the review document? Bullet form will do.
Comment 5 Alvaro Sanchez-Leon CLA 2011-11-29 14:50:40 EST
Update:

* IP log has been approved and attached to parent bug 364860.
* The proposal has been updated as requested by Wayne and distributed by e-mail.
Comment 6 Alvaro Sanchez-Leon CLA 2011-12-28 01:08:43 EST
Review scheduled for Jan 4th 2012, see.

http://eclipse.org/projects/
Comment 7 Steffen Pingel CLA 2012-01-17 11:23:30 EST
The review has been approved! The new sub-sub-project now exists: http://www.eclipse.org/projects/project.php?id=mylyn.reviews.r4e.